search

grame-cncm / libmusicxml

A C/C++ library to support the MusicXML format.
Mozilla Public License 2.0
152 stars 33 forks source link

Make xml2lilypond's --help and --version options emit data to stdout #17

Closed lemzwerg closed 4 years ago

lemzwerg commented 6 years ago

On Unix platforms it is common that the output of --help and --version is sent to stdout. Reason is that you then can easily pipe the output to a pager, e.g., xml2lilypond --help | less.

Currently, it is sent to stderr.

Additionally, for --help, the final message

---No arguments have been supplied, exiting. ---

should be suppressed.

jacques-menu commented 6 years ago

Le 31 oct. 2017 à 14:22, lemzwerg notifications@github.com a écrit :

On Unix platforms it is common that the output of --help and --version is sent to stdout. Reason is that you then can easily pipe the output to a pager, e.g., xml2lilypond --help | less.

Currently, it is sent to stderr.

Additionally, for --help, the final message

---No arguments have been supplied, exiting. --- should be suppressed.

OK, I’ll do that and merge with you changes.

I’ve never done a git merge yet.vWhat are the difficulties that may arise?

The changes I made to the library are:

JM

dfober commented 6 years ago

Le 31 oct. 2017 à 16:35, jacques-menu notifications@github.com a écrit :

Le 31 oct. 2017 à 14:22, lemzwerg notifications@github.com a écrit :

On Unix platforms it is common that the output of --help and --version is sent to stdout. Reason is that you then can easily pipe the output to a pager, e.g., xml2lilypond --help | less.

Currently, it is sent to stderr.

Additionally, for --help, the final message

---No arguments have been supplied, exiting. --- should be suppressed.

OK, I’ll do that and merge with you changes.

I’ve never done a git merge yet.vWhat are the difficulties that may arise?

you don’t have to merge for changes on your branch. Just make a

git pull to update your local repository.

The changes I made to the library are:

  • adding the input line number to xmlelement, crucial;
  • adding to rational, that should be no problem.

JM

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/grame-cncm/libmusicxml/issues/17#issuecomment-340801688, or mute the thread https://github.com/notifications/unsubscribe-auth/AAaTw1GoyxIq2pNQ504ub0cS2hAyit-6ks5sxz42gaJpZM4QMusV.

dfober commented 4 years ago

@jacques-menu can we close this issue ? (I guess yes)

jacques-menu commented 4 years ago

Le 27 mai 2020 à 21:04, Dominique Fober notifications@github.com a écrit : @jacques-menu https://github.com/jacques-menu can we close this issue ? (I guess yes)

Yes, we can!

JM

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/grame-cncm/libmusicxml/issues/17#issuecomment-634876938, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRJX6SNRO6W4CTHVG2E6LTRTVP5PANCNFSM4EBS5MKQ.