---

yaml
---
r: 316655
b: refs/heads/master-rebranch
c: 93aa7c1
h: refs/heads/master
i:
  316653: 672f40d
  316651: 52c7929
  316647: 2ccda9c
  316639: 86811df

Author: swift-ci

[refs]

@@ -1457,4 +1457,4 @@ refs/tags/swift-DEVELOPMENT-SNAPSHOT-2019-08-02-a: ddd2b2976aa9bfde5f20fe37f6bd2
 refs/tags/swift-DEVELOPMENT-SNAPSHOT-2019-08-03-a: 171cc166f2abeb5ca2a4003700a8a78a108bd300
 refs/heads/benlangmuir-patch-1: baaebaf39d52f3bf36710d4fe40cf212e996b212
 refs/heads/i-do-redeclare: 8c4e6d5de5c1e3f0a2cedccf319df713ea22c48e
-refs/heads/master-rebranch: 2bcd5e4d1dae90d7608085744ce1da6fd47637aa
+refs/heads/master-rebranch: 93aa7c19cdcf8f043124dc340c324ca830fef7d1

branches/master-rebranch/test/remote-run/custom-options.test-sh

@@ -3,11 +3,6 @@ RUN: %utils/remote-run -n --remote-dir /xyz-REMOTE -o FIRST_OPT -o SECOND_OPT --
 CHECK: /usr/bin/ssh -n
 CHECK-DAG: -p 12345
 CHECK-DAG: -o FIRST_OPT -o SECOND_OPT
-CHECK-SAME: some_user@some_host -- '/usr/bin/env' '/bin/rm' '-rf' '{{.+}}-REMOTE/output'
-
-CHECK-NEXT: /usr/bin/ssh -n
-CHECK-DAG: -p 12345
-CHECK-DAG: -o FIRST_OPT -o SECOND_OPT
 CHECK-SAME: some_user@some_host -- '/usr/bin/env' '/bin/mkdir' '-p' '{{.+}}-REMOTE/output/nested'
 
 CHECK-NEXT: /usr/bin/sftp

branches/master-rebranch/test/remote-run/identity.test-sh

@@ -3,11 +3,6 @@ RUN: %utils/remote-run -n --remote-dir /xyz-REMOTE -i spiderman --output-prefix
 CHECK: /usr/bin/ssh -n
 CHECK-DAG: -p 12345
 CHECK-DAG: -i spiderman
-CHECK-SAME: some_user@some_host -- '/usr/bin/env' '/bin/rm' '-rf' '{{.+}}-REMOTE/output'
-
-CHECK-NEXT: /usr/bin/ssh -n
-CHECK-DAG: -p 12345
-CHECK-DAG: -i spiderman
 CHECK-SAME: some_user@some_host -- '/usr/bin/env' '/bin/mkdir' '-p' '{{.+}}-REMOTE/output/nested'
 
 CHECK-NEXT: /usr/bin/sftp

branches/master-rebranch/utils/remote-run

@@ -28,7 +28,7 @@ class CommandRunner(object):
 
     @staticmethod
     def _dirnames(files):
-        return sorted(set(os.path.dirname(f) for f in files))
+        return list(set(os.path.dirname(f) for f in files))
 
     def popen(self, command, **kwargs):
         if self.verbose:
@@ -195,32 +195,23 @@ def main():
     runner.dry_run = args.dry_run
     runner.verbose = args.verbose or args.dry_run
 
-    assert not args.remote_dir == '/'
-
     upload_files = dict()
     download_files = dict()
-    remote_test_specific_dir = None
     if args.input_prefix:
-        assert not args.remote_input_prefix.startswith("..")
         remote_dir = os.path.join(args.remote_dir, args.remote_input_prefix)
         input_files = find_transfers(args.command, args.input_prefix, 
                                      remote_dir)
         assert not any(upload_files.has_key(f) for f in input_files)
         upload_files.update(input_files)
     if args.output_prefix:
-        assert not args.remote_output_prefix.startswith("..")
         remote_dir = os.path.join(args.remote_dir, args.remote_output_prefix)
         test_files = find_transfers(args.command, args.output_prefix, 
                                     remote_dir)
         assert not any(upload_files.has_key(f) for f in test_files)
         upload_files.update(test_files)
         assert not any(download_files.has_key(f) for f in test_files)
         download_files.update(test_files)
-        remote_test_specific_dir = remote_dir
 
-    if remote_test_specific_dir:
-        assert remote_test_specific_dir.startswith(args.remote_dir)
-        runner.run_remote(['/bin/rm', '-rf', remote_test_specific_dir])
     if upload_files:
         runner.send(upload_files)
 

branches/master-rebranch/validation-test/Reflection/lit.local.cfg

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