CheckStructPacking: use LINK_OPTIONS

revert of 0cce80d Debian buster is now oldoldstable, and there are other features in cmake 3.14 that I'll eventually use.
thpatch · Oct 29, 2023 · 80a3697 · 80a3697
1 parent 663477e
commit 80a3697
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/INSTALL b/INSTALL
@@ -1,5 +1,5 @@
 Build requirements:
-  cmake >=3.13
+  cmake >=3.14
   flex >=2.5.31
   bison >=3.6.4
   (optional) pkg-config

diff --git a/cmake/CheckStructPacking.cmake b/cmake/CheckStructPacking.cmake
@@ -10,8 +10,7 @@ function(__check_struct_packing_impl what preamble var)
   configure_file(${__check_struct_packing_dir}/CheckStructPacking.c.in ${src} @ONLY)
   try_compile(${var} ${CMAKE_BINARY_DIR} ${src}
     COMPILE_DEFINITIONS ${CMAKE_REQUIRED_DEFINITIONS}
-    # CMake 3.14
-    #LINK_OPTIONS ${CMAKE_REQUIRED_LINK_OPTIONS}
+    LINK_OPTIONS ${CMAKE_REQUIRED_LINK_OPTIONS}
     LINK_LIBRARIES ${CMAKE_REQUIRED_LIBRARIES}
     CMAKE_FLAGS
       "-DCOMPILE_DEFINITIONS:STRING=${CMAKE_REQUIRED_FLAGS}"