Bug 30302 - [Gtk] Update Fedora httpd.conf settings to closely match the generic httpd.conf
Summary: [Gtk] Update Fedora httpd.conf settings to closely match the generic httpd.conf
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: Gtk
Depends on:
Blocks:
 
Reported: 2009-10-12 03:20 PDT by Jan Alonzo
Modified: 2009-10-12 12:38 PDT (History)
1 user (show)

See Also:


Attachments
Patch v1 (8.19 KB, patch)
2009-10-12 03:22 PDT, Jan Alonzo
xan.lopez: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Alonzo 2009-10-12 03:20:18 PDT
The fedora httpd.conf config currently has a lot unnecessary modules and options enabled. We should make it match the default LayoutTest httpd.conf as closely as possible.
Comment 1 Jan Alonzo 2009-10-12 03:22:33 PDT
Created attachment 41030 [details]
Patch v1
Comment 2 Xan Lopez 2009-10-12 07:13:54 PDT
(In reply to comment #1)
> Created an attachment (id=41030) [details]
> Patch v1

Shouldn't we just remove those lines instead of commenting them? In any case assuming things still work it makes sense to me.
Comment 3 Jan Alonzo 2009-10-12 07:26:20 PDT
(In reply to comment #2)
> (In reply to comment #1)
> > Created an attachment (id=41030) [details] [details]
> > Patch v1
> 
> Shouldn't we just remove those lines instead of commenting them? In any case
> assuming things still work it makes sense to me.

I think it's OK to comment them for now. Those commented options would make a good reference in case we need to enable some of them in the future.

Also, the config still work. :)
Comment 4 Xan Lopez 2009-10-12 07:29:14 PDT
Comment on attachment 41030 [details]
Patch v1

ok, r=me
Comment 5 Jan Alonzo 2009-10-12 12:38:35 PDT
(In reply to comment #4)
> (From update of attachment 41030 [details])
> ok, r=me

Thanks. Landed as http://trac.webkit.org/changeset/49446