Bug 50036 – update run-webkit-httpd to be able to launch Apache2.2 in a MSYS environment

• Component: Tools / Tests
• Status: RESOLVED FIXED

I'd like to propose adding functionality of running an HTTPD service via run-webkit-httpd in an MSYS environment.

This would require that the user installs the Win32 Binary including OpenSSL from http://httpd.apache.org/download.cgi#apache22.

Apache2.2 for Win32 does not come with PHP5 modules.  The loading of these modules will therefore be disabled in the Apache2.2 configuration that I propose.  They could however be installed separately from http://windows.php.net/download/.

Created attachment 74785 [details]
Added httpd configuration file and modification to run Apache2.2 in MSYS environment

NOTE: The apache2-msys-httpd.conf has been merged with apache2-httpd.conf.
The diff< ServerRoot "/usr"
---
> ServerRoot "C:/Program Files/Apache Software Foundation/Apache2.2"
68c68
< LockFile "/tmp/WebKit/httpd.lock"
---
> #LockFile "/tmp/WebKit/httpd.lock"
121,122c121,122
< MinSpareServers 1
< MaxSpareServers 5
---
> #MinSpareServers 1
> #MaxSpareServers 5
128c128
< StartServers 1
---
> #StartServers 1
137c137
< MaxClients 150
---
> #MaxClients 150
153c153
< MaxRequestsPerChild 100000
---
> #MaxRequestsPerChild 100000
169,171c169,171
< Listen [::1]:8000
< Listen [::1]:8080
< Listen [::1]:8443
---
> #Listen [::1]:8000
> #Listen [::1]:8080
> #Listen [::1]:8443
189,251c189,252

With the addition of some differences in modules.

[Eric Seidel (no email)]

Comment on attachment 74785 [details]
Added httpd configuration file and modification to run Apache2.2 in MSYS environment

The madness must top!  Yet another copy/paste httpd.conf :(

[Eric Seidel (no email)]

Also, I thougth the world had moved to new-run-webkit-httpd?

Created attachment 77891 [details]
Same patch with the addition of ChangeLogs and moved from WebKitTools/ to Tools/

This is the same patch with the addition of Tools/ChangeLog and LayoutTests/ChangeLog files.
Also, the patch has been moved from WebKitTools/ to Tools/ to match recent changes.

[Daniel Bates]

(In reply to comment #2)
> (From update of attachment 74785 [details])
> The madness must top!  Yet another copy/paste httpd.conf :(

The httpd.conf file included in the patch consists of some Windows-specific changes related to running run-webkit-httpd from within the MinGW command prompt. These changes include changing the location of the ServerRoot, and the location of Apache2 modules (from libexec/apache2/ to modules/). We were unclear how to change such settings using the command line and/or use Windows junctions effectively.

We should look to try to extract the common bits from these Apache configuration file into shared files. I suggest we do such refactoring in a separate bug.

If you have any additional ideas/suggestions on this issue, we would appreciate it.

[Daniel Bates]

Comment on attachment 77891 [details]
Same patch with the addition of ChangeLogs and moved from WebKitTools/ to Tools/

View in context: https://bugs.webkit.org/attachment.cgi?id=77891&action=review

Looks good to me.

> Tools/Scripts/run-webkit-httpd:5
> +# Copyright (C) Research In Motion Limited 2010. All rights reserved.

Nit: 2010 should be 2011.

> Tools/Scripts/webkitperl/httpd.pm:4
> +# Copyright (C) Research In Motion Limited 2010. All rights reserved.

Ditto.

> Tools/Scripts/webkitperl/httpd.pm:82
> +        $httpdPath = "/c/program\ files/apache\ software\ foundation/apache2.2/bin/httpd.exe";

It would be nicer if we could write this in such a way that we didn't need to escape the space characters in the path.

[Daniel Bates]

(In reply to comment #3)
> Also, I thougth the world had moved to new-run-webkit-httpd?

Not yet. It's probably best to address such a transition on webkit-dev.

[Daniel Bates]

Committed r77161: <http://trac.webkit.org/changeset/77161>