Summary: | REGRESSION(52325) Chromium build broken due to link failure | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | mjs, pkasting, sam | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | OS X 10.5 | ||||||||||
Attachments: |
|
Description
Eric Seidel (no email)
2009-12-18 16:08:22 PST
Edited IRC log for posterity: <pkasting> othermaciej mentioned the idea of adding a config.h for this section of the project and just having it #include Platform.h <othermaciej> WebCore and JavaScriptCore have a rule that config.h must be the first include in every file <othermaciej> it seems reasonable to me to do that for WebKit too <eseidel> sounds like othermaciej's idea is a good one, as usual :) <eseidel> I think the utility of a non-PCH build has not at least historically been well understood <eseidel> with XCode eventually moving to distcc 3, everyone will want a non-PCH build eventually <pkasting> So is the resolution here that we should, in fact, add a config.h to WebKit, have it #include Platform.h, and add #include config.h to all the source files? <eseidel> I would just add it to the files you need to un-break the build <eseidel> and we can let webkit.org bake on that policy for a while <eseidel> I think it's important to un-break the build, but we can figure out what the final full-scale policy change looks like later Created attachment 45202 [details]
Patch
Comment on attachment 45202 [details]
Patch
ok
Comment on attachment 45202 [details] Patch > +// Neded builds not using PCH to expose BUILDING_ macros, see bug 32753. "Neded" -> "Needed for" Created attachment 45204 [details]
Now with moar betr smelling
Created attachment 45205 [details]
I should re-enroll in 3rd grade, clearly.
Comment on attachment 45205 [details]
I should re-enroll in 3rd grade, clearly.
ok. I spelt goood to
Committed r52356: <http://trac.webkit.org/changeset/52356> |