Add BUILD_EXE as cmake option#717
Conversation
| # Install a unit test | ||
| macro (dagmc_install_test test_name ext) | ||
| message(STATUS "Building unit tests: ${test_name}") | ||
| if (NOT BUILD_EXEC) |
There was a problem hiding this comment.
This should be BUILD_EXE not BUILD_EXEC
|
I am now missing the includes :( in the install folder.... |
|
I think that's a problem in the while the static library section starts like this: The static library section doesn't set the public headers as part of the target properties. |
|
included the static header install fix in this PR.... |
pshriwise
left a comment
There was a problem hiding this comment.
Just a few small items from me here.
| option(BUILD_SHARED_LIBS "Build shared libraries" ON) | ||
| option(BUILD_STATIC_LIBS "Build static libraries" ON) | ||
|
|
||
| option(BUILD_EXE "Build DAGMC executables" ON) |
There was a problem hiding this comment.
Let's indent the comment string to match the other options.
Co-authored-by: Patrick Shriwise <pshriwise@gmail.com>
|
@pshriwise is there some still some undressed comments in here ? |
|
@bam241 can you remind me what the use case is for building without executables? |
The plugin. |
This should allow to build dagmc libs (shared and.or static) without the exe.