Title: [130202] trunk
Revision
130202
Author
commit-qu...@webkit.org
Date
2012-10-02 13:28:05 -0700 (Tue, 02 Oct 2012)

Log Message

Remove MSYS-related changes to the http testing infrastructure.
https://bugs.webkit.org/show_bug.cgi?id=98183

Patch by Raphael Kubo da Costa <raphael.kubo.da.co...@intel.com> on 2012-10-02
Reviewed by Eric Seidel.

In the process of removing the cruft that has accumulated in our
Apache configuration code, we can basically revert r77161 which
added some MSYS (mingw)-specific code to old-run-webkit-tests,
since it is not being used by any port anymore.

Tools:

* Scripts/run-webkit-httpd: Remove isMsys()-dependent hacks.
* Scripts/webkitperl/httpd.pm:
(getHTTPDPath): Remove isMsys() code path.
(getDefaultConfigForTestDirectory): Revert isMsys() hacks.
(getHTTPDConfigPathForTestDirectory): Do not point to the noew
removed apache2-msys-httpd.conf.
(convertMsysPath): Removed.

LayoutTests:

* http/conf/apache2-msys-httpd.conf: Removed.

Modified Paths

Removed Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (130201 => 130202)


--- trunk/LayoutTests/ChangeLog	2012-10-02 20:19:48 UTC (rev 130201)
+++ trunk/LayoutTests/ChangeLog	2012-10-02 20:28:05 UTC (rev 130202)
@@ -1,5 +1,19 @@
 2012-10-02  Raphael Kubo da Costa  <raphael.kubo.da.co...@intel.com>
 
+        Remove MSYS-related changes to the http testing infrastructure.
+        https://bugs.webkit.org/show_bug.cgi?id=98183
+
+        Reviewed by Eric Seidel.
+
+        In the process of removing the cruft that has accumulated in our
+        Apache configuration code, we can basically revert r77161 which
+        added some MSYS (mingw)-specific code to old-run-webkit-tests,
+        since it is not being used by any port anymore.
+
+        * http/conf/apache2-msys-httpd.conf: Removed.
+
+2012-10-02  Raphael Kubo da Costa  <raphael.kubo.da.co...@intel.com>
+
         Remove cases of testRunner.disableImageLoading() from the layout tests.
         https://bugs.webkit.org/show_bug.cgi?id=98187
 

Deleted: trunk/LayoutTests/http/conf/apache2-msys-httpd.conf (130201 => 130202)


