Differential D23791

ECMAddQch: add INCLUDE_DIRS argument
ClosedPublic
Actions

Authored by kossebau on Sep 8 2019, 4:15 PM.

Details

Reviewers
dfaure
Group Reviewers
Build System
Commits
R240:ee4c3fd91eea: ECMAddQch: add INCLUDE_DIRS argument
Summary

Allows to have headers be found by doxygen, e.g. to properly process
preprocessor macros.

Diff Detail

Repository
R240 Extra CMake Modules
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kossebau created this revision.Sep 8 2019, 4:15 PM
Restricted Application added projects: Frameworks, Build System. · View Herald TranscriptSep 8 2019, 4:15 PM
Restricted Application added subscribers: kde-buildsystem, kde-frameworks-devel. · View Herald Transcript
kossebau requested review of this revision.Sep 8 2019, 4:15 PM
kossebau added a comment.Sep 10 2019, 12:34 PM

Would push upcoming WE if there are no objections.
And then also adapt all KF modules to make use of this new argument, for better results and less need for predefined macros.

dfaure accepted this revision.Sep 14 2019, 11:16 AM
dfaure added a subscriber: dfaure.

I don't know doxygen very well, but the cmake change seems sensible.

This revision is now accepted and ready to land.Sep 14 2019, 11:16 AM
Closed by commit R240:ee4c3fd91eea: ECMAddQch: add INCLUDE_DIRS argument (authored by kossebau). · Explain WhySep 14 2019, 11:53 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 66099

View Options

modules/ECMAddQch.cmake

Loading...
View Options

modules/ECMQchDoxygen.config.in

Loading...