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 723436 - sanitycheck can't find xmlcatalog modules even after installing docbook-xsl
sanitycheck can't find xmlcatalog modules even after installing docbook-xsl
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
Depends on:
Blocks:
 
 
Reported: 2014-02-01 22:39 UTC by Andrés G. Aragoneses (IRC: knocte)
Modified: 2014-02-28 21:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (1.42 KB, patch)
2014-02-01 22:40 UTC, Andrés G. Aragoneses (IRC: knocte)
committed Details | Review
Another nice to have (1.42 KB, patch)
2014-02-01 22:45 UTC, Andrés G. Aragoneses (IRC: knocte)
reviewed Details | Review
Another nice to have v2 (1.44 KB, patch)
2014-02-28 20:22 UTC, Andrés G. Aragoneses (IRC: knocte)
reviewed Details | Review

Description Andrés G. Aragoneses (IRC: knocte) 2014-02-01 22:39:39 UTC
I was getting the following when running jhbuild sanitycheck:

$ jhbuild sanitycheck
Could not find DocBook XML DTD V4.1.2 in XML catalog (usually part of package 'docbook-xsl')
Could not find DocBook XSL Stylesheets in XML catalog (usually part of package 'docbook-xsl')

Even after installing docbook-xsl, same error occurred... Well, what happens is that this is the command that was run:

xmlcatalog /etc/xml/catalog http://docbook.sourceforge.net/release/xsl/current/html/chunk.xsl

But it wasn't failing because it couldn't find that XSL in the XML catalog, but because the command xmlcatalog didn't exist in the system!

Patch coming...
Comment 1 Andrés G. Aragoneses (IRC: knocte) 2014-02-01 22:40:20 UTC
Created attachment 267806 [details] [review]
Proposed patch
Comment 2 Andrés G. Aragoneses (IRC: knocte) 2014-02-01 22:45:01 UTC
Created attachment 267807 [details] [review]
Another nice to have
Comment 3 Frederic Peters 2014-02-28 19:53:21 UTC
Comment on attachment 267806 [details] [review]
Proposed patch

great, thanks.
Comment 4 Frederic Peters 2014-02-28 19:53:31 UTC
Comment on attachment 267807 [details] [review]
Another nice to have

Is this also libxml-parser-perl on fedora & friends?
Comment 5 Andrés G. Aragoneses (IRC: knocte) 2014-02-28 19:55:09 UTC
(In reply to comment #4)
> Is this also libxml-parser-perl on fedora & friends?

No idea sorry, don't use Fedora.
Comment 6 Andrés G. Aragoneses (IRC: knocte) 2014-02-28 20:22:25 UTC
Created attachment 270599 [details] [review]
Another nice to have v2

Now including fedora package name (confirmed in https://apps.fedoraproject.org/packages/s/perl-XML-Parser and freenode#fedora).
Comment 7 Frederic Peters 2014-02-28 20:31:16 UTC
Comment on attachment 270599 [details] [review]
Another nice to have v2

Sorry I didn't have a proper look before; as the error message is now specific to that perl module, and the loop has just one element, we should remove the loop.

That would be a diff like this, and you could apply your patch on top of it:

+        # Perl module used by tools such as intltool:
+        perlmod = 'XML::Parser'
+        try:
+            get_output(['perl', '-M%s' % perlmod, '-e', 'exit'])
+        except:
+            uprint(_('Could not find the Perl module %s') % perlmod)
Comment 8 Andrés G. Aragoneses (IRC: knocte) 2014-02-28 21:30:18 UTC
Cool thanks, all pushed!