Skip to content

Commit d8d9dc4

Browse files
committed
Include target variant suffix in swift-reflection-test dependencies
swift-reflection-test isn't a standalone host-side tool anymore, but a target executable, so the dependency needs to include the variant suffix (e.g. -macosx-x86_64) in the name.
1 parent d9f24d0 commit d8d9dc4

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

test/CMakeLists.txt

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ function(get_test_dependencies SDK result_var_name)
4141

4242
set(deps_binaries
4343
swift swift-ide-test sil-opt swift-llvm-opt swift-demangle sil-extract
44-
lldb-moduleimport-test swift-reflection-dump swift-reflection-test)
44+
lldb-moduleimport-test swift-reflection-dump)
4545
if(NOT SWIFT_BUILT_STANDALONE)
4646
list(APPEND deps_binaries llc)
4747
endif()
@@ -184,6 +184,9 @@ if(PYTHONINTERP_FOUND)
184184
list(APPEND test_dependencies
185185
"swift-test-stdlib-${SWIFT_SDK_${SDK}_LIB_SUBDIR}")
186186

187+
list(APPEND test_dependencies
188+
"swift-reflection-test${VARIANT_SUFFIX}")
189+
187190
set(validation_test_dependencies
188191
"swiftStdlibCollectionUnittest-${SWIFT_SDK_${SDK}_LIB_SUBDIR}")
189192

0 commit comments

Comments
 (0)