Skip to content

Commit

Permalink
Changes from review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
bhourahine committed May 8, 2022
1 parent 9f812ae commit 0adfd6d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,8 @@ endif()
include(GNUInstallDirs)

add_subdirectory(lib)
if(NOT BUILD_EXPORTED_TARGETS_ONLY AND BUILD_TESTING)
include(CTest) # note: this adds a BUILD_TESTING which defaults to ON
if(BUILD_TESTING)
enable_testing()
add_subdirectory(test)
endif()
Expand Down
4 changes: 2 additions & 2 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ You can influence the configuration via CMake-variables, which are listed in
`config.cmake <config.cmake>`_. You can either modify the values directly there
or pass them as command line options at the configuration phase, e.g.::

FC=ifort cmake -B _build -DBUILD_LIBRARY_ONLY=True
FC=ifort cmake -B _build -DBUILD_TESTING=NO .

Testing
Expand All @@ -49,7 +49,7 @@ A few tests / usage examples can be found in the `test/` subdirectory,
a sub-set of which can be checked with ctest. The compiled test
programs will be in the `test/` subfolder of your build directory.

To enable building, include the cmake option -DBUILD_TESTING=ON
To disable building tests, include the cmake option -DBUILD_TESTING=OFF


Using the library
Expand Down

0 comments on commit 0adfd6d

Please sign in to comment.