Title: [210358] trunk/Tools
Revision
210358
Author
[email protected]
Date
2017-01-05 09:34:47 -0800 (Thu, 05 Jan 2017)

Log Message

[Win] Archive test result is failing.
https://bugs.webkit.org/show_bug.cgi?id=166720

Reviewed by Alex Christensen.

Deleting the layout test result folder is sometimes failing
on Windows. This is not a critical error.

* BuildSlaveSupport/test-result-archive:
(archiveTestResults):

Modified Paths

Diff

Modified: trunk/Tools/BuildSlaveSupport/test-result-archive (210357 => 210358)


--- trunk/Tools/BuildSlaveSupport/test-result-archive	2017-01-05 17:30:57 UTC (rev 210357)
+++ trunk/Tools/BuildSlaveSupport/test-result-archive	2017-01-05 17:34:47 UTC (rev 210358)
@@ -79,11 +79,11 @@
         # deleting its contents, claiming "Directory not empty".
         # We can safely ignore this since it was the directory contents that
         # we are most interested in deleting.
-        # Python in Cygwin will also sometimes throw errno 2 if a process is
-        # holding a file open. There's no point in failing to create the
+        # Python in Cygwin will also sometimes throw errno 2 or 16 if a process
+        # is holding a file open. There's no point in failing to create the
         # archive just because some other process is behaving badly. See
-        # <http://webkit.org/b/55581>.
-        if e.errno != 90 and e.errno != 2:
+        # <http://webkit.org/b/55581> and <http://webkit.org/b/166720>.
+        if e.errno != 90 and e.errno != 2 and e.errno != 16:
             raise
 
 if __name__ == '__main__':

Modified: trunk/Tools/ChangeLog (210357 => 210358)


--- trunk/Tools/ChangeLog	2017-01-05 17:30:57 UTC (rev 210357)
+++ trunk/Tools/ChangeLog	2017-01-05 17:34:47 UTC (rev 210358)
@@ -1,3 +1,16 @@
+2017-01-05  Per Arne Vollan  <[email protected]>
+
+        [Win] Archive test result is failing.
+        https://bugs.webkit.org/show_bug.cgi?id=166720
+
+        Reviewed by Alex Christensen.
+
+        Deleting the layout test result folder is sometimes failing
+        on Windows. This is not a critical error.
+
+        * BuildSlaveSupport/test-result-archive:
+        (archiveTestResults):
+
 2017-01-04  Tomas Popela  <[email protected]>
 
         [GTK] Minibrowser: Add printing support
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to