[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5472 flower/file-name.cc: Fix canonical file name representation.

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

• Description has changed:

Diff:

• assigned_to: Werner LEMBERG
• Needs: -->
• Patch: new --> needs_work
• Comment:

This breaks make check.

Seems to throw a permission violation and running this patch test a few times it is always on the same file (if that is significant).

make[2]: Entering directory '/home/jlowe/lilypond-git/build/input/regression/midi'
touch  ./out-test/crescendo-gap-compatible-target.texidoc  ./out-test/crescendo-gap-compatible-target.options
touch  ./out-test/crescendo-return-crescendo.texidoc  ./out-test/crescendo-return-crescendo.options
touch  ./out-test/crescendo-return-louder-target.texidoc  ./out-test/crescendo-return-louder-target.options
/home/jlowe/lilypond-git/build/scripts/build/out/run-and-check "/home/jlowe/lilypond-git/build/out/bin/lilypond -H texidoc -H options -o ./out-test /home/jlowe/lilypond-git/input/regression/midi/crescendo-gap-compatible-target.ly" "crescendo-gap-compatible-target.log"
/home/jlowe/lilypond-git/build/scripts/build/out/run-and-check "/home/jlowe/lilypond-git/build/out/bin/lilypond -H texidoc -H options -o ./out-test /home/jlowe/lilypond-git/input/regression/midi/crescendo-return-crescendo.ly" "crescendo-return-crescendo.log"
/home/jlowe/lilypond-git/build/scripts/build/out/run-and-check "/home/jlowe/lilypond-git/build/out/bin/lilypond -H texidoc -H options -o ./out-test /home/jlowe/lilypond-git/input/regression/midi/crescendo-return-louder-target.ly" "crescendo-return-louder-target.log"

Please check the logfile crescendo-gap-compatible-target.log for errors

/home/jlowe/lilypond-git/./make/midi-rules.make:7: recipe for target 'out-test/crescendo-gap-compatible-target.midi' failed
make[2]: *** [out-test/crescendo-gap-compatible-target.midi] Error 1
make[2]: *** Waiting for unfinished jobs....

Please check the logfile crescendo-return-crescendo.log for errors

/home/jlowe/lilypond-git/./make/midi-rules.make:7: recipe for target 'out-test/crescendo-return-crescendo.midi' failed
make[2]: *** [out-test/crescendo-return-crescendo.midi] Error 1

Please check the logfile crescendo-return-louder-target.log for errors

/home/jlowe/lilypond-git/./make/midi-rules.make:7: recipe for target 'out-test/crescendo-return-louder-target.midi' failed
make[2]: *** [out-test/crescendo-return-louder-target.midi] Error 1
make[2]: Leaving directory '/home/jlowe/lilypond-git/build/input/regression/midi'
/home/jlowe/lilypond-git/./make/lysdoc-targets.make:12: recipe for target 'local-test' failed
make[1]: *** [local-test] Error 2
make[1]: Leaving directory '/home/jlowe/lilypond-git/build/input/regression/midi'
/home/jlowe/lilypond-git/GNUmakefile.in:308: recipe for target 'test' failed
make: *** [test] Error 2

this is what the log file shows

Changing working directory to: `./out-test'
Processing `/home/jlowe/lilypond-git/input/regression/midi/crescendo-return-crescendo.ly'
Parsing...
Interpreting music...
fatal error: cannot open for write: /crescendo-return-crescendo.midi: Permission denied

I had two patches to test this morning and the other patch of Valentin's went through with no issues so something else is at work here.

---

[issues:#5472] flower/file-name.cc: Fix canonical file name representation.

Status: Started
Created: Mon Feb 04, 2019 05:37 AM UTC by Werner LEMBERG
Last Updated: Mon Feb 04, 2019 05:37 AM UTC
Owner: Werner LEMBERG

flower/file-name.cc: Fix canonical file name representation.

Before this fix, /foo' was canonicalized tofoo', incorrectly dropping the
leading slash. (However, `/foo/bar' was handled fine – this is perhaps the
reason that this quite severe bug wasn't detected earlier.)

Also add proper tests.

This patch also contains the following commit:

yaffut.hh: Fix compilation warning.

http://codereview.appspot.com/355920043

---

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto