Skip to content
Snippets Groups Projects

Move CMAKE_INSTALL_INCLUDEDIR after inclusion of GNUInstallDirs

Merged Jane Tournois requested to merge Fix-CMakeLists-GF into master

This MR implements two small fixes for the CoMISo CMakeLists :

  • Before inclusion of GNUInstallDirs, CMAKE_INSTALL_INCLUDEDIR is empty so the command target_include_directories does not include the right directory.

  • When CoMISo is part of a larger project, we do not want to move the config file to the general binary directory, but the one specific to CoMISo, namely the CMAKE_CURRENT_BINARY_DIR

Merge request reports

Pipeline #12822 passed

Pipeline passed for f566d92d on Fix-CMakeLists-GF

Approval is optional

Merged by Max LyonMax Lyon 5 years ago (Nov 15, 2019 1:40pm UTC)

Loading

Pipeline #12827 passed

Pipeline passed for fba5e448 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading