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 683554 - JH_PATH_XML_CATALOG ignores XML_CATALOG_FILES
JH_PATH_XML_CATALOG ignores XML_CATALOG_FILES
Status: RESOLVED WONTFIX
Product: gtk-doc
Classification: Platform
Component: general
unspecified
Other All
: Normal minor
: ---
Assigned To: gtk-doc maintainers
gtk-doc maintainers
Depends on:
Blocks:
 
 
Reported: 2012-09-07 08:48 UTC by LRN
Modified: 2013-05-30 19:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fixes the problem by trying to get XML_CATALOG_FILES first (607 bytes, patch)
2012-09-07 08:56 UTC, LRN
none Details | Review

Description LRN 2012-09-07 08:48:52 UTC
JH_PATH_XML_CATALOG uses user-provided catalog file.
If no --with-xml-catalog is given, it uses /etc/xml/catalog instead.

However, it should look at XML_CATALOG_FILES environment variable instead of hard-coding /etc/xml/catalog.

XML_CATALOG_FILES is a space-separated list of files.
Comment 1 LRN 2012-09-07 08:56:15 UTC
Created attachment 223728 [details] [review]
Fixes the problem by trying to get XML_CATALOG_FILES first

This still won't pass configure checks if XML_CATALOG_FILES has more than one catalog in it, and the URI configure script looks for is not in the _first_ catalog. To fix _that_, one would have to fix xmlcatalog itself (to also read XML_CATALOG_FILES when doing lookups, allowing the user to not to specify the catalog file).
Comment 2 Matthias Clasen 2013-04-02 02:31:52 UTC
I don't think there's a huge win here. As you say, the patch doesn't even support the full format of XML_CATALOG_FILES. You can always just use --with-xml-catalog to specify your location
Comment 3 LRN 2013-04-02 12:07:43 UTC
--with-xml-catalog is not supported by [1]

[1] http://cgit.freedesktop.org/gstreamer/common/tree/m4/as-docbook.m4
Comment 4 LRN 2013-04-02 12:11:12 UTC
OK, that's kind of stupid of me to point at a different .m4 file that has nothing to do with JH_PATH_XML_CATALOG...
Comment 5 Stefan Sauer (gstreamer, gtkdoc dev) 2013-05-30 18:22:04 UTC
I aggree with matthias. LRN, if you are fine with that please close the ticket again.