Print

Print


This is a follow-up to the discussion in #1506.
Using ${CMAKE_THREAD_LIBS_INIT} everywhere is more consistent than doing a one-off for the one place where it is strictly necessary.
This also avoids that things break elsewhere in the future if some other part of the code starts using new parts of std::atomic that need libatomic.

You can view, comment on, or merge this pull request online at:

  https://github.com/xrootd/xrootd/pull/1509

-- Commit Summary --

  * Find pthreads using the CMake Thread module
  * Revert "Link to libatomic on riscv64"
  * Correct the variable name from ${ZLIB_LIBRARY} to ${ZLIB_LIBRARIES}
  * Use ${CMAKE_DL_LIBS} consistently

-- File Changes --

    M cmake/XRootDSystemCheck.cmake (6)
    M src/XrdApps.cmake (8)
    M src/XrdCl/CMakeLists.txt (6)
    M src/XrdCrypto.cmake (2)
    M src/XrdDaemons.cmake (4)
    M src/XrdFfs.cmake (4)
    M src/XrdFrm.cmake (8)
    M src/XrdHttp.cmake (2)
    M src/XrdOssCsi.cmake (2)
    M src/XrdPfc.cmake (2)
    M src/XrdPlugins.cmake (4)
    M src/XrdPosix.cmake (2)
    M src/XrdSciTokens.cmake (4)
    M src/XrdSec.cmake (8)
    M src/XrdSecgsi.cmake (2)
    M src/XrdServer.cmake (9)
    M src/XrdSsi.cmake (6)
    M src/XrdTpc.cmake (2)
    M src/XrdUtils.cmake (2)
    M src/XrdXml.cmake (2)
    M tests/XrdCephTests/CMakeLists.txt (4)
    M tests/XrdClTests/CMakeLists.txt (4)
    M tests/XrdClTests/tls/CMakeLists.txt (6)
    M tests/common/CMakeLists.txt (18)

-- Patch Links --

https://github.com/xrootd/xrootd/pull/1509.patch
https://github.com/xrootd/xrootd/pull/1509.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/pull/1509

########################################################################
Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1