diff options
author | Pavel Labath <labath@google.com> | 2017-03-20 16:07:17 +0000 |
---|---|---|
committer | Pavel Labath <labath@google.com> | 2017-03-20 16:07:17 +0000 |
commit | 6b42b3b7a3577d3d61a6d7552f147b6d2afef737 (patch) | |
tree | 2df801d85807b5f7d4baff65e17296ec64b78f0f /lldb/packages/Python/lldbsuite/test/dotest.py | |
parent | 5d59a4ee19cfe1580932fb5315c1bad5cb8ef767 (diff) | |
download | bcm5719-llvm-6b42b3b7a3577d3d61a6d7552f147b6d2afef737.tar.gz bcm5719-llvm-6b42b3b7a3577d3d61a6d7552f147b6d2afef737.zip |
Fix remote test suite directory creation
r298203 make SBPlatform::MakeDirectory less recursive, which breaks the
test suite creation of test directory hierarchy creation on the remote
target. Since the function was never fully recursive, and the name does
not imply recursiveness, I fix the problem by modifying the test runner
to do the recursion manually.
I also make the runner complain more loudly when it fails to create the
directory -- previously it just printed the error to stdout and caused
most of the tests to hang, which is not very helpful in diagnosing the
problem.
llvm-svn: 298261
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/dotest.py')
-rw-r--r-- | lldb/packages/Python/lldbsuite/test/dotest.py | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/lldb/packages/Python/lldbsuite/test/dotest.py b/lldb/packages/Python/lldbsuite/test/dotest.py index aa0c2ff9334..a6bb01ea608 100644 --- a/lldb/packages/Python/lldbsuite/test/dotest.py +++ b/lldb/packages/Python/lldbsuite/test/dotest.py @@ -1147,8 +1147,15 @@ def run_suite(): if configuration.lldb_platform_working_dir: print("Setting remote platform working directory to '%s'..." % (configuration.lldb_platform_working_dir)) - lldb.remote_platform.SetWorkingDirectory( - configuration.lldb_platform_working_dir) + error = lldb.remote_platform.MakeDirectory( + configuration.lldb_platform_working_dir, 448) # 448 = 0o700 + if error.Fail(): + raise Exception("making remote directory '%s': %s" % ( + remote_test_dir, error)) + + if not lldb.remote_platform.SetWorkingDirectory( + configuration.lldb_platform_working_dir): + raise Exception("failed to set working directory '%s'" % remote_test_dir) lldb.DBG.SetSelectedPlatform(lldb.remote_platform) else: lldb.remote_platform = None |