martinzink commented on code in PR #1487:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1487#discussion_r1065465776


##########
cmake/BundledLibXml2.cmake:
##########
@@ -38,6 +38,10 @@ function(use_bundled_libxml2 SOURCE_DIR BINARY_DIR)
                 
"-DCMAKE_INSTALL_PREFIX=${BINARY_DIR}/thirdparty/libxml2-install")
     endif()
 
+    if (CMAKE_VERSION VERSION_GREATER_EQUAL 3.24)
+        cmake_policy(SET CMP0135 OLD) # Restore the timestamps from the 
archive https://gitlab.kitware.com/cmake/cmake/-/issues/24003
+    endif()

Review Comment:
   Yeah that was my original intent, but the latest cmake on centos7 doesnt 
have this feature, so that would only work if I duplicated the whole 
ExternalProject_Add into an `if (CMAKE_VERSION VERSION_GREATER_EQUAL 3.24)` 
statement. 
   
   I also tried to put the "DOWNLOAD_EXTRACT_TIMESTAMP ON" and "" (based on 
cmakeversion) into a variable and pass that into the ExternalProject_Add but I 
wanst able to make that work.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to