Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(558)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: webkit/tools/layout_tests/layout_package/http_server.py

Issue 10653: Add Linux lighttpd for running LayoutTests on Linux. (Closed)
Patch Set: Added friendly error-message Created 12 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/install-build-deps.sh ('k') | webkit/tools/layout_tests/layout_package/platform_utils_linux.py » ('j') | webkit/tools/layout_tests/layout_package/platform_utils_linux.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/tools/layout_tests/layout_package/http_server.py
diff --git a/webkit/tools/layout_tests/layout_package/http_server.py b/webkit/tools/layout_tests/layout_package/http_server.py
index 0369f81758ce00f8663847bd9dfdfb025c04f7a5..c8ca8675fdc7ccd635e672a857523727caa95f25 100644
--- a/webkit/tools/layout_tests/layout_package/http_server.py
+++ b/webkit/tools/layout_tests/layout_package/http_server.py
@@ -127,8 +127,9 @@ class Lighttpd:
# Put the cygwin directory first in the path to find cygwin1.dll
env = os.environ
- env['PATH'] = '%s;%s' % (
- PathFromBase('third_party', 'cygwin', 'bin'), env['PATH'])
+ if sys.platform in ('cygwin', 'win32'):
+ env['PATH'] = '%s;%s' % (
+ PathFromBase('third_party', 'cygwin', 'bin'), env['PATH'])
logging.info('Starting http server')
self._process = subprocess.Popen(start_cmd, env=env)
« no previous file with comments | « build/install-build-deps.sh ('k') | webkit/tools/layout_tests/layout_package/platform_utils_linux.py » ('j') | webkit/tools/layout_tests/layout_package/platform_utils_linux.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698