Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-23 Thread lilypond

Updates:
Labels: -Patch-review Patch-new

Comment #14 on issue 2216 by gra...@percival-music.ca: AU: Document all  
options for lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216

Sorry, I'm changing this to patch-new so that Marek can practice on it.   
The patch itself is fine.



___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-23 Thread lilypond

Updates:
Labels: Patch-review

Comment #15 on issue 2216 by d...@gnu.org: AU: Document all options for  
lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216#c15

Patchy the autobot says: LGTM.


___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-22 Thread lilypond

Updates:
Labels: Patch-new

Comment #12 on issue 2216 by pkx1...@gmail.com: AU: Document all options  
for lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216#c12

Doc: AU - document -dhelp CLI options

Issue 2216

Add new @node for the -d[Scheme_val] option.
Renamed old node so that it is distinguishable from the
cli using sceheme variables.

Moved all the original -d[symbol] information over and added
some @ref{} for different options.

Other minor syntax changes and spacing edits on surrounding text.

http://codereview.appspot.com/5843060


___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-20 Thread James
Ian

On 19 March 2012 13:45, Ian Hulin i...@hulin.org.uk wrote:
 Hi Phil,
 On 19/03/12 11:31, lilyp...@googlecode.com wrote:

 Comment #4 on issue 2216 by pkx1...@gmail.com: AU: Document all
 options for lilypond -dhelp
 http://code.google.com/p/lilypond/issues/detail?id=2216

 I noticed Graham has some commments. I'd also like to make sure
 that my terminology is correct.

 I.e. the command

 -d[symbol]=[value] or --define-default=[symbol]=[value]

 is this a Scheme 'symbol' or something else?

 I'd just like it to use the correct terminology.
 'symbol' ==  a command-line option name which is handled at scheme
 level rather from within the C++ code.

 Quite often they are boolean flags so they default to using #t or #f
 (because they are handled using scheme).  In other cases, like
 loglevel, they can use a whole set of string values.

 Maybe this?
 -d[option-name]=[value] or --define-default=[option-name]=[value]


Thanks, that helps a lot.

-- 
--

James

___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-20 Thread lilypond

Updates:
Labels: Patch-new

Comment #8 on issue 2216 by pkx1...@gmail.com: AU: Document all options for  
lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216#c8

Doc: AU - document -dhelp CLI options

Issue 2216

Add new @node for the -d[Scheme_val] option.
Renamed old node so that it is distinguishable from the
cli using sceheme variables.

Moved all the original -d[symbol] information over and added
some @ref{} for different options.

Other minor syntax changes and spacing edits on surrounding text.

http://codereview.appspot.com/5843060


___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-20 Thread lilypond

Updates:
Labels: Patch-needs_work

Comment #9 on issue 2216 by d...@gnu.org: AU: Document all options for  
lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216#c9

Patchy the autobot says: Patch does not apply to master.


___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-20 Thread lilypond


Comment #10 on issue 2216 by pkx1...@gmail.com: AU: Document all options  
for lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216

David,

I have just applied my patch to current master with no issue

--snip--
First, rewinding head to replay your work on top of it...
Fast-forwarded master to 7a0c49b741de8aa998e17c310d7a4703d7b1712c.
james@DellVM:~/lilypond-git$ git  
am ../Dropbox/LPPatches/0002-Doc-AU-document-dhelp-CLI-options.patch

Applying: Doc: AU - document -dhelp CLI options
james@DellVM:~/lilypond-git$
--snip--

git-cl always complains if you haven't done a 'pull' before uploading a  
patch (dirty tree or something). So have I missed something or is the  
problem on your master?


James


___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-20 Thread lilypond


Comment #11 on issue 2216 by d...@gnu.org: AU: Document all options for  
lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216

