Skip to content
Snippets Groups Projects

GenericSystem: Remove misspelled flag default

Merged Christian Pfeiffer requested to merge ChrisTX/cmake:generic-inc-sep-typo-fix into master

In CMakeGenericSystem, it should be CMAKE_INCLUDE_SEP_<LANG> and not CMAKE_INCLUDE_<LANG>_SEP. However, if the flag is defined, include behavior changes to Java-style non-repeating flags, which isn't intended. Therefore, removing the (misspelled) default initialization should help remove confusion and unintended side effects.

Topic-rename: generic-inc-sep-typo-remove

Edited by Brad King

Merge request reports

Pipeline #88766 passed

Pipeline passed for cd3e1600 on ChrisTX:generic-inc-sep-typo-fix

Approval is optional

Merged by Kitware RobotKitware Robot 7 years ago (Jan 24, 2018 1:49pm UTC)

Merge details

  • Changes merged into with cd3e1600.
  • Deleted the source branch.

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