After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 595197 - gnome-doc-utils.make uses non-portable shell constructs
gnome-doc-utils.make uses non-portable shell constructs
Status: RESOLVED WONTFIX
Product: gnome-doc-utils
Classification: Deprecated
Component: build utils
unspecified
Other Linux
: Normal minor
: ---
Assigned To: gnome-doc-utils maintainers
gnome-doc-utils maintainers
gnome[unmaintained]
Depends on:
Blocks:
 
 
Reported: 2009-09-14 17:42 UTC by Daniel Elstner
Modified: 2020-03-03 18:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix gnome-doc-utils.make parallel build failure (957 bytes, patch)
2009-09-14 17:42 UTC, Daniel Elstner
committed Details | Review
Avoid non-portable shell exit code negation (5.23 KB, patch)
2009-09-14 17:43 UTC, Daniel Elstner
none Details | Review
Update .gitignore files (1.89 KB, patch)
2009-09-14 17:44 UTC, Daniel Elstner
none Details | Review

Description Daniel Elstner 2009-09-14 17:42:42 UTC
Created attachment 143174 [details] [review]
Fix gnome-doc-utils.make parallel build failure

The clean-doc-dir rule in gnome-doc-utils.make removes subdirectories with
rmdir. Before rmdir can succeed, all files in the directory need to be deleted
first. However, the clean-doc-dir rule deletes only a subset of the files
itself, and relies on the clean-doc-lc rule to delete the .xml2po.mo files.

Since the clean-doc-lc rule is completely independent, they will be executed
concurrently by parallel make. If the rmdir runs before the clean-doc-lc rule
removed the .xml2po.mo file, the build fails. For me, this always happens when
trying to distcheck the gtkmm-documentation module on my desktop.

Three patches follow: The first fixes the parallel build breakage by making the
clean-doc-dir rule depend on clean-doc-lc. The second patch replaces a number
of non-portable shell constructs by more portable alternatives. I also updated
the .gitignore files while I was at it, and the changes can be found in the
third patch.
Comment 1 Daniel Elstner 2009-09-14 17:43:18 UTC
Created attachment 143175 [details] [review]
Avoid non-portable shell exit code negation
Comment 2 Daniel Elstner 2009-09-14 17:44:18 UTC
Created attachment 143176 [details] [review]
Update .gitignore files

OK to push?
Comment 3 Daniel Elstner 2009-09-16 18:32:24 UTC
The first patch was approved by the release team and has been pushed to master. Retitling because the other two patches are unrelated to the original bug.
Comment 4 André Klapper 2020-03-03 18:34:10 UTC
gnome-doc-utils has been superseded by yelp-xsl, yelp-tools, and itstool.

gnome-doc-utils will not see any further development, hence closing as WONTFIX.

See https://gitlab.gnome.org/Infrastructure/Infrastructure/issues/255