It is obvious that your patch on Dropbox does not correspond to the  
latest patch uploaded to the Rietveld issue (I checked using the same  
master commit you just specified and downloaded the latest patch from  
Rietveld manually):
dak@lola:/usr/local/tmp/lilypond$ git checkout  
7a0c49b741de8aa998e17c310d7a4703d7b1712c

Note: checking out '7a0c49b741de8aa998e17c310d7a4703d7b1712c'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b new_branch_name

HEAD is now at 7a0c49b... Doc: CG Typo of Typo
dak@lola:/usr/local/tmp/lilypond$ git apply /tmp/issue5843060_2002.diff
error: patch failed: Documentation/contributor/administration.itexi:330
error: Documentation/contributor/administration.itexi: patch does not apply



___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-19 Thread lilypond

Updates:
Labels: Patch-review

Comment #3 on issue 2216 by d...@gnu.org: AU: Document all options for  
lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216#c3

Patchy the autobot says: LGTM.  But test-patches does not do a doc build.   
Given sufficient horsepower, it might be prudent to do a make doc before  
pushing to staging.



___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-19 Thread lilypond


Comment #4 on issue 2216 by pkx1...@gmail.com: AU: Document all options for  
lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216

I noticed Graham has some commments. I'd also like to make sure that my  
terminology is correct.


I.e. the command

-d[symbol]=[value] or --define-default=[symbol]=[value]

is this a Scheme 'symbol' or something else?

I'd just like it to use the correct terminology.


___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-19 Thread lilypond


Comment #5 on issue 2216 by elu...@gmail.com: AU: Document all options for  
lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216

are you sure symbol is correct? imo it should be variable or var:  
this variable is assigned a value.



___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-19 Thread lilypond


Comment #6 on issue 2216 by pkx1...@gmail.com: AU: Document all options for  
lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216

No I am not sure it is correct, however 'variable' isn't that helpful  
either. Hence my request from the devs to point me in the right direction  
if they can.



___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-19 Thread lilypond


Comment #7 on issue 2216 by d...@gnu.org: AU: Document all options for  
lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216

Well, that's what happens if you say (set! some-symbol value): the variable  
addressed by the interned symbol some-symbol is assigned a value.


The respective code in main.cc is:
case 'd':
  {
string arg (option_parser-optional_argument_str0_);
ssize eq = arg.find ('=');

string key = arg;
string val = #t;

if (eq != NPOS)
  {
key = arg.substr (0, eq);
val = arg.substr (eq + 1, arg.length () - 1);
  }

init_scheme_variables_global
+= (cons \' + key +  ' + val + )\n;
  }

This is somewhat disconcerting since init_scheme_variables_global is  
augmented with text, meaning that it is reasonably easy to sneak in  
arbitrary code here by supplying a key containing spaces.  To me, it would  
seem reasonably to have the whole of init_scheme_variables_global not  
evaluated but just read, in which case the expression would rather be

+= (  + key +  .  + val +  )\n;
of course, if unmatched parens or quotes are contained in key and val, one  
can again meddle with what is happening here.  Instead of building strings  
here, directly building a Scheme expression would likely make more sense.



___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-19 Thread Ian Hulin
Hi Phil,
On 19/03/12 11:31, lilyp...@googlecode.com wrote:
 
 Comment #4 on issue 2216 by pkx1...@gmail.com: AU: Document all
 options for lilypond -dhelp 
 http://code.google.com/p/lilypond/issues/detail?id=2216
 
 I noticed Graham has some commments. I'd also like to make sure
 that my terminology is correct.
 
 I.e. the command
 
 -d[symbol]=[value] or --define-default=[symbol]=[value]
 
 is this a Scheme 'symbol' or something else?
 
 I'd just like it to use the correct terminology.
'symbol' ==  a command-line option name which is handled at scheme
level rather from within the C++ code.

Quite often they are boolean flags so they default to using #t or #f
(because they are handled using scheme).  In other cases, like
loglevel, they can use a whole set of string values.

Maybe this?
-d[option-name]=[value] or --define-default=[option-name]=[value]

