[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5968 Prevent race condition in `-dfont-ps-resdir`

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

• Description has changed:

Diff:

• Needs: -->
• Patch: new --> review
• Comment:

Passes make, make check and a full make doc.

---

[issues:#5968] Prevent race condition in -dfont-ps-resdir

Status: Started
Created: Sat May 09, 2020 10:26 AM UTC by Masamichi Hosoda
Last Updated: Sat May 09, 2020 10:26 AM UTC
Owner: Masamichi Hosoda

Prevent race condition in -dfont-ps-resdir

Issue 5968/4: Add Windows support for -dfont-ps-resdir

Windows environment could not use -dfont-ps-resdir option
because it used symlink.

This commit makes using file copying instead symlink when Windows.

Issue 5968/3: Prevent creating file race condition in -dfont-ps-resdir

For -dfont-ps-resdir,
we used creating file after checking the file existence.
However, when multiple lilypond processes were running,
as when using lilypond-book,
a race condition could cause creating file to fail.

This commit uses creating file method for -dfont-export-dir
that can prevent race condition.

Issue 5968/2: Prevent symlink race condition in -dfont-ps-resdir

For -dfont-ps-resdir,
we used symlink after checking the file existence.
However, when multiple lilypond processes were running,
as when using lilypond-book, a race condition could cause symlink to fail.

This commit makes preventing the race condition.

Issue 5968/1: Prevent mkdir race condition in -dfont-ps-resdir

For -dfont-ps-resdir,
we used mkdir after checking the directory existence.
However, when multiple lilypond processes were running,
as when using lilypond-book, a race condition could cause mkdir to fail.

This commit uses making directory method for -dfont-export-dir
that can prevent race condition.

http://codereview.appspot.com/561810045

---

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