Create a QCH file with the API dox, optionally, using ECMAddQCH
ClosedPublic

Authored by kossebau on Nov 21 2016, 6:52 PM.

Details

Summary

See https://phabricator.kde.org/D2854 for central discussion of new ECMAddQCH

Test Plan

KF5JobWidgets.qch created by this: https://share.kde.org/index.php/s/UhVPFAy2cTb8cBL

Diff Detail

Branch
addQCHBuild
Lint
No Linters Available
Unit
No Unit Test Coverage
kossebau updated this revision to Diff 8357.Nov 21 2016, 6:52 PM
kossebau retitled this revision from to Create a QCH file with the API dox, optionally, using ECMAddQCH.
kossebau updated this object.
kossebau edited the test plan for this revision. (Show Details)
kossebau added reviewers: Frameworks, ochurlaud.
shumski added inline comments.
CMakeLists.txt
83

I wonder about the reasoning behind the existence of this export, and installation of it...

src/CMakeLists.txt
98

Is this really the right place?

kossebau added inline comments.Nov 21 2016, 9:01 PM
CMakeLists.txt
83

Imagine another project with a lib MyLib linking against KJobWidgets (in the public interface). For MyLib also a QCH file should be created. To create proper links from the QCH file for MyLib into the QCH file of KJobWidgets (via qthelp:// urls), doxygen/the qt tools need to have access to some information. This information is bound to the KF5JobWidgets_QCH target (see https://phabricator.kde.org/D2854).
By exporting the target here and including it into KF5JobWidgetsConfig.cmake, this information then is automatically available to the ecm_add_qch call with MyLib and passing just the target name in the LINK_QCH argument.

shumski added inline comments.Nov 21 2016, 9:18 PM
CMakeLists.txt
83

Allright, make sense.

kossebau updated this revision to Diff 8488.Nov 24 2016, 10:39 PM

Use new KDE_INSTALL_FULL_QTQCHDIR

kossebau updated this revision to Diff 8589.Nov 29 2016, 1:36 AM

update to latest version of proposed ECMAddQCH

kossebau updated this revision to Diff 8792.Dec 5 2016, 6:33 PM

update to latest ECMAddQch

kossebau updated this revision to Diff 14669.May 18 2017, 1:35 PM

Update to latest ECMAddQch

Restricted Application added a project: Frameworks. · View Herald TranscriptMay 18 2017, 1:35 PM
This revision was automatically updated to reflect the committed changes.