Cheers,
Ian

___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-18 Thread lilypond

Updates:
Summary: AU: Document all options for lilypond -dhelp
Labels: -Needs-policy

Comment #1 on issue 2216 by pkx1...@gmail.com: AU: Document all options for  
lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216

(No comment was entered for this change.)


___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: Issue 2216 in lilypond: AU: Document all options for lilypond -dhelp

2012-03-18 Thread lilypond

Updates:
Status: Started
Owner: pkx1...@gmail.com
Labels: Patch-new

Comment #2 on issue 2216 by pkx1...@gmail.com: AU: Document all options for  
lilypond -dhelp

http://code.google.com/p/lilypond/issues/detail?id=2216

Doc: AU - document -dhelp CLI options

Issue 2216

Add new @node for the -d[Scheme_val] option.
Renamed old node so that it is distinguishable from the
cli using sceheme variables.

Moved all the original -d[symbol] information over and added
some @ref{} for different options.

Other minor syntax changes and spacing edits on surrounding text.

http://codereview.appspot.com/5843060

Passes make and make doc


___
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: lilypond -dhelp

2008-09-23 Thread James E. Bailey
Apparently, if I use the instructions on making a link to the  
lilypond binary, it works. If I just call lilypond -dhelp from  
within the folder Lilypond.app/Contents/Resources/bin/ it doesn't work.

On Sep 22, 2008, at 5:07 PM, James E. Bailey wrote:

 Yeah, still same problem. I even moved the program to the  
 Applications directory.
 vanessa_imac:/Applications/LilyPond.app/Contents/Resources/bin  
 jamesebailey$ ./lilypond -dhelp
 GNU LilyPond 2.11.59
 warning: not relocating, no 2.11.59/ or current/ found under / 
 Applications/LilyPond.app/Contents/Resources/bin/share/lilypond/
 ERROR: In procedure primitive-load-path:
 ERROR: Unable to find file lily.scm in load path

 And to be fair, it makes sense, the share/lilypond/current isn't in  
 bin/, it's in Resources/.
 I guess I'll try making a link and see if that solves the problem.

 …sigh…

 On Tuesday, September 23, 2008, at 01:58AM, Graham Percival  
 [EMAIL PROTECTED] wrote:
 Interesting.  Works here on 10.4.11 intel.

 tsubasa:~ gperciva$ lilypond -dhelp
 GNU LilyPond 2.11.59
 Options supported by ly:set-option

  anti-alias-factor (1)
 ...


 Could you try deleting the LilyPond.app and installing it again?

 Cheers,
 - Graham

 On Tue, 23 Sep 2008 01:49:38 +0200
 James E. Bailey [EMAIL PROTECTED] wrote:

 On osx 10.4, using the 2.11.59-1 intel binary, I get the following:
 vanessa_imac:~/Desktop/LilyPond.app/Contents/Resources/bin
 jamesebailey$ ./lilypond -dhelp GNU LilyPond 2.11.59
 warning: not relocating, no 2.11.59/ or current/ found
 under /Users/jamesebailey/Desktop/LilyPond.app/Contents/Resources/ 
 bin/share/lilypond/
 ERROR: In procedure primitive-load-path: ERROR: Unable to find file
 lily.scm in load path

 And then I don't get the -dhelp options.


 ___
 bug-lilypond mailing list
 bug-lilypond@gnu.org
 http://lists.gnu.org/mailman/listinfo/bug-lilypond




 ___
 lilypond-devel mailing list
 [EMAIL PROTECTED]
 http://lists.gnu.org/mailman/listinfo/lilypond-devel




___
bug-lilypond mailing list
bug-lilypond@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-lilypond


lilypond -dhelp

2008-09-22 Thread James E. Bailey
On osx 10.4, using the 2.11.59-1 intel binary, I get the following:
vanessa_imac:~/Desktop/LilyPond.app/Contents/Resources/bin jamesebailey$ 
./lilypond -dhelp
GNU LilyPond 2.11.59
warning: not relocating, no 2.11.59/ or current/ found under 
/Users/jamesebailey/Desktop/LilyPond.app/Contents/Resources/bin/share/lilypond/
ERROR: In procedure primitive-load-path:
ERROR: Unable to find file lily.scm in load path

And then I don't get the -dhelp options.


___
bug-lilypond mailing list
bug-lilypond@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: lilypond -dhelp

2008-09-22 Thread Graham Percival
Interesting.  Works here on 10.4.11 intel.

tsubasa:~ gperciva$ lilypond -dhelp
GNU LilyPond 2.11.59
Options supported by ly:set-option

  anti-alias-factor (1)
...


Could you try deleting the LilyPond.app and installing it again?

Cheers,
- Graham

On Tue, 23 Sep 2008 01:49:38 +0200
James E. Bailey [EMAIL PROTECTED] wrote:

 On osx 10.4, using the 2.11.59-1 intel binary, I get the following:
 vanessa_imac:~/Desktop/LilyPond.app/Contents/Resources/bin
 jamesebailey$ ./lilypond -dhelp GNU LilyPond 2.11.59
 warning: not relocating, no 2.11.59/ or current/ found
 under 
 /Users/jamesebailey/Desktop/LilyPond.app/Contents/Resources/bin/share/lilypond/
 ERROR: In procedure primitive-load-path: ERROR: Unable to find file
 lily.scm in load path
 
 And then I don't get the -dhelp options.
 
 
 ___
 bug-lilypond mailing list
 bug-lilypond@gnu.org
 http://lists.gnu.org/mailman/listinfo/bug-lilypond


___
bug-lilypond mailing list
bug-lilypond@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-lilypond


Re: lilypond -dhelp

2008-09-22 Thread James E. Bailey
Yeah, still same problem. I even moved the program to the Applications 
directory.
vanessa_imac:/Applications/LilyPond.app/Contents/Resources/bin jamesebailey$ 
./lilypond -dhelp
GNU LilyPond 2.11.59
warning: not relocating, no 2.11.59/ or current/ found under 
/Applications/LilyPond.app/Contents/Resources/bin/share/lilypond/
ERROR: In procedure primitive-load-path:
ERROR: Unable to find file lily.scm in load path

And to be fair, it makes sense, the share/lilypond/current isn't in bin/, it's 
in Resources/.
I guess I'll try making a link and see if that solves the problem.

…sigh…
 
On Tuesday, September 23, 2008, at 01:58AM, Graham Percival [EMAIL 
PROTECTED] wrote:
Interesting.  Works here on 10.4.11 intel.

tsubasa:~ gperciva$ lilypond -dhelp
GNU LilyPond 2.11.59
Options supported by ly:set-option

  anti-alias-factor (1)
...


Could you try deleting the LilyPond.app and installing it again?

Cheers,
- Graham

On Tue, 23 Sep 2008 01:49:38 +0200
James E. Bailey [EMAIL PROTECTED] wrote:

 On osx 10.4, using the 2.11.59-1 intel binary, I get the following:
 vanessa_imac:~/Desktop/LilyPond.app/Contents/Resources/bin
 jamesebailey$ ./lilypond -dhelp GNU LilyPond 2.11.59
 warning: not relocating, no 2.11.59/ or current/ found
 under 
 /Users/jamesebailey/Desktop/LilyPond.app/Contents/Resources/bin/share/lilypond/
 ERROR: In procedure primitive-load-path: ERROR: Unable to find file
 lily.scm in load path
 
 And then I don't get the -dhelp options.
 
 
 ___
 bug-lilypond mailing list
 bug-lilypond@gnu.org
 http://lists.gnu.org/mailman/listinfo/bug-lilypond




___
bug-lilypond mailing list
bug-lilypond@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-lilypond