2010-03-26 Fumitoshi Ukai <ukai@chromium.org>
authorukai@chromium.org <ukai@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Mar 2010 08:45:28 +0000 (08:45 +0000)
committerukai@chromium.org <ukai@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Mar 2010 08:45:28 +0000 (08:45 +0000)
        Reviewed by Shinichiro Hamaji.

        [Chromium-Win] websocket_server.py failed to start
        https://bugs.webkit.org/show_bug.cgi?id=36586

        subproess.Popen() on Python 2.4/Windows with stdout,stdout, but no stdin will fail, because it uses return value of GetStdHandle(STD_INPUT_HANDLE), but DuplicateHandle requires integer, not the handle.
        We don't need stdin, so use devnull as stdin.

        * Scripts/webkitpy/layout_tests/port/websocket_server.py: open os.devnull for stdin

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

WebKitTools/ChangeLog
WebKitTools/Scripts/webkitpy/layout_tests/port/websocket_server.py

index e62ab7b..3346410 100644 (file)
@@ -1,3 +1,15 @@
+2010-03-26  Fumitoshi Ukai  <ukai@chromium.org>
+
+        Reviewed by Shinichiro Hamaji.
+
+        [Chromium-Win] websocket_server.py failed to start
+        https://bugs.webkit.org/show_bug.cgi?id=36586
+        
+        subproess.Popen() on Python 2.4/Windows with stdout,stdout, but no stdin will fail, because it uses return value of GetStdHandle(STD_INPUT_HANDLE), but DuplicateHandle requires integer, not the handle.
+        We don't need stdin, so use devnull as stdin.
+
+        * Scripts/webkitpy/layout_tests/port/websocket_server.py: open os.devnull for stdin
+
 2010-03-26  Kent Tamura  <tkent@chromium.org>
 
         Reviewed by Dimitri Glazkov.
 2010-03-26  Kent Tamura  <tkent@chromium.org>
 
         Reviewed by Dimitri Glazkov.
index 1b1af71..b2835f7 100644 (file)
@@ -200,7 +200,9 @@ class PyWebSocket(http_server.Lighttpd):
         _log.debug('Starting %s server on %d.' % (
                    self._server_name, self._port))
         _log.debug('cmdline: %s' % ' '.join(start_cmd))
         _log.debug('Starting %s server on %d.' % (
                    self._server_name, self._port))
         _log.debug('cmdline: %s' % ' '.join(start_cmd))
-        self._process = subprocess.Popen(start_cmd, stdout=self._wsout,
+        self._process = subprocess.Popen(start_cmd,
+                                         stdin=open(os.devnull, 'r'),
+                                         stdout=self._wsout,
                                          stderr=subprocess.STDOUT,
                                          env=env)
 
                                          stderr=subprocess.STDOUT,
                                          env=env)