--- trunk/LayoutTests/http/conf/apache2-msys-httpd.conf	2012-10-02 20:19:48 UTC (rev 130201)
+++ trunk/LayoutTests/http/conf/apache2-msys-httpd.conf	2012-10-02 20:28:05 UTC (rev 130202)
@@ -1,162 +0,0 @@
-ServerRoot "C:/Program Files/Apache Software Foundation/Apache2.2"
-
-PidFile "/tmp/WebKit/httpd.pid"
-ScoreBoardFile "/tmp/WebKit/httpd.scoreboard"
-
-Timeout 300
-KeepAlive On
-MaxKeepAliveRequests 100
-KeepAliveTimeout 15
-
-# Configured from the httpd command line for WebKit layout tests.
-# Port 8443 is hard-coded as it is needed for the HTTPS configuration later on.
-#
-#Listen 3000
-Listen 127.0.0.1:8080
-Listen 127.0.0.1:8443
-# We listen to both IPv4 and IPv6 loop-back addresses, but ignore
-# requests to 8000 from random users on network.
-# See https://bugs.webkit.org/show_bug.cgi?id=37104
-#Listen [::1]:8000
-#Listen [::1]:8080
-#Listen [::1]:8443
-
-LoadModule authz_host_module modules/mod_authz_host.so
-LoadModule include_module modules/mod_include.so
-LoadModule log_config_module modules/mod_log_config.so
-LoadModule headers_module modules/mod_headers.so
-LoadModule ssl_module modules/mod_ssl.so
-LoadModule mime_module modules/mod_mime.so
-LoadModule autoindex_module modules/mod_autoindex.so
-LoadModule asis_module modules/mod_asis.so
-LoadModule cgi_module modules/mod_cgi.so
-LoadModule negotiation_module modules/mod_negotiation.so
-LoadModule imagemap_module modules/mod_imagemap.so
-LoadModule actions_module modules/mod_actions.so
-LoadModule alias_module modules/mod_alias.so
-LoadModule rewrite_module modules/mod_rewrite.so
-
-ServerName 127.0.0.1
-
-<Directory />
-    Options Indexes FollowSymLinks MultiViews ExecCGI Includes
-    AllowOverride All
-    Order allow,deny
-    Allow from all
-</Directory>
-
-AccessFileName .htaccess
-
-<Files ~ "^\.([Hh][Tt]|[Dd][Ss]_[Ss])">
-    Order allow,deny
-    Deny from all
-    Satisfy All
-</Files>
-
-UseCanonicalName On
-DefaultType text/plain
-HostnameLookups Off
-
-LogLevel warn
-LogFormat "%h %l %u %t \"%r\" %>s %b \"%{Referer}i\" \"%{User-Agent}i\"" combined
-LogFormat "%h %l %u %t \"%r\" %>s %b" common
-LogFormat "%{Referer}i -> %U" referer
-LogFormat "%{User-agent}i" agent
-
-ServerSignature On
-
-<IfModule mod_alias.c>
-</IfModule>
-
-<IfModule mod_mime.c>
-    AddLanguage da .dk
-    AddLanguage nl .nl
-    AddLanguage en .en
-    AddLanguage et .ee
-    AddLanguage fr .fr
-    AddLanguage de .de
-    AddLanguage el .el
-    AddLanguage he .he
-    AddCharset ISO-8859-8 .iso8859-8
-    AddLanguage it .it
-    AddLanguage ja .ja
-    AddCharset ISO-2022-JP .jis
-    AddLanguage kr .kr
-    AddCharset ISO-2022-KR .iso-kr
-    AddLanguage nn .nn
-    AddLanguage no .no
-    AddLanguage pl .po
-    AddCharset ISO-8859-2 .iso-pl
-    AddLanguage pt .pt
-    AddLanguage pt-br .pt-br
-    AddLanguage ltz .lu
-    AddLanguage ca .ca
-    AddLanguage es .es
-    AddLanguage sv .sv
-    AddLanguage cs .cz .cs
-    AddLanguage ru .ru
-    AddLanguage zh-TW .zh-tw
-    AddCharset Big5         .Big5    .big5
-    AddCharset WINDOWS-1251 .cp-1251
-    AddCharset CP866        .cp866
-    AddCharset ISO-8859-5   .iso-ru
-    AddCharset KOI8-R       .koi8-r
-    AddCharset UCS-2        .ucs2
-    AddCharset UCS-4        .ucs4
-    AddCharset UTF-8        .utf8
-
-    <IfModule mod_negotiation.c>
-        LanguagePriority en da nl et fr de el it ja kr no pl pt pt-br ru ltz ca es sv tw
-    </IfModule>
-
-    AddType application/x-tar .tgz
-
-    AddEncoding x-compress .Z
-    AddEncoding x-gzip .gz .tgz
-
-    AddHandler cgi-script .cgi .pl
-
-    AddType text/html .shtml
-    AddHandler server-parsed .shtml
-
-    AddHandler send-as-is asis
-</IfModule>
-
-<IfModule mod_php5.c>
-    AddType application/x-httpd-php .php
-    AddType application/x-httpd-php .bat
-    AddType application/x-httpd-php-source .phps
-
-    <IfModule mod_dir.c>
-        DirectoryIndex index.html index.php
-    </IfModule>
-
-    php_flag log_errors on
-    php_flag short_open_tag on
-</IfModule>
-
-<IfModule mod_rewrite.c>
-    RewriteEngine On
-    RewriteCond %{REQUEST_METHOD} ^TRACE
-    RewriteRule .* - [F]
-</IfModule>
-
-<VirtualHost *:8443>
-    ServerName 127.0.0.1
-    SSLEngine On
-</VirtualHost>
-
-#
-# Apple-specific filesystem protection.
-#
-<Files "rsrc">
-    Order allow,deny
-    Deny from all
-    Satisfy All
-</Files>
-
-<Directory  ~ ".*\.\.namedfork">
-    Order allow,deny
-    Deny from all
-    Satisfy All
-</Directory>

Modified: trunk/Tools/ChangeLog (130201 => 130202)


--- trunk/Tools/ChangeLog	2012-10-02 20:19:48 UTC (rev 130201)
+++ trunk/Tools/ChangeLog	2012-10-02 20:28:05 UTC (rev 130202)
@@ -1,3 +1,23 @@
+2012-10-02  Raphael Kubo da Costa  <raphael.kubo.da.co...@intel.com>
+
+        Remove MSYS-related changes to the http testing infrastructure.
+        https://bugs.webkit.org/show_bug.cgi?id=98183
+
+        Reviewed by Eric Seidel.
+
+        In the process of removing the cruft that has accumulated in our
+        Apache configuration code, we can basically revert r77161 which
+        added some MSYS (mingw)-specific code to old-run-webkit-tests,
+        since it is not being used by any port anymore.
+
+        * Scripts/run-webkit-httpd: Remove isMsys()-dependent hacks.
+        * Scripts/webkitperl/httpd.pm:
+        (getHTTPDPath): Remove isMsys() code path.
+        (getDefaultConfigForTestDirectory): Revert isMsys() hacks.
+        (getHTTPDConfigPathForTestDirectory): Do not point to the noew
+        removed apache2-msys-httpd.conf.
+        (convertMsysPath): Removed.
+
 2012-10-02  Sadrul Habib Chowdhury  <sad...@chromium.org>
 
         [chromium] Make sure the touch-points in the touch-events have the correct state.

