KDE/kdelibs/cmake/modules

[Matthew Woehlke]

Allen Winter wrote:
> SVN commit 1125486 by winterz:
>
> Super-duper-hackarific for supporting Fedora12 and Fedora13.
> Older Fedoras will need to be added as required.
> Rex is talking to Fedora about this.
> CCMAIL: kde-buildsystem at kde.org
>
>
>   M  +10 -0     FindDocBookXML.cmake
>
>
> --- trunk/KDE/kdelibs/cmake/modules/FindDocBookXML.cmake #1125485:1125486
> @@ -29,6 +29,16 @@
>      share/xml/docbook/${DOCBOOKXML_CURRENTDTD_VERSION}
>   )
>
> +#hacks for Fedora
> +if(NOT ${DOCBOOKXML_CURRENTDTD_DIR})
> +find_path(DOCBOOKXML_CURRENTDTD_DIR catalog
> +   PATHS ${CMAKE_SYSTEM_PREFIX_PATH}
> +   PATH_SUFFIXES
> +   share/sgml/docbook/xml-dtd-4.2-1.0-48.fc12
> +   share/sgml/docbook/xml-dtd-4.2-1.0-50.fc13
> +)
> +endif(NOT ${DOCBOOKXML_CURRENTDTD_DIR})
> +
>   #set (DOCBOOKXML_OLDDTD_DIR ${DOCBOOKXML_CURRENTDTD_DIR})
>   #set (DOCBOOKXML_OLDDTD_VERSION "4.1.2")
>   #find_path (DOCBOOKXML_OLDDTD_DIR catalog.xml

Thanks. I guess I don't need to tell you it's ugly (thanks Fedora :P ). 
I had my own idea, here: http://reviewboard.kde.org/r/3954/

It's also ugly, but at least doesn't hard-code the rpm version. Does it 
look okay?

(I thought there was a Review Board group for buildsystem? ...but I 
can't find it, so I assigned the review to myself and point at it here.)

-- 
Matthew
Please do not quote my e-mail address unobfuscated in message bodies.
-- 
When in doubt, give it a good swift kick.