Uses convert-ly to update changes stemming from the creation of a Flag grob.

Uses convert-ly to update changes stemming from the creation of a Flag grob.

[dak, 2011-10-24 21:41:04]

Seems like my suggestion to update the version numbers of files with manual
fixes (in a separate commit) in order not to get them updated again got lost, as
well as the bit about rereading the diff for problems.

http://codereview.appspot.com/5309059/diff/1/Documentation/es/learning/tweaks...
File Documentation/es/learning/tweaks.itely (right):

http://codereview.appspot.com/5309059/diff/1/Documentation/es/learning/tweaks...
Documentation/es/learning/tweaks.itely:3637: \once \override Flag #'transparent
= ##t
Doubled Flag override here.

http://codereview.appspot.com/5309059/diff/1/Documentation/es/notation/ancien...
File Documentation/es/notation/ancient.itely (right):

http://codereview.appspot.com/5309059/diff/1/Documentation/es/notation/ancien...
Documentation/es/notation/ancient.itely:2702: \override Flag #'transparent = ##t
Doubled Flag override here.

http://codereview.appspot.com/5309059/diff/1/Documentation/es/notation/changi...
File Documentation/es/notation/changing-defaults.itely (right):

http://codereview.appspot.com/5309059/diff/1/Documentation/es/notation/changi...
Documentation/es/notation/changing-defaults.itely:852: \override Flag
#'transparent = ##t
Doubled Flag override here.

http://codereview.appspot.com/5309059/diff/1/Documentation/es/notation/changi...
Documentation/es/notation/changing-defaults.itely:876: \override Flag
#'transparent = ##t
Doubled Flag override here.

http://codereview.appspot.com/5309059/diff/1/Documentation/hu/usage/lilypond-...
File Documentation/hu/usage/lilypond-book.itely (right):

http://codereview.appspot.com/5309059/diff/1/Documentation/hu/usage/lilypond-...
Documentation/hu/usage/lilypond-book.itely:11: @c \version "2.14.0"
Strange: version number change (to an unrelated version) but nothing else.  Not
necessarily your fault.

http://codereview.appspot.com/5309059/diff/1/Documentation/it/usage/external....
File Documentation/it/usage/external.itely (right):

http://codereview.appspot.com/5309059/diff/1/Documentation/it/usage/external....
Documentation/it/usage/external.itely:11: @c \version "2.14.0"
Another unexpected version number change.

http://codereview.appspot.com/5309059/diff/1/Documentation/it/usage/lilypond-...
File Documentation/it/usage/lilypond-book.itely (right):

http://codereview.appspot.com/5309059/diff/1/Documentation/it/usage/lilypond-...
Documentation/it/usage/lilypond-book.itely:11: @c \version "2.14.0"
Another.

http://codereview.appspot.com/5309059/diff/1/Documentation/ja/notation/changi...
File Documentation/ja/notation/changing-defaults.itely (right):

http://codereview.appspot.com/5309059/diff/1/Documentation/ja/notation/changi...
Documentation/ja/notation/changing-defaults.itely:898: \override Flag
#'transparent = ##t
Doubled Flag override.

http://codereview.appspot.com/5309059/diff/1/Documentation/ja/notation/changi...
Documentation/ja/notation/changing-defaults.itely:923: \override Flag
#'transparent = ##t
Doubled Flag override.

http://codereview.appspot.com/5309059/diff/1/Documentation/learning/tweaks.itely
File Documentation/learning/tweaks.itely (right):

http://codereview.appspot.com/5309059/diff/1/Documentation/learning/tweaks.it...
Documentation/learning/tweaks.itely:3471: \once \override Flag #'transparent =
##t
Doubled Flag override

http://codereview.appspot.com/5309059/diff/1/Documentation/learning/tweaks.it...
Documentation/learning/tweaks.itely:3489: \once \override Flag #'transparent =
##t
Doubled Flag override.

