[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: DocBook XML & XSL: work in progess, new dependencies
From:       Dawit A <adawit () kde ! org>
Date:       2010-05-13 18:40:28
Message-ID: 201005131440.28835.adawit () kde ! org
[Download RAW message or body]

On Thursday, May 13, 2010 14:00:04 Parker Coates wrote:
> On Thu, May 13, 2010 at 12:04, Dawit A wrote:
> > Any objection in me committing the attached patch to fix the location
> > detection of docbook xsl style sheet in platforms that use version
> > number in the path names, e.g. ArchLinux ?
> 
> There's an issue with the file you attached. The contents when opened
> here are just " [H [2J".

Indeed... Dunno what happened there, but here is the right patch.

["cmake_FindDocBookXSL.patch" (text/x-patch)]

Index: FindDocBookXSL.cmake
===================================================================
--- FindDocBookXSL.cmake	(revision 1126296)
+++ FindDocBookXSL.cmake	(working copy)
@@ -10,12 +10,24 @@
 # Redistribution and use is allowed according to the terms of the BSD license.
 # For details see the accompanying COPYING-CMAKE-SCRIPTS file.
 
+set (STYLESHEET_PATH_LIST
+    share/xml/docbook/stylesheet/docbook-xsl
+    share/sgml/docbook/xsl-stylesheets
+    share/xml/docbook/stylesheet/nwalsh/current
+)
+
+foreach (STYLESHEET_PREFIX_ITER ${CMAKE_SYSTEM_PREFIX_PATH})
+   file(GLOB STYLESHEET_SUFFIX_ITER RELATIVE ${STYLESHEET_PREFIX_ITER}
+        ${STYLESHEET_PREFIX_ITER}/share/xml/docbook/xsl-stylesheets-*
+   )
+   if (STYLESHEET_SUFFIX_ITER)
+      list (APPEND STYLESHEET_PATH_LIST ${STYLESHEET_SUFFIX_ITER})
+   endif ()
+endforeach ()
+
 find_path (DOCBOOKXSL_DIR VERSION
    PATHS ${CMAKE_SYSTEM_PREFIX_PATH}
-   PATH_SUFFIXES
-   share/xml/docbook/stylesheet/docbook-xsl
-   share/sgml/docbook/xsl-stylesheets
-   share/xml/docbook/stylesheet/nwalsh/current
+   PATH_SUFFIXES ${STYLESHEET_PATH_LIST}
 )
 
 find_package_handle_standard_args (DocBookXSL


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic