[Lldb-commits] [lldb] r256961 - fixup lldbinline-style tests to clean up Makefiles and *.d files

[Todd Fiala via lldb-commits]

Author: tfiala
Date: Wed Jan  6 13:16:45 2016
New Revision: 256961

URL: http://llvm.org/viewvc/llvm-project?rev=256961&view=rev
Log:
fixup lldbinline-style tests to clean up Makefiles and *.d files

The lldbinline inline-test mechanism will create a Makefile
if one does not exist in the test directory.  This Makefile
and its *.d files were not getting cleaned up after a test run,
leaving trash in the source tree.

Modified:
    lldb/trunk/packages/Python/lldbsuite/test/lldbinline.py

Modified: lldb/trunk/packages/Python/lldbsuite/test/lldbinline.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/lldbinline.py?rev=256961&r1=256960&r2=256961&view=diff
==============================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/lldbinline.py (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/lldbinline.py Wed Jan  6 13:16:45 2016
@@ -3,7 +3,6 @@ from __future__ import absolute_import
 
 # System modules
 import os
-import sys
 
 # Third-party modules
 
@@ -23,6 +22,7 @@ def source_type(filename):
         '.mm' : 'OBJCXX_SOURCES'
     }.get(extension, None)
 
+
 class CommandParser:
     def __init__(self):
         self.breakpoints = []
@@ -34,7 +34,7 @@ class CommandParser:
         new_breakpoint = True
 
         if len(parts) == 2:
-            command = parts[1].strip() # take off whitespace
+            command = parts[1].strip()  # take off whitespace
             new_breakpoint = parts[0].strip() != ""
 
         return (command, new_breakpoint)
@@ -85,7 +85,7 @@ class InlineTest(TestBase):
             return "-N dwarf %s" % (self.mydir)
         else:
             return "-N dsym %s" % (self.mydir)
-        
+
     def BuildMakefile(self):
         if os.path.exists("Makefile"):
             return
@@ -116,11 +116,11 @@ class InlineTest(TestBase):
         if ('CXX_SOURCES' in list(categories.keys())):
             makefile.write("CXXFLAGS += -std=c++11\n")
 
+        makefile.write("\ncleanup:\n\trm -f Makefile *.d\n\n")
         makefile.write("include $(LEVEL)/Makefile.rules\n")
         makefile.flush()
         makefile.close()
 
-
     @skipUnlessDarwin
     def __test_with_dsym(self):
         self.using_dsym = True