DocBook XML & XSL: new dependencies

Luigi Toscano luigi.toscano at tiscali.it
Thu May 6 00:53:48 CEST 2010


Alexander Neundorf wrote:
 > On Tuesday 04 May 2010, Luigi Toscano wrote:
 >> as I announced on kde-core-devel[1], I'm working to remove the embedded
 >> copies of XML and XSLT files from kdelibs/kdoctools. This process will
 >> introduce two external dependencies; I'd like to complete the migration
 >> before the release of 4.5, so the proposed Find*.cmake modules for kdelibs
 >
 > Are the two find-modules supposed to be installed ?
 > IOW, are docbookxml and docbook xsl only required when building kdelibs or for
 > all other KDE modules too ?
They are required whenever you need to compile documentation, but the path to 
the DTDs/XMLT will be discovered and hardcoded (like now), so exporting them 
shouldn't be needed. But there are two possible consumer for them:
- the xmlcheck plugin for kate (kdesdk) which needs to be touched anyway 
because refers to the internal copy of XML DTDs, and
- the docbook exporter for umbrello, which does the same for xslt.


 > Things which should be changed:
 > * use the find_package_handle_standard_args() macro for the status output at
 > the end and setting<package>_FOUND.
Is it possible to specify a custom success message, other than the failure 
message?

 > * I would remove the shortcuts at the top of the file "if (DOCBOOKXSL_DIR)",
 > the results of the find_file/path/program/library() commands are cached, so I
 > don't think it really makes it noticable faster.
Which component/function sets xyz_FIND_QUIETLY, then?


 > * for the search paths, I would suggest you try something with PATH_SUFFIXES,
 > like[...]
 >   find_path (DOCBOOKXML_CURRENTDTD_DIR catalog.xml
 >        PATHS ${CMAKE_SYSTEM_PREFIX_PATH}
 >
 >        PATH_SUFFIXES share/xml/docbook/${DOCBOOK_XML_CURRENTDTD_VERSION}
 >            share/sgml/docbook/xml-dtd-${DOCBOOK_XML_CURRENTDTD_VERSION}
 >            share/xml/docbook/xml-dtd-${DOCBOOK_XML_CURRENTDTD_VERSION}
 >            share/xml/docbook/schema/dtd/${DOCBOOK_XML_CURRENTDTD_VERSION}
 >     )
Thanks, I was not sure about how the handle this in a portable way.

Regards
-- 
Luigi


More information about the Kde-buildsystem mailing list