Bugzilla Bug 219698

(Reported to upstream at the URL above already, reporting here to avoid
duplication of work and to provide ebuild patch too.)

The install-default-links target in sounds/Makefile is missing dependency on
install-files. Because of it, make may execute install-default-links first or
in parallel (as it does with -jN for N>1). "ln -s" command then fails because
the target directory doesn't exist yet:

make -C sounds install
make[1]: Entering directory
`/var/tmp/portage/gnome-extra/gnome-audio-2.22.1/work/gnome-audio-2.22.1/sounds'
for I in *.wav */*.wav ; do \
                mkdir -p
/var/tmp/portage/gnome-extra/gnome-audio-2.22.1/image///usr/share/sounds/`dirname
$I`; \
                install -c -m 444  $I
/var/tmp/portage/gnome-extra/gnome-audio-2.22.1/image///usr/share/sounds/$I; \
        done
ln -sf startup3.wav
/var/tmp/portage/gnome-extra/gnome-audio-2.22.1/image///usr/share/sounds/login.wav
ln: creating symbolic link
`/var/tmp/portage/gnome-extra/gnome-audio-2.22.1/image///usr/share/sounds/login.wav':
No such file or directory
make[1]: *** [install-default-links] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: Leaving directory
`/var/tmp/portage/gnome-extra/gnome-audio-2.22.1/work/gnome-audio-2.22.1/sounds'
make: *** [install] Error 2

Attached patches take care of it.

Reproducible: Always

Created an attachment (id=151314) [details]
files/gnome-audio-2.22.1-makefile.patch

Created an attachment (id=151315) [details]
gnome-audio-2.22.1.ebuild patch to apply the fix

Hrm.  I cannot reproduce.  It installs fine for me with -j3 on a dualcore (I
ran it in a loop for 37 times).

(In reply to comment #3)
> Hrm.  I cannot reproduce.  It installs fine for me with -j3 on a dualcore (I
> ran it in a loop for 37 times).

What can I say? It's the nature of parallel make that a task sometimes finishes
sooner or later *nondeterministically*. The bug is obvious if you understand
how make's dependencies graph works (together with the fact that you cannot
create a file or symlink in nonexistent directory).

FYI, upstream already applied the (no-brainer) patch.

I didn't say it wasn't broken, just that I couldn't reproduce it.  Hard to test
a fix when you can't reproduce it.

2.22.2 which has the fix is now in tree.