[webkit-reviews] review denied: [Bug 17952] Add "--cairo" flag to
'build-webkit' command : [Attachment 19887] Extend
build-webkit with new "--cairo" build option
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Apr 2 02:43:54 PDT 2008
Darin Adler <darin at apple.com> has denied Brent Fulgham <bfulgham at gmail.com>'s
request for review:
Bug 17952: Add "--cairo" flag to 'build-webkit' command
http://bugs.webkit.org/show_bug.cgi?id=17952
Attachment 19887: Extend build-webkit with new "--cairo" build option
http://bugs.webkit.org/attachment.cgi?id=19887&action=edit
------- Additional Comments from Darin Adler <darin at apple.com>
+ if (checkArgv("--cairo")) {
+ $isCairo = 1;
+ } else {
+ $isWx = 0;
+ }
Seems wrong to set $isWx here.
But also, I think you'd want this setting to be a persistent one. It seems
really inconvenient to have to pass --cairo every single time. It should work
more like configuration where you can set it once and then it's persistent
until you reset it.
review- because of the $isWx mistake, but please consider my suggestion.
More information about the webkit-reviews
mailing list