RESOLVED FIXED 59748
[chromium] support components build on Linux 
https://bugs.webkit.org/show_bug.cgi?id=59748
Summary [chromium] support components build on Linux
Evan Martin
Reported 2011-04-28 15:25:23 PDT
[chromium] support components build on Linux
Attachments
Patch (5.96 KB, patch)
2011-04-28 15:26 PDT, Evan Martin 		no flags
DetailsFormatted DiffDiff
Patch (5.96 KB, patch)
2011-04-29 14:40 PDT, Evan Martin 		no flags
DetailsFormatted DiffDiff
Patch (6.02 KB, patch)
2011-04-29 14:43 PDT, Evan Martin 		tony: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Evan Martin
Comment 1 2011-04-28 15:26:44 PDT
Created attachment 91567 [details] Patch
Tony Chang
Comment 2 2011-04-28 15:30:13 PDT
Comment on attachment 91567 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=91567&action=review > Source/WebKit/chromium/WebKit.gyp:641 > + ['OS=="win" or OS=="mac"', { Why mac for this one? Wasn't this in a win only block above?
Evan Martin
Comment 3 2011-04-29 14:40:55 PDT
Created attachment 91741 [details] Patch
Evan Martin
Comment 4 2011-04-29 14:41:31 PDT
(In reply to comment #2) > (From update of attachment 91567 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=91567&action=review > > > Source/WebKit/chromium/WebKit.gyp:641 > > + ['OS=="win" or OS=="mac"', { > > Why mac for this one? Wasn't this in a win only block above? Components was win-only overall. I believe this is correct for Mac, though I'm not sure.
Evan Martin
Comment 5 2011-04-29 14:43:19 PDT
Created attachment 91743 [details] Patch
Tony Chang
Comment 6 2011-04-29 15:07:03 PDT
Comment on attachment 91743 [details] Patch OK!
Evan Martin
Comment 7 2011-04-29 15:20:15 PDT
Committed r85362: <http://trac.webkit.org/changeset/85362>
Note You need to log in before you can comment on or make changes to this bug.