Skip to content

Commit c52809b

Browse files
committed
[test] One last tweak to "%target-run-reflection-test"
Copy %target-swift-reflection-test into %t instead of using a hard link. The problem with hard links is that they can be overwritten when the test fails, which would then truncate the original executable to an empty file!
1 parent bcb10b5 commit c52809b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

validation-test/Reflection/lit.local.cfg

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22
config.substitutions = list(config.substitutions)
33

44
config.substitutions.insert(0, ("%target-run-reflection-test",
5-
# Link %target-swift-reflection-test into %t to convince %target-run to copy
5+
# Copy %target-swift-reflection-test into %t to convince %target-run to copy
66
# it, then use DYLD_LIBRARY_PATH to make sure it can find SwiftRemoteMirrors
77
# from its new location when /not/ run remotely.
8-
("ln %target-swift-reflection-test %t/swift-reflection-test && "
8+
("cp %target-swift-reflection-test %t/swift-reflection-test && "
99
"env %env-DYLD_LIBRARY_PATH=%platform-module-dir/../ %target-run "
1010
" %t/swift-reflection-test")))

0 commit comments

Comments
 (0)