ECMAddQch: add INCLUDE_DIRS argument
ClosedPublic

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

Details

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.Sun, Sep 8, 4:15 PM
Restricted Application added projects: Frameworks, Build System. · View Herald TranscriptSun, Sep 8, 4:15 PM
Restricted Application added subscribers: kde-buildsystem, kde-frameworks-devel. · View Herald Transcript
kossebau requested review of this revision.Sun, Sep 8, 4:15 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.Sat, Sep 14, 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.Sat, Sep 14, 11:16 AM
This revision was automatically updated to reflect the committed changes.