Review Request 118004: Fix support for installing files named the same in different directories with kdoctools_install

Luigi Toscano luigi.toscano at tiscali.it
Tue May 6 23:54:57 UTC 2014



> On May 6, 2014, 11:01 a.m., Aurélien Gâteau wrote:
> > I realized I left one code untouched: the KDOCTOOLS_ENABLE_HTMLHANDBOOK block. What should be done with it? The option is not documented: is it obsolete? If I understand it correctly, the code generates .html files in the source directory, and does not install them. Is this the intended behavior?
> 
> David Faure wrote:
>     In the longer term, I think we do want to generate HTML and install that. This will allow reading the help in a webbrowser, on platforms/systems where khelpcenter is not installed.

Yes, but for now let's fix the regressions.


- Luigi


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


On May 6, 2014, 10:59 a.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118004/
> -----------------------------------------------------------
> 
> (Updated May 6, 2014, 10:59 a.m.)
> 
> 
> Review request for Documentation, KDE Frameworks and Luigi Toscano.
> 
> 
> Repository: kdoctools
> 
> 
> Description
> -------
> 
> This is a big patch made of 4 commits (can provide them separately if needed). This is similar to https://git.reviewboard.kde.org/r/117997/ but for handbook and man pages.
> 
> The patch includes a test cmake project which for now must be installed separately because it relies on kdoctools to be already installed. I also refreshed the code a bit, turning macros into functions and using cmake_parse_arguments().
> 
> 
> Diffs
> -----
> 
>   KF5DocToolsMacros.cmake 51caf34 
>   tests/kdoctools_install-test/CMakeLists.txt PRE-CREATION 
>   tests/kdoctools_install-test/po/es/docs/index.docbook PRE-CREATION 
>   tests/kdoctools_install-test/po/es/docs/kioslave5/fooslave/index.docbook PRE-CREATION 
>   tests/kdoctools_install-test/po/es/docs/man-kjscmd.1.docbook PRE-CREATION 
>   tests/kdoctools_install-test/po/fr/docs/index.docbook PRE-CREATION 
>   tests/kdoctools_install-test/po/fr/docs/man-kjscmd.1.docbook PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/118004/diff/
> 
> 
> Testing
> -------
> 
> Tested with supplied test project.
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140506/46c3763b/attachment.html>


More information about the Kde-frameworks-devel mailing list