Re: Review Request 124714: Add support to qmake's options to api.kde.org
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124714/ --- (Updated Nov. 8, 2015, 9:40 p.m.) Status -- This change has been marked as submitted. Review request for KDE Frameworks and Alex Merry. Changes --- Submitted with commit b3693ce2960c96b6bfec5d557ef75df5e18ef05d by Lamarque V. Souza to branch master. 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 ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 124714: Add support to qmake's options to api.kde.org
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124714/#review88165 --- Ship it! Ship It! - Alex Merry 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 > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 124714: Add support to qmake's options to api.kde.org
--- 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. Changes --- This version allows adding all lines from qmakepro section to the output html file. 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 (updated) - src/kapidox/data/templates/fwinfo.html a20c1d1 Diff: https://git.reviewboard.kde.org/r/124714/diff/ Testing (updated) --- 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 ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 124714: Add support to qmake's options to api.kde.org
> 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 > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 124714: Add support to qmake's options to api.kde.org
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124714/#review87955 --- 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). - Alex Merry On Aug. 12, 2015, 1:13 p.m., Lamarque Souza wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124714/ > --- > > (Updated Aug. 12, 2015, 1:13 p.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 > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Review Request 124714: Add support to qmake's options to api.kde.org
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124714/ --- 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 ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel