Archives/ohwgiles_laminar
1
0
Fork 0
mirror of https://github.com/ohwgiles/laminar.git synced 2025-06-13 12:54:29 +00:00
Code Issues Projects Releases Wiki Activity

Fix sourcing the wrong file location, remove conditional example writeout.

Browse Source
This commit is contained in:
Tessa L. H. Lovelace 2019-06-30 06:49:16 -07:00
parent a73c945f45
commit 06834fddff
1 changed files with 4 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
CMakeLists.txt
View File

@ -123,21 +123,12 @@ install(FILES etc/laminarc-completion.zsh DESTINATION ${ZSH_COMPLETIONS_DIR} REN
install(TARGETS laminard laminarc RUNTIME DESTINATION usr/bin)
install(FILES etc/laminar.service DESTINATION ${SYSTEMD_UNITDIR})
# An initial attempt at solving https://github.com/ohwgiles/laminar/issues/92
# Don't overwrite the config file if it already exists at the target location
# @todo - migration/override script if the defaults change or the options get pruned?
if(NOT EXISTS "${CMAKE_INSTALL_PREFIX}etc/laminar.conf")
install(
FILES ${CMAKE_INSTALL_PREFIX}etc/laminar.conf
DESTINATION etc
)
else(NOT EXISTS "${CMAKE_INSTALL_PREFIX}etc/laminar.conf")
# hardcoded values are examples of technical debt, and in an ideal world, are eliminated.
MESSAGE("-- A ${CMAKE_INSTALL_PREFIX}etc/laminar.conf file exists, will create ${CMAKE_INSTALL_PREFIX}etc/laminar.conf.example instead of overwriting.")
# This also displays at config time, rather than install time.
# In an ideal world, the user gets offered the option to overwrite, compare, or keep/copy the file
install(
FILES ${CMAKE_INSTALL_PREFIX}etc/laminar.conf
DESTINATION etc
RENAME "laminar.conf.example"
FILES etc/laminar.conf
DESTINATION ${CMAKE_INSTALL_PREFIX}etc
)
endif(NOT EXISTS "${CMAKE_INSTALL_PREFIX}etc/laminar.conf")