diff options
author | Pavel Labath <labath@google.com> | 2018-03-16 12:04:46 +0000 |
---|---|---|
committer | Pavel Labath <labath@google.com> | 2018-03-16 12:04:46 +0000 |
commit | 63047d05c040c8377ba493b3e0ea6176c402edc1 (patch) | |
tree | 49e17a59da9bd2831aa5f8598b7efcaa87a0522c /lldb/packages/Python/lldbsuite/test/macosx/add-dsym/TestAddDsymMidExecutionCommand.py | |
parent | eacfefd0563fa886e6d34758867150e6f6eae13e (diff) | |
download | bcm5719-llvm-63047d05c040c8377ba493b3e0ea6176c402edc1.tar.gz bcm5719-llvm-63047d05c040c8377ba493b3e0ea6176c402edc1.zip |
[dotest] Clean up test folder clean-up
Summary:
This patch implements a unified way of cleaning the build folder of each
test. This is done by completely removing the build folder before each
test, in the respective setUp() method. Previously, we were using a
combination of several methods, each with it's own drawbacks:
- nuking the entire build tree before running dotest: the issue here is
that this did not take place if you ran dotest manually
- running "make clean" before the main "make" target: this relied on the
clean command being correctly implemented. This was usually true, but
not always.
- for files which were not produced by make, each python file was
responsible for ensuring their deleting, using a variety of methods.
With this approach, the previous methods become redundant. I remove the
first two, since they are centralized. For the other various bits of
clean-up code in python files, I indend to delete it when I come
across it.
Reviewers: aprantl
Subscribers: emaste, ki.stfu, mgorny, eraman, lldb-commits
Differential Revision: https://reviews.llvm.org/D44526
llvm-svn: 327703
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/macosx/add-dsym/TestAddDsymMidExecutionCommand.py')
-rw-r--r-- | lldb/packages/Python/lldbsuite/test/macosx/add-dsym/TestAddDsymMidExecutionCommand.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lldb/packages/Python/lldbsuite/test/macosx/add-dsym/TestAddDsymMidExecutionCommand.py b/lldb/packages/Python/lldbsuite/test/macosx/add-dsym/TestAddDsymMidExecutionCommand.py index 7e96d90fa1a..be122f4f500 100644 --- a/lldb/packages/Python/lldbsuite/test/macosx/add-dsym/TestAddDsymMidExecutionCommand.py +++ b/lldb/packages/Python/lldbsuite/test/macosx/add-dsym/TestAddDsymMidExecutionCommand.py @@ -25,7 +25,7 @@ class AddDsymMidExecutionCommandCase(TestBase): @no_debug_info_test # Prevent the genaration of the dwarf version of this test def test_add_dsym_mid_execution(self): """Test that add-dsym mid-execution loads the symbols at the right place for a slid binary.""" - self.buildDefault(clean=True, dictionary={'MAKE_DSYM':'YES'}) + self.buildDefault(dictionary={'MAKE_DSYM':'YES'}) exe = self.getBuildArtifact("a.out") self.target = self.dbg.CreateTarget(exe) |