]> git.ipfire.org Git - thirdparty/xz.git/commitdiff
CMake: Always add pthread flags into CMAKE_REQUIRED_LIBRARIES
authorLasse Collin <lasse.collin@tukaani.org>
Tue, 25 Jun 2024 12:51:48 +0000 (15:51 +0300)
committerLasse Collin <lasse.collin@tukaani.org>
Tue, 25 Jun 2024 12:56:39 +0000 (15:56 +0300)
It was weird to add CMAKE_THREAD_LIBS_INIT in CMAKE_REQUIRED_LIBRARIES
only if CLOCK_MONOTONIC is available. Alternative would be to remove
the thread libs from CMAKE_REQUIRED_LIBRARIES after the check for
pthread_condattr_setclock() but keeping the libs should be fine too.
Then it's ready in case more pthread functions were wanted some day.

CMakeLists.txt

index 9db8cb81f3e7a802b589d552dec4ff4520821362..011aa1999162d33dc62e22bc82a00ffab1dae5fa 100644 (file)
@@ -645,11 +645,13 @@ if(XZ_THREADS)
             target_link_libraries(liblzma PRIVATE Threads::Threads)
             add_compile_definitions(MYTHREAD_POSIX)
 
+            # Make the thread libs available in later checks. In practice
+            # only pthread_condattr_setclock check should need this.
+            list(INSERT CMAKE_REQUIRED_LIBRARIES 0 "${CMAKE_THREAD_LIBS_INIT}")
+
             # Check if pthread_condattr_setclock() exists to
             # use CLOCK_MONOTONIC.
             if(HAVE_CLOCK_MONOTONIC)
-                list(INSERT CMAKE_REQUIRED_LIBRARIES 0
-                     "${CMAKE_THREAD_LIBS_INIT}")
                 check_symbol_exists(pthread_condattr_setclock pthread.h
                                     HAVE_PTHREAD_CONDATTR_SETCLOCK)
                 tuklib_add_definition_if(ALL HAVE_PTHREAD_CONDATTR_SETCLOCK)