[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] MIDI command line option
From: |
Neil Puttock |
Subject: |
Re: [PATCH] MIDI command line option |
Date: |
Thu, 28 Aug 2008 01:08:54 +0100 |
2008/8/6 Neil Puttock <address@hidden>:
> 2008/8/5 Han-Wen Nienhuys <address@hidden>:
>> There is also some code in lily.scm that checks for a windows system.
>> It's easier to add to there rather than the .nsh
>
> I've tried setting 'midi-extension here:
>
> (case PLATFORM
> ((windows)
> ;; change midi default for Windows
> (ly:set-option 'midi-extension "mid")
> (define native-getcwd getcwd)
> (define (slashify x)
> (if (string-index x #\\)
> x
> (string-regexp-substitute
> "//*" "/"
> (string-regexp-substitute "\\\\" "/" x))))
> ;; FIXME: this prints a warning.
> (define-public (ly-getcwd)
> (slashify (native-getcwd))))
> (else (define-public ly-getcwd getcwd)))
>
> Since I can't check whether this works on Windows, I temporarily moved
> the option setting to the else clause. Unfortunately, though this
> overrides the default, setting midi-extension via the command line no
> longer works.
OK, I've had another look at this.
The command line options using -d are set near the top of the file,
96 (for-each
97 (lambda (x)
98 (ly:set-option (car x) (cdr x)))
99 (eval-string (ly:command-line-options)))
whereas the entry point code evaluates a Scheme expression using -e.
Can the code which checks for a windows system be moved above the
command line setting?
- Re: [PATCH] MIDI command line option, Han-Wen Nienhuys, 2008/08/04
- Re: [PATCH] MIDI command line option, Mats Bengtsson, 2008/08/05
- Re: [PATCH] MIDI command line option, Han-Wen Nienhuys, 2008/08/05
- Re: [PATCH] MIDI command line option, Neil Puttock, 2008/08/06
- Re: [PATCH] MIDI command line option,
Neil Puttock <=
- Re: [PATCH] MIDI command line option, Han-Wen Nienhuys, 2008/08/28
- Re: [PATCH] MIDI command line option, Neil Puttock, 2008/08/29
- Re: [PATCH] MIDI command line option, Han-Wen Nienhuys, 2008/08/29
- Re: [PATCH] MIDI command line option, Neil Puttock, 2008/08/29
- Re: [PATCH] MIDI command line option, Han-Wen Nienhuys, 2008/08/29
- Re: [PATCH] MIDI command line option, Neil Puttock, 2008/08/30