D23791: ECMAddQch: add INCLUDE_DIRS argument

2019-09-14 Thread Friedrich W. H. Kossebau
This revision was automatically updated to reflect the committed changes.
Closed by commit R240:ee4c3fd91eea: ECMAddQch: add INCLUDE_DIRS argument 
(authored by kossebau).

REPOSITORY
  R240 Extra CMake Modules

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D23791?vs=65657&id=66099

REVISION DETAIL
  https://phabricator.kde.org/D23791

AFFECTED FILES
  modules/ECMAddQch.cmake
  modules/ECMQchDoxygen.config.in

To: kossebau, #build_system, dfaure
Cc: dfaure, kde-frameworks-devel, kde-buildsystem, LeGast00n, GB_2, bencreasy, 
michaelh, ngraham, bruns


D23791: ECMAddQch: add INCLUDE_DIRS argument

2019-09-14 Thread David Faure
dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.


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

REPOSITORY
  R240 Extra CMake Modules

BRANCH
  supoortincludedirsinaddqch

REVISION DETAIL
  https://phabricator.kde.org/D23791

To: kossebau, #build_system, dfaure
Cc: dfaure, kde-frameworks-devel, kde-buildsystem, LeGast00n, GB_2, bencreasy, 
michaelh, ngraham, bruns


D23791: ECMAddQch: add INCLUDE_DIRS argument

2019-09-10 Thread Friedrich W. H. Kossebau
kossebau added a comment.


  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.

REPOSITORY
  R240 Extra CMake Modules

REVISION DETAIL
  https://phabricator.kde.org/D23791

To: kossebau, #build_system
Cc: kde-frameworks-devel, kde-buildsystem, LeGast00n, GB_2, bencreasy, 
michaelh, ngraham, bruns


D23791: ECMAddQch: add INCLUDE_DIRS argument

2019-09-08 Thread Friedrich W. H. Kossebau
kossebau created this revision.
kossebau added a reviewer: Build System.
Herald added projects: Frameworks, Build System.
Herald added subscribers: kde-buildsystem, kde-frameworks-devel.
kossebau requested review of this revision.

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

REPOSITORY
  R240 Extra CMake Modules

BRANCH
  supoortincludedirsinaddqch

REVISION DETAIL
  https://phabricator.kde.org/D23791

AFFECTED FILES
  modules/ECMAddQch.cmake
  modules/ECMQchDoxygen.config.in

To: kossebau, #build_system
Cc: kde-frameworks-devel, kde-buildsystem, LeGast00n, GB_2, bencreasy, 
michaelh, ngraham, bruns