Bug 28263

Summary: Add support for Fedora distros in the http tests
Product: WebKit Reporter: Xan Lopez <xan.lopez>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
httpdconf.patch
none
fedorahttpd.patch gustavo: review+, gustavo: commit-queue-

Xan Lopez
Reported 2009-08-13 09:43:53 PDT
Subject says it all, add the needed detection code and a working conf file.
Attachments
httpdconf.patch (38.73 KB, patch)
2009-08-13 09:48 PDT, Xan Lopez 		no flags
DetailsFormatted DiffDiff
fedorahttpd.patch (34.78 KB, patch)
2009-08-26 05:13 PDT, Xan Lopez 		gustavo: review+
gustavo: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Xan Lopez
Comment 1 2009-08-13 09:48:15 PDT
Created attachment 34751 [details] httpdconf.patch Add support for Fedora in http tests that use Apache.
Eric Seidel (no email)
Comment 2 2009-08-13 10:07:25 PDT
Comment on attachment 34751 [details] httpdconf.patch It's very sad that our apache conf files are just copy/paste of one another. I would think we could share a lot more code between them. :(
Xan Lopez
Comment 3 2009-08-26 05:12:35 PDT
(In reply to comment #2) > (From update of attachment 34751 [details]) > It's very sad that our apache conf files are just copy/paste of one another. I > would think we could share a lot more code between them. :( So... I can agree with the sentiment here, but it kind of sucks to block this because of that now that there's already a lot of httpd.conf files in the tree :/ Adding the latest revision of the patch.
Xan Lopez
Comment 4 2009-08-26 05:13:21 PDT
Created attachment 38605 [details] fedorahttpd.patch httpd support for Fedora.
Gustavo Noronha (kov)
Comment 5 2009-09-01 05:43:51 PDT
Comment on attachment 38605 [details] fedorahttpd.patch Yep!
Xan Lopez
Comment 6 2009-09-01 05:51:10 PDT
Landed in r47930, closing.
Note You need to log in before you can comment on or make changes to this bug.