Skip to content

Commit

Permalink
Merge pull request #19 from Goddard-Fortran-Ecosystem/bugfix/mathomp4…
Browse files Browse the repository at this point in the history
…/18-default-install

Fixes #18. Guard against default install prefix
  • Loading branch information
tclune authored Nov 15, 2022
2 parents 4ccac88 + fc2d6e1 commit 6018886
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 0 deletions.
1 change: 1 addition & 0 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
* @tclune
10 changes: 10 additions & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,16 @@ project (GFE
)
cmake_policy(SET CMP0074 NEW)

# Most users of this software do not (should not?) have permissions to
# install in the cmake default of /usr/local (or equiv on other os's).
# Below, the default is changed to a directory within the build tree
# unless the user explicitly sets CMAKE_INSTALL_PREFIX in the cache.
if (CMAKE_INSTALL_PREFIX_INITIALIZED_TO_DEFAULT)
set (CMAKE_INSTALL_PREFIX "${CMAKE_BINARY_DIR}/installed" CACHE PATH "default install path" FORCE )
message(STATUS "*** Setting default install prefix to ${CMAKE_INSTALL_PREFIX}.")
message(STATUS "*** Override with -DCMAKE_INSTALL_PREFIX=<path>.")
endif()

enable_testing()

add_subdirectory(gFTL)
Expand Down
11 changes: 11 additions & 0 deletions ChangeLog.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

## [1.8.0] - 2022-11-15

### Changed

- Change behavior if user does not pass in `CMAKE_INSTALL_PREFIX` to put install directory inside build (as we are know we can
write there; default for CMake install is `/usr/local`)

### Added

- Added `CODEOWNERS` file

## [1.7.0] - 2022-11-15

### Changed
Expand Down

0 comments on commit 6018886

Please sign in to comment.