Skip to content

Commit

Permalink
Adding standard includes for libraries, executbles, tests, etc.
Browse files Browse the repository at this point in the history
Packages need almost all of these anyway at some point so they might as well
just be included in the top level package include.  This should help avoid
problems that I have seen on some systems and some versions of CMake where if
you miss an include CMake will do strange things.

Note that I would have liked to have avoiding having to include
TribitsPackageMacros.cmake at all in a package's CMakeLiss.txt file but the
current contents of TribitsAddExecutable.cmake set variables that require
package-level scope.  Hopefully I can clean this up soon.
  • Loading branch information
Roscoe A. Bartlett committed Feb 6, 2014
1 parent 48bf98b commit 081f8f9
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions package_arch/TribitsPackageMacros.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,13 @@ INCLUDE(PrependGlobalSet)
INCLUDE(RemoveGlobalDuplicates)
INCLUDE(TribitsAddOptionAndDefine)

INCLUDE(TribitsLibraryMacros)
INCLUDE(TribitsAddExecutable)
INCLUDE(TribitsAddExecutableAndTest)
INCLUDE(TribitsAddTest)
INCLUDE(TribitsAddAdvancedTest)
INCLUDE(TribitsCopyFilesToBinaryDir)

###
### WARNING: See "NOTES TO DEVELOPERS" at the bottom of file!
###
Expand Down

0 comments on commit 081f8f9

Please sign in to comment.