Modified: trunk/Tools/Scripts/run-webkit-httpd (130201 => 130202)


--- trunk/Tools/Scripts/run-webkit-httpd	2012-10-02 20:19:48 UTC (rev 130201)
+++ trunk/Tools/Scripts/run-webkit-httpd	2012-10-02 20:28:05 UTC (rev 130202)
@@ -70,7 +70,6 @@
 my $productDir = productDir();
 chdirWebKit();
 my $testDirectory = File::Spec->catfile(getcwd(), "LayoutTests");
-$testDirectory = convertMsysPath($testDirectory) if isMsys();
 my $listen = "127.0.0.1:$httpdPort";
 $listen = "$httpdPort" if ($allInterfaces);
 
@@ -84,16 +83,14 @@
 
 my @args = (
     "-C", "Listen $listen",
-    # Disable Keep-Alive support. Makes testing in multiple browsers easier (no need to wait
-    # for another browser's connection to expire).
-    "-c", "KeepAlive 0"
-);
-push @args, (
     "-c", "CustomLog |/usr/bin/tee common",
     "-c", "ErrorLog |/usr/bin/tee",
     # Run in single-process mode, do not detach from the controlling terminal.
     "-X",
-) unless isMsys();
+    # Disable Keep-Alive support. Makes testing in multiple browsers easier (no need to wait
+    # for another browser's connection to expire).
+    "-c", "KeepAlive 0"
+);
 
 my @defaultArgs = getDefaultConfigForTestDirectory($testDirectory);
 @args = (@defaultArgs, @args);

Modified: trunk/Tools/Scripts/webkitperl/httpd.pm (130201 => 130202)


--- trunk/Tools/Scripts/webkitperl/httpd.pm	2012-10-02 20:19:48 UTC (rev 130201)
+++ trunk/Tools/Scripts/webkitperl/httpd.pm	2012-10-02 20:28:05 UTC (rev 130202)
@@ -60,7 +60,6 @@
 }
 
 my $tmpDir = "/tmp";
-$tmpDir = convertMsysPath($tmpDir) if isMsys();
 my $httpdLockPrefix = "WebKitHttpd.lock.";
 my $myLockFile;
 my $exclusiveLockFile = File::Spec->catfile($tmpDir, "WebKit.lock");
@@ -79,8 +78,6 @@
     my $httpdPath;
     if (isDebianBased()) {
         $httpdPath = "/usr/sbin/apache2";
-    } elsif (isMsys()) {
-        $httpdPath = 'c:\program files\apache software foundation\apache2.2\bin\httpd.exe';
     } else {
         $httpdPath = "/usr/sbin/httpd";
     }
@@ -113,16 +110,13 @@
         "-c", "Alias /js-test-resources \"$jsTestResourcesDirectory\"",
         "-c", "Alias /media-resources \"$mediaResourcesDirectory\"",
         "-c", "TypesConfig \"$typesConfig\"",
+        # Apache wouldn't run CGIs with permissions==700 otherwise
+        "-c", "User \"#$<\"",
+        "-c", "LockFile \"$httpdLockFile\"",
         "-c", "PidFile \"$httpdPidFile\"",
         "-c", "ScoreBoardFile \"$httpdScoreBoardFile\"",
     );
 
-    push @httpdArgs, (
-        # Apache wouldn't run CGIs with permissions==700 otherwise
-        "-c", "User \"#$<\"",
-        "-c", "LockFile \"$httpdLockFile\""
-    ) unless isMsys();
-
     # FIXME: Enable this on Windows once <rdar://problem/5345985> is fixed
     # The version of Apache we use with Cygwin does not support SSL
     my $sslCertificate = "$testDirectory/http/conf/webkit-httpd.pem";
@@ -149,8 +143,6 @@
             chmod(0755, $libPHP4DllPath);
         }
         $httpdConfig = "cygwin-httpd.conf";  # This is an apache 1.3 config.
-    } elsif (isMsys()) {
-        $httpdConfig = "apache2-msys-httpd.conf";
     } elsif (isDebianBased()) {
         $httpdConfig = "apache2-debian-httpd.conf";
     } elsif (isFedoraBased()) {
@@ -341,13 +333,3 @@
     }
     return $waitTime;
 }
-
-sub convertMsysPath
-{
-    my ($path) = @_;
-    return unless isMsys();
-
-    $path = `cmd.exe //c echo $path`;
-    $path =~ s/\r\n$//;
-    return $path;
-}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to