Review Request 124714: Add support to qmake's options to api.kde.org

[Lamarque Souza]

> On Nov. 3, 2015, 7:42 p.m., Alex Merry wrote:
> > Sorry I've let this sit for so long - Real Life(TM) got in the way.
> > 
> > Rather than adding variables for each of these, I'd rather have some way to add free text to this field. Also, have you checked that an extra line break doesn't get added when the fields are not set?
> > 
> > Finally, have you explored ways to make these *not* be required? I don't know QMake's build system very well, but it seems to me that at least some of these might be able to go in the .pri file (and it might be possible to protect client projects from the need to set QT_NO_KEYWORDS).

I have changed the patch to allow adding (almost) free text to the field. The dash at each line start is still needed to mark linebreaks in the output html file.

I am not able to compile networkmanager-qt using qmake from Qt 5.4.2 anymore, so I cannot test your suggestion to move some of the configurations to .pri file. Anyway, the current patch seems to work and does not add unnecessary fields like the first version.


- Lamarque


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


On Nov. 8, 2015, 1:21 a.m., Lamarque Souza wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124714/
> -----------------------------------------------------------
> 
> (Updated Nov. 8, 2015, 1:21 a.m.)
> 
> 
> Review request for KDE Frameworks and Alex Merry.
> 
> 
> Repository: kapidox
> 
> 
> Description
> -------
> 
> NetworkManagerQt requires some options to be added to .pro file to compile:
> 
> QT += NetworkManagerQt dbus network
> CONFIG += link_pkgconfig
> PKGCONFIG += libnm # from NetworkManager package
> DEFINES += QT_NO_KEYWORDS # libnm declares a variable named signals
> 
> kapidox already supports the first option (QT), but not the others. This patch adds support to adding the other three options.
> 
> 
> Diffs
> -----
> 
>   src/kapidox/data/templates/fwinfo.html a20c1d1 
> 
> Diff: https://git.reviewboard.kde.org/r/124714/diff/
> 
> 
> Testing
> -------
> 
> I added the lines below to networkmanager-qt/metainfo.html and kapidox created the correct output:
> 
> qmakepro:
>  - CONFIG += link_pkgconfig
>  - DEFINES += QT_NO_KEYWORDS
>  - PKGCONFIG += libnm
> 
> 
> Thanks,
> 
> Lamarque Souza
> 
>

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