Re: convert-ly: option to only modify changed files. (issue2642041)

2010-10-25 Thread percival . music . ca
On 2010/10/24 10:50:52, Carl wrote: LGTM. Thanks, pushed. http://codereview.appspot.com/2642041/ ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: convert-ly: option to only modify changed files. (issue2642041)

2010-10-24 Thread Carl . D . Sorensen
LGTM. This is a feature that has been needed. Thanks for working on it. Carl http://codereview.appspot.com/2642041/ ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: convert-ly: option to only modify changed files. (issue2642041)

2010-10-21 Thread percival . music . ca
New patch: it still updates the version for a stable release. running convert-ly -d -eon: \version "2.11.0" { c'4 } produces: \version "2.12.0" { c'4 } which is exactly what we want for the regtests. http://codereview.appspot.com/2642041/ ___

Re: convert-ly: option to only modify changed files. (issue2642041)

2010-10-21 Thread percival . music . ca
On 2010/10/21 08:09:23, Graham Percival wrote: Pretty simple, but useful. argh, I meant to add: I'm not overly confident about the -d letter, nor the --diff-version-update name. Better suggestions are welcome. http://codereview.appspot.com/2642041/ ___

convert-ly: option to only modify changed files. (issue2642041)

2010-10-21 Thread percival . music . ca
Reviewers: , Message: Pretty simple, but useful. Description: convert-ly: option to only modify changed files. This useful for the regtests -- with the extra -d flag, we will only change the \version string if the actual contents of the file has changed. This lets us see which tests were modif