gnucash master: Can't set target link options until we've defined the target.

John Ralls jralls at code.gnucash.org
Fri Jun 12 17:22:37 EDT 2020


Updated	 via  https://github.com/Gnucash/gnucash/commit/e5a14b89 (commit)
	from  https://github.com/Gnucash/gnucash/commit/49e394e3 (commit)



commit e5a14b89463344b54c5c7fe926f707c5781dc04d
Author: John Ralls <jralls at ceridwen.us>
Date:   Fri Jun 12 23:22:27 2020 +0200

    Can't set target link options until we've defined the target.

diff --git a/gnucash/CMakeLists.txt b/gnucash/CMakeLists.txt
index 9e41c3c51..cda38cf1c 100644
--- a/gnucash/CMakeLists.txt
+++ b/gnucash/CMakeLists.txt
@@ -74,7 +74,6 @@ set(gnucash_cli_SOURCES
 
 if (MINGW)
   list(APPEND gnucash_cli_SOURCES "gnucash-windows-locale.c")
-  target_link_options(gnucash-cli PRIVATE -mconsole)
 endif()
 
 add_executable (gnucash-cli
@@ -83,6 +82,10 @@ add_executable (gnucash-cli
     ${gnucash_noinst_HEADERS}
 )
 
+if (MINGW)
+    target_link_options(gnucash-cli PRIVATE -mconsole)
+endif()
+
 add_dependencies (gnucash-cli gnucash)
 
 target_compile_definitions(gnucash-cli PRIVATE -DG_LOG_DOMAIN=\"gnc.bin\")



Summary of changes:
 gnucash/CMakeLists.txt | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)



More information about the gnucash-changes mailing list