arcolight commented on code in PR #7929:
URL: https://github.com/apache/ignite-3/pull/7929#discussion_r3033310925
##########
modules/platforms/cpp/tests/package-test/compile_public_headers/CMakeLists.txt:
##########
@@ -49,13 +49,18 @@ find_package(ignite REQUIRED COMPONENTS client)
include("${IGNITE_HEADERS_LIST_FILE}")
foreach(H IN LISTS IGNITE_PUBLIC_HEADERS)
- # Derive a CMake-identifier-safe target name from the header path.
- string(MAKE_C_IDENTIFIER "${H}" CPH_SAFE_NAME)
+ # Use a short MD5 hash of the header path as the target/file name.
+ # A full MAKE_C_IDENTIFIER name can exceed 260-char Windows path limits
+ # when combined with MSBuild's .tlog directory structure (VS 2017 does not
+ # honour the LongPathsEnabled registry key).
+ string(MD5 CPH_HASH "${H}")
+ string(SUBSTRING "${CPH_HASH}" 0 8 CPH_SHORT)
+ message(STATUS "HEADER MAP: ${H} -> ${CPH_HASH}")
Review Comment:
Fixed
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
