delete-stale-build-files is too aggressive
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Mar 2013 18:34:14 +0000 (18:34 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Mar 2013 18:34:14 +0000 (18:34 +0000)
https://bugs.webkit.org/show_bug.cgi?id=112595

Reviewed by Tim Horton.

It appears that whitelisting file extensions to keep is not a good idea.
Blacklist files to delete instead, and only delete .o files for now.
We can add more file extensions as needed.

* BuildSlaveSupport/delete-stale-build-files:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@146093 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/BuildSlaveSupport/delete-stale-build-files
Tools/ChangeLog

index 7467bd0..8933d3b 100755 (executable)
@@ -62,7 +62,7 @@ def main():
             ext = os.path.splitext(full_path)[1]
 
             try:
-                if ext in ('.dep', '.timestamp', '.txt', '.html', '.js', '.generated') or os.path.getsize(full_path):
+                if ext not in ('.o',) or os.path.getsize(full_path):
                     continue
             except OSError as exception:
                 print exception
index 9aa99e0..b302b84 100644 (file)
@@ -1,3 +1,16 @@
+2013-03-18  Ryosuke Niwa  <rniwa@webkit.org>
+
+        delete-stale-build-files is too aggressive
+        https://bugs.webkit.org/show_bug.cgi?id=112595
+
+        Reviewed by Tim Horton.
+
+        It appears that whitelisting file extensions to keep is not a good idea.
+        Blacklist files to delete instead, and only delete .o files for now.
+        We can add more file extensions as needed.
+
+        * BuildSlaveSupport/delete-stale-build-files:
+
 2013-03-18  Simon Fraser  <simon.fraser@apple.com>
 
         Disable accessibility notifications after each test