http://codereview.appspot.com/5309059/diff/1/Documentation/snippets/new/guita...
File Documentation/snippets/new/guitar-slides.ly (right):

http://codereview.appspot.com/5309059/diff/1/Documentation/snippets/new/guita...
Documentation/snippets/new/guitar-slides.ly:21: \once \override Flag
#'transparent = ##t
Doubled Flag override.

http://codereview.appspot.com/5309059/diff/1/Documentation/web/ly-examples/ca...
File Documentation/web/ly-examples/cary.ly (right):

http://codereview.appspot.com/5309059/diff/1/Documentation/web/ly-examples/ca...
Documentation/web/ly-examples/cary.ly:108: \override Flag #'stroke-style =
#"grace"
Unrelated spacing fix.

http://codereview.appspot.com/5309059/diff/1/Documentation/web/ly-examples/ca...
Documentation/web/ly-examples/cary.ly:251: \override Flag #'stroke-style =
#"grace"
Unrelated spacing fix.

http://codereview.appspot.com/5309059/diff/1/input/regression/grid-lines.ly
File input/regression/grid-lines.ly (right):

http://codereview.appspot.com/5309059/diff/1/input/regression/grid-lines.ly#n...
input/regression/grid-lines.ly:77: \override Flag #'transparent = ##t
Doubled Flag override.

http://codereview.appspot.com/5309059/diff/1/ly/gregorian.ly
File ly/gregorian.ly (right):

http://codereview.appspot.com/5309059/diff/1/ly/gregorian.ly#newcode304
ly/gregorian.ly:304: \override Flag #'transparent = ##t
Doubled Flag override.

http://codereview.appspot.com/5309059/diff/1/ly/property-init.ly
File ly/property-init.ly (right):

http://codereview.appspot.com/5309059/diff/1/ly/property-init.ly#newcode236
ly/property-init.ly:236: \override Flag #'transparent = ##t
Doubled Flag override.

http://codereview.appspot.com/5309059/diff/1/ly/property-init.ly#newcode245
ly/property-init.ly:245: \revert Flag #'transparent
Doubled Flag override.

[mike_apollinemike.com, 2011-10-25 05:08:38]

On Oct 24, 2011, at 11:41 PM, dak@gnu.org wrote:

> Seems like my suggestion to update the version numbers of files with
> manual fixes (in a separate commit) in order not to get them updated
> again got lost, as well as the bit about rereading the diff for
> problems.
> 
> 

I'll redo the patch - I did read the diff for updates, and I have no clue why
these didn't show up (maybe I was reading it from the wrong directory - I read
what came out from git diff).
I didn't realize that the international snippets would not get regenerated from
scratch with the new updated snippets before having the convert-ly rules applied
to them.  All of the offending examples come from them.
As for the versioning, I am not sure how to do manual updating and run the
conversion scripts at the same time.

Cheers,
MS

[dak, 2011-10-25 09:52:04]

"mike@apollinemike.com" <mike@apollinemike.com> writes:

> On Oct 24, 2011, at 11:41 PM, dak@gnu.org wrote:
>
>> Seems like my suggestion to update the version numbers of files with
>> manual fixes (in a separate commit) in order not to get them updated
>> again got lost, as well as the bit about rereading the diff for
>> problems.
>> 
>> 
>
> I'll redo the patch - I did read the diff for updates, and I have no
> clue why these didn't show up (maybe I was reading it from the wrong
> directory - I read what came out from git diff).
> I didn't realize that the international snippets would not get
> regenerated from scratch with the new updated snippets before having
> the convert-ly rules applied to them.  All of the offending examples
> come from them.
> As for the versioning, I am not sure how to do manual updating and run
> the conversion scripts at the same time.

You don't.  When you manually update a file, update it complete _and_
edit its version string to indicate it has been updated.

After that, the conversion scripts will not touch it.

-- 
David Kastrup