Summary: | [chromium] Move more common settings to webcore_prerequisites | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Marc-Antoine Ruel <maruel> | ||||||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | commit-queue, dglazkov, mal, tkent | ||||||||||
Priority: | P2 | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
Bug Depends on: | 44796 | ||||||||||||
Bug Blocks: | 44814 | ||||||||||||
Attachments: |
|
Description
Marc-Antoine Ruel
2010-08-27 19:42:34 PDT
Created attachment 65805 [details]
webcore.gyp patch
Created attachment 65808 [details]
webcore.gyp patch
Fixes ordering on mac
Created attachment 65810 [details]
webcore.gyp patch
rebased on HEAD
Comment on attachment 65810 [details]
webcore.gyp patch
rs=me.
Comment on attachment 65810 [details] webcore.gyp patch Rejecting patch 65810 from commit-queue. Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Dimitri Glazkov', u'--force']" exit_code: 1 Last 500 characters of output: d at 1 with fuzz 3. patching file WebCore/WebCore.gyp/WebCore.gyp Hunk #2 succeeded at 846 (offset 2 lines). Hunk #3 succeeded at 859 with fuzz 1 (offset 2 lines). Hunk #4 succeeded at 954 with fuzz 2 (offset 6 lines). Hunk #5 FAILED at 1082. Hunk #6 succeeded at 1143 (offset 24 lines). Hunk #7 succeeded at 1162 (offset 24 lines). Hunk #8 FAILED at 1277. Hunk #9 succeeded at 1299 with fuzz 1 (offset 33 lines). 2 out of 9 hunks FAILED -- saving rejects to file WebCore/WebCore.gyp/WebCore.gyp.rej Full output: http://queues.webkit.org/results/3874049 I'm pretty sure this failed becuase it depends on bug 44796, which hasn't even been reviewed yet. Can someone r+ that and get it into the queue? Sorry, this patch still has issues on Chromium on Mac, trying to fix it. I don't know how to change the page to remove its review flag, will post another patch once I've figured out how to fix includes paths for .mm files. Created attachment 65852 [details] webcore.gyp patch This is patchset 7 of http://codereview.chromium.org/3223009. Tested to work on Mac. Comment on attachment 65852 [details]
webcore.gyp patch
Please make the patch against the latest WebKit source.
Comment on attachment 65852 [details]
webcore.gyp patch
Oh, I'm sorry. This depends on another patch.
Comment on attachment 65852 [details] webcore.gyp patch > diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog > index bfcce4f..2da30fe 100644 > --- a/WebCore/ChangeLog > +++ b/WebCore/ChangeLog > @@ -2,6 +2,14 @@ > > Reviewed by NOBODY (OOPS!). > > + Move more common settings to webcore_prerequisites > + > + * WebCore.gyp/WebCore.gyp: > + > +2010-08-27 Marc-Antoine Ruel <maruel@chromium.org> > + > + Reviewed by NOBODY (OOPS!). > + > Rename webcore to 'webcore_remaining'. > Create a meta-target webcore that inherits from every webcore_XXX targets. (Currently 1) > Move direct_dependent_settings and export_dependent_settings from webcore_remaining to webcore Add the bug URL to the ChangeLog entry please. |