Re: Updates makelsr to point to lilypond exe (issue 5489134)

2012-01-04 Thread graham
ok, I'm fine with removing the -V. http://codereview.appspot.com/5489134/diff/1/scripts/auxiliar/makelsr.py File scripts/auxiliar/makelsr.py (right): http://codereview.appspot.com/5489134/diff/1/scripts/auxiliar/makelsr.py#newcode184 scripts/auxiliar/makelsr.py:184: e = os.system (lilypondexe +

Re: Updates makelsr to point to lilypond exe (issue 5489134)

2012-01-04 Thread Phil Holmes
- Original Message - From: To: ; Cc: ; Sent: Monday, January 02, 2012 9:11 PM Subject: Re: Updates makelsr to point to lilypond exe (issue 5489134) great idea! A few implementation quibbles, though. Thought I'd responded to this, but can't find a record of doing s

Re: Updates makelsr to point to lilypond exe (issue 5489134)

2012-01-02 Thread graham
great idea! A few implementation quibbles, though. http://codereview.appspot.com/5489134/diff/1/scripts/auxiliar/makelsr.py File scripts/auxiliar/makelsr.py (right): http://codereview.appspot.com/5489134/diff/1/scripts/auxiliar/makelsr.py#newcode76 scripts/auxiliar/makelsr.py:76: lilypondexe=c

Updates makelsr to point to lilypond exe (issue 5489134)

2012-01-02 Thread PhilEHolmes
Reviewers: percivall, Message: Please review Description: The CG discussing makelsr says "Make sure that convert-ly and lilypond commands in current PATH are in a bleeding edge version" but in practice the it constructs a path to convert-ly, but not to the lilypond exe. The only indication that