Skip to content

Commit 7f00a1c

Browse files
authored
Store Proxy Verifier in .git (#12664)
- Store proxy-verifier in .git instead of the cmake build directory to share across build directories. - Fix AUTEST_OPTIONS space escaping issue in cmake targets.
1 parent 129a638 commit 7f00a1c

File tree

3 files changed

+14
-9
lines changed

3 files changed

+14
-9
lines changed

CMakeLists.txt

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -589,6 +589,11 @@ if(ENABLE_AUTEST)
589589
""
590590
CACHE STRING "Additional options for autest (default \"\")"
591591
)
592+
# Also create AUTEST_OPTIONS_LIST from the string for use in cmake targets.
593+
# This prevents cmake from escaping spaces in the arguments, which confuses
594+
# the autest command. The original AUTEST_OPTIONS string is used in the
595+
# autest.sh script.
596+
separate_arguments(AUTEST_OPTIONS_LIST UNIX_COMMAND "${AUTEST_OPTIONS}")
592597
set(PROXY_VERIFIER_VERSION "v2.12.0")
593598
set(PROXY_VERIFIER_HASH "SHA1=8e9adc6e0b31251ca8e6fc2064ae54e5d752bb72")
594599
include(proxy-verifier)

cmake/proxy-verifier.cmake

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
#
1616
#######################
1717

18-
# This will download and extract proxy-verifier and setup variables to point to it.
18+
# This will download and extract proxy-verifier to .git directory and setup variables to point to it.
1919
#
2020
# Required variables:
2121
# PROXY_VERIFIER_VERSION
@@ -35,14 +35,14 @@ if(NOT PROXY_VERIFIER_HASH)
3535
message(FATAL_ERROR "PROXY_VERIFIER_HASH Required")
3636
endif()
3737

38-
# Download proxy-verifier
39-
set(PV_ARCHIVE ${CMAKE_BINARY_DIR}/proxy-verifier/proxy-verifier.tar.gz)
38+
# Download proxy-verifier to .git directory.
39+
set(PV_ARCHIVE ${CMAKE_SOURCE_DIR}/.git/proxy-verifier/proxy-verifier.tar.gz)
4040
file(
4141
DOWNLOAD https://ci.trafficserver.apache.org/bintray/proxy-verifier-${PROXY_VERIFIER_VERSION}.tar.gz ${PV_ARCHIVE}
4242
EXPECTED_HASH ${PROXY_VERIFIER_HASH}
4343
SHOW_PROGRESS
4444
)
45-
file(ARCHIVE_EXTRACT INPUT ${PV_ARCHIVE})
45+
file(ARCHIVE_EXTRACT INPUT ${PV_ARCHIVE} DESTINATION ${CMAKE_SOURCE_DIR}/.git)
4646

4747
if(CMAKE_HOST_SYSTEM_NAME STREQUAL "Linux")
4848
if(CMAKE_HOST_SYSTEM_PROCESSOR STREQUAL "x86_64"
@@ -70,7 +70,7 @@ else()
7070
message(FATAL_ERROR "Host ${CMAKE_HOST_SYSTEM_NAME} doesnt support running proxy verifier")
7171
endif()
7272

73-
set(PROXY_VERIFIER_PATH ${CMAKE_BINARY_DIR}/proxy-verifier-${PROXY_VERIFIER_VERSION}/${PV_SUBDIR})
73+
set(PROXY_VERIFIER_PATH ${CMAKE_SOURCE_DIR}/.git/proxy-verifier-${PROXY_VERIFIER_VERSION}/${PV_SUBDIR})
7474
set(PROXY_VERIFIER_CLIENT ${PROXY_VERIFIER_PATH}/verifier-client)
7575
set(PROXY_VERIFIER_SERVER ${PROXY_VERIFIER_PATH}/verifier-server)
7676

tests/CMakeLists.txt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ add_custom_target(
6464
COMMAND
6565
${CMAKE_COMMAND} -E env PYTHONPATH=${CMAKE_GOLD_DIR}/remap:$ENV{PYTHONPATH} ${RUNPIPENV} run env autest --directory
6666
${CMAKE_GOLD_DIR} --ats-bin=${CMAKE_INSTALL_PREFIX}/bin --proxy-verifier-bin ${PROXY_VERIFIER_PATH} --build-root
67-
${CMAKE_BINARY_DIR} --sandbox ${AUTEST_SANDBOX} ${CURL_UDS_FLAG} ${AUTEST_OPTIONS}
67+
${CMAKE_BINARY_DIR} --sandbox ${AUTEST_SANDBOX} ${CURL_UDS_FLAG} ${AUTEST_OPTIONS_LIST}
6868
WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
6969
USES_TERMINAL
7070
)
@@ -75,7 +75,7 @@ add_custom_target(
7575
COMMAND
7676
${CMAKE_COMMAND} -E env PYTHONPATH=${CMAKE_GOLD_DIR}/remap:$ENV{PYTHONPATH} ${RUNPIPENV} run env autest --directory
7777
${CMAKE_GOLD_DIR} --ats-bin=${CMAKE_INSTALL_PREFIX}/bin --proxy-verifier-bin ${PROXY_VERIFIER_PATH} --build-root
78-
${CMAKE_BINARY_DIR} --sandbox ${AUTEST_SANDBOX} ${CURL_UDS_FLAG} ${AUTEST_OPTIONS}
78+
${CMAKE_BINARY_DIR} --sandbox ${AUTEST_SANDBOX} ${CURL_UDS_FLAG} ${AUTEST_OPTIONS_LIST}
7979
WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
8080
USES_TERMINAL
8181
)
@@ -91,8 +91,8 @@ add_custom_target(
9191
COMMAND
9292
${CMAKE_COMMAND} -E env PYTHONPATH=${CMAKE_GOLD_DIR}/remap:$ENV{PYTHONPATH} ${RUNPIPENV} run env autest --directory
9393
${CMAKE_GOLD_DIR} --ats-bin=${CMAKE_INSTALL_PREFIX}/bin --proxy-verifier-bin ${PROXY_VERIFIER_PATH} --build-root
94-
${CMAKE_BINARY_DIR} --sandbox ${AUTEST_SANDBOX} ${CURL_UDS_FLAG} ${AUTEST_OPTIONS}
95-
COMMAND ${CMAKE_COMMAND} -E rename {CMAKE_SKIP_GOLD_DIR}/h2 ${CMAKE_GOLD_DIR}/h2
94+
${CMAKE_BINARY_DIR} --sandbox ${AUTEST_SANDBOX} ${CURL_UDS_FLAG} ${AUTEST_OPTIONS_LIST}
95+
COMMAND ${CMAKE_COMMAND} -E rename ${CMAKE_SKIP_GOLD_DIR}/h2 ${CMAKE_GOLD_DIR}/h2
9696
COMMAND ${CMAKE_COMMAND} -E rename ${CMAKE_SKIP_GOLD_DIR}/tls ${CMAKE_GOLD_DIR}/tls
9797
COMMAND ${CMAKE_COMMAND} -E rename ${CMAKE_SKIP_GOLD_DIR}/tls_hooks ${CMAKE_GOLD_DIR}/tls_hooks
9898
WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}

0 commit comments

Comments
 (0)