Title: [267417] trunk/LayoutTests/imported/w3c
Revision
267417
Author
[email protected]
Date
2020-09-22 09:53:31 -0700 (Tue, 22 Sep 2020)

Log Message

REGRESSION(r267354): Only import resource module on Darwin
https://bugs.webkit.org/show_bug.cgi?id=216823

Patch by Sam Sneddon <[email protected]> on 2020-09-22
Reviewed by Darin Adler.

* web-platform-tests/tools/serve/serve.py:
Move the resource import to the local point behind the Darwin platform
test. This code is only run once per process initiation, hence it makes
no difference to have the import locally.

Modified Paths

Diff

Modified: trunk/LayoutTests/imported/w3c/ChangeLog (267416 => 267417)


--- trunk/LayoutTests/imported/w3c/ChangeLog	2020-09-22 16:45:48 UTC (rev 267416)
+++ trunk/LayoutTests/imported/w3c/ChangeLog	2020-09-22 16:53:31 UTC (rev 267417)
@@ -1,3 +1,15 @@
+2020-09-22  Sam Sneddon  <[email protected]>
+
+        REGRESSION(r267354): Only import resource module on Darwin
+        https://bugs.webkit.org/show_bug.cgi?id=216823
+
+        Reviewed by Darin Adler.
+
+        * web-platform-tests/tools/serve/serve.py:
+        Move the resource import to the local point behind the Darwin platform
+        test. This code is only run once per process initiation, hence it makes
+        no difference to have the import locally.
+
 2020-09-22  Sam Weinig  <[email protected]>
 
         Update User Timing interfaces to User Timing Level 3

Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/tools/serve/serve.py (267416 => 267417)


--- trunk/LayoutTests/imported/w3c/web-platform-tests/tools/serve/serve.py	2020-09-22 16:45:48 UTC (rev 267416)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/tools/serve/serve.py	2020-09-22 16:53:31 UTC (rev 267417)
@@ -8,7 +8,6 @@
 import logging
 import os
 import platform
-import resource
 import signal
 import socket
 import subprocess
@@ -421,6 +420,7 @@
         if sys.platform == "darwin":
             # on Darwin, NOFILE starts with a very low limit (256), so bump it up a little
             # by way of comparison, Debian starts with a limit of 1024, Windows 512
+            import resource  # local, as it only exists on Unix-like systems
             maxfilesperproc = int(subprocess.check_output(
                 ["sysctl", "-n", "kern.maxfilesperproc"]
             ).strip())
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to