Review Request 115897: Remove FindDocBook*.cmake

Alex Merry kde at randomguy3.me.uk
Sun Feb 23 14:58:50 UTC 2014



> On Feb. 22, 2014, 6:29 p.m., Luigi Toscano wrote:
> > As this was already committed, can this RR be closed?
> 
> Alex Merry wrote:
>     Actually, I never committed this one, since I never got a ship it.  I added the files to kdoctools, but never removed them from e-c-m.
> 
> Luigi Toscano wrote:
>     Is a shipit really needed, given https://git.reviewboard.kde.org/r/115896/ ?

I tend to err on the side of caution.  If you're of the view that this should get shipped, you could always give it the shipit :-)


- Alex


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115897/#review50553
-----------------------------------------------------------


On Feb. 19, 2014, 11:45 p.m., Alex Merry wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115897/
> -----------------------------------------------------------
> 
> (Updated Feb. 19, 2014, 11:45 p.m.)
> 
> 
> Review request for Build System, Extra Cmake Modules, KDE Frameworks, David Faure, Kevin Ottens, and Luigi Toscano.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> Remove FindDocBook*.cmake
> 
> These are only really useful to kdoctools, so they may as well live
> there.
> 
> (NB: after looking at how kdoctools uses the information from these files, I suspect they won't even be needed for the compatibility macros that are intended to end up in kde4support).
> 
> 
> Diffs
> -----
> 
>   find-modules/FindDocBookXML.cmake b6d623e4e5ca40cdda4c895a19a0dc273831703a 
>   find-modules/FindDocBookXSL.cmake a7320aed66b72c92f0286658e38b7fc32266790c 
> 
> Diff: https://git.reviewboard.kde.org/r/115897/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alex Merry
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20140223/81ee1b9a/attachment.html>


More information about the Kde-buildsystem mailing list