Skip to content

Commit 57d03df

Browse files
Joe Shajrawimikeash
authored andcommitted
[5.1] lit.cfg: Respect the DYLD_LIBRARY_PATH restrictions when not using %target-run
Some test cases use something like ``` config.target_run_simple_swift = ( '%%empty-directory(%%t) && ' '%s %s %%s -o %%t/a.out -module-name main && ' '%s %%t/a.out &&' '%s %%t/a.out’ ``` That does not respect the %target-run DYLD_LIBRARY_PATH for choosing the just built libraries and/or OS libraries. This PR respects said library path in such situations rdar://problem/49835064
1 parent 7c97325 commit 57d03df

File tree

1 file changed

+41
-47
lines changed

1 file changed

+41
-47
lines changed

test/lit.cfg

Lines changed: 41 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -1077,53 +1077,6 @@ if sftp_server_path:
10771077
config.substitutions.append(('%sftp-server',
10781078
sftp_server_path or 'no-sftp-server'))
10791079

1080-
1081-
if not getattr(config, 'target_run_simple_swift', None):
1082-
config.target_run_simple_swift = (
1083-
'%%empty-directory(%%t) && '
1084-
'%s %s %%s -o %%t/a.out -module-name main && '
1085-
'%s %%t/a.out &&'
1086-
'%s %%t/a.out'
1087-
% (config.target_build_swift, mcp_opt, config.target_codesign, config.target_run))
1088-
config.target_run_simple_opt_O_swift = (
1089-
'%%empty-directory(%%t) && '
1090-
'%s %s -O %%s -o %%t/a.out -module-name main && '
1091-
'%s %%t/a.out &&'
1092-
'%s %%t/a.out'
1093-
% (config.target_build_swift, mcp_opt, config.target_codesign, config.target_run))
1094-
config.target_run_simple_opt_Osize_swift = (
1095-
'%%empty-directory(%%t) && '
1096-
'%s %s -Osize %%s -o %%t/a.out -module-name main && '
1097-
'%s %%t/a.out &&'
1098-
'%s %%t/a.out'
1099-
% (config.target_build_swift, mcp_opt, config.target_codesign, config.target_run))
1100-
config.target_run_stdlib_swift = (
1101-
'%%empty-directory(%%t) && '
1102-
'%s %s %%s -o %%t/a.out -module-name main '
1103-
'-Xfrontend -disable-access-control && '
1104-
'%s %%t/a.out &&'
1105-
'%s %%t/a.out'
1106-
% (config.target_build_swift, mcp_opt, config.target_codesign, config.target_run))
1107-
config.target_run_simple_swiftgyb = (
1108-
'%%empty-directory(%%t) && '
1109-
'%%gyb %%s -o %%t/main.swift && '
1110-
'%%line-directive %%t/main.swift -- '
1111-
'%s %s %%t/main.swift -o %%t/a.out -module-name main && '
1112-
'%s %%t/a.out &&'
1113-
'%%line-directive %%t/main.swift -- '
1114-
'%s %%t/a.out'
1115-
% (config.target_build_swift, mcp_opt, config.target_codesign, config.target_run))
1116-
config.target_run_stdlib_swiftgyb = (
1117-
'%%empty-directory(%%t) && '
1118-
'%%gyb %%s -o %%t/main.swift && '
1119-
'%%line-directive %%t/main.swift -- '
1120-
'%s %s %%t/main.swift -o %%t/a.out -module-name main '
1121-
'-Xfrontend -disable-access-control && '
1122-
'%s %%t/a.out &&'
1123-
'%%line-directive %%t/main.swift -- '
1124-
'%s %%t/a.out'
1125-
% (config.target_build_swift, mcp_opt, config.target_codesign, config.target_run))
1126-
11271080
subst_target_jit_run = ""
11281081
if 'swift_interpreter' in config.available_features:
11291082
subst_target_jit_run = (
@@ -1212,6 +1165,47 @@ if not kIsWindows:
12121165
"LD_LIBRARY_PATH='{0}' " # Linux option
12131166
"SIMCTL_CHILD_DYLD_LIBRARY_PATH='{0}' " # Simulator option
12141167
.format(all_stdlib_path)) + config.target_run
1168+
1169+
if not getattr(config, 'target_run_simple_swift', None):
1170+
config.target_run_simple_swift_parameterized = \
1171+
(SubstituteCaptures('%%empty-directory(%%t) && '
1172+
'%s %s %%s \\1 -o %%t/a.out -module-name main && '
1173+
'%s %%t/a.out &&'
1174+
'%s %%t/a.out' % (config.target_build_swift,
1175+
mcp_opt, config.target_codesign,
1176+
config.target_run)))
1177+
config.target_run_simple_swift = (
1178+
'%%empty-directory(%%t) && '
1179+
'%s %s %%s -o %%t/a.out -module-name main && '
1180+
'%s %%t/a.out &&'
1181+
'%s %%t/a.out'
1182+
% (config.target_build_swift, mcp_opt, config.target_codesign, config.target_run))
1183+
config.target_run_stdlib_swift = (
1184+
'%%empty-directory(%%t) && '
1185+
'%s %s %%s -o %%t/a.out -module-name main '
1186+
'-Xfrontend -disable-access-control && '
1187+
'%s %%t/a.out &&'
1188+
'%s %%t/a.out'
1189+
% (config.target_build_swift, mcp_opt, config.target_codesign, config.target_run))
1190+
config.target_run_simple_swiftgyb = (
1191+
'%%empty-directory(%%t) && '
1192+
'%%gyb %%s -o %%t/main.swift && '
1193+
'%%line-directive %%t/main.swift -- '
1194+
'%s %s %%t/main.swift -o %%t/a.out -module-name main && '
1195+
'%s %%t/a.out &&'
1196+
'%%line-directive %%t/main.swift -- '
1197+
'%s %%t/a.out'
1198+
% (config.target_build_swift, mcp_opt, config.target_codesign, config.target_run))
1199+
config.target_run_stdlib_swiftgyb = (
1200+
'%%empty-directory(%%t) && '
1201+
'%%gyb %%s -o %%t/main.swift && '
1202+
'%%line-directive %%t/main.swift -- '
1203+
'%s %s %%t/main.swift -o %%t/a.out -module-name main '
1204+
'-Xfrontend -disable-access-control && '
1205+
'%s %%t/a.out &&'
1206+
'%%line-directive %%t/main.swift -- '
1207+
'%s %%t/a.out'
1208+
% (config.target_build_swift, mcp_opt, config.target_codesign, config.target_run))
12151209

12161210
#
12171211
# When changing substitutions, update docs/Testing.rst.

0 commit comments

Comments
 (0)