Bug 5854 - Please update build-webkit script to accept additional build arguments from command line
Summary: Please update build-webkit script to accept additional build arguments from c...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Enhancement
Assignee: Eric Seidel (no email)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-11-28 13:50 PST by Dan Wood
Modified: 2006-01-07 23:39 PST (History)
0 users

See Also:


Attachments
fix. (1.79 KB, patch)
2005-12-30 22:25 PST, Eric Seidel (no email)
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dan Wood 2005-11-28 13:50:09 PST
Often, I want to build webkit with some additional parameters passed into xcodebuild, for instance, 
DEPLOYMENT_POSTPROCESSING=YES.

Currently, this does not seem reasonably easy to do; you have to modify the project files or the build-
webkit script.

If we could have additional parameters from the command line passed to xcodebuild, it would be possible 
without much hassle to vary the building process, to do useful things like strip symbols, build universal 
binary, change the install path, and so forth.
Comment 1 Dan Wood 2005-11-29 11:40:44 PST
MacDome says it's just a matter of adding @ARGV to the lines with xcodebuild ... can somebody confirm?
Comment 2 Eric Seidel (no email) 2005-12-30 22:25:46 PST
Created attachment 5393 [details]
fix.
Comment 3 Dan Wood 2005-12-30 22:50:30 PST
The patch seems to do the trick for me.  I built webkit as a fat, stripped binary using the following 
invocation:

build-webkit GCC_OPTIMIZATION_LEVEL=s ARCHS='ppc i386' SDKROOT=/Developer/SDKs/
MacOSX10.4u.sdk DEPLOYMENT_POSTPROCESSING=YES GCC_TREAT_WARNINGS_AS_ERRORS=NO

Being the probable only client for this fix (except for perhaps bdash), I'm satisfied with the patch.
Comment 4 Darin Adler 2005-12-30 23:46:40 PST
Comment on attachment 5393 [details]
fix.

What removes the "-svg" (or is it "--svg"?) if that is passed as an argument?
Comment 5 Maciej Stachowiak 2006-01-02 20:34:59 PST
Moving to r- until Darin's comment is addressed.
Comment 6 Eric Seidel (no email) 2006-01-02 20:40:16 PST
(In reply to comment #4)
> (From update of attachment 5393 [details] [edit])
> What removes the "-svg" (or is it "--svg"?) if that is passed as an argument?
> 

Oh, that's automatically done by Getopt::Long

At least when it's loaded wiht the "pass through" option, like it is in this script:

use Getopt::Long qw(:config pass_through);
Comment 7 Eric Seidel (no email) 2006-01-02 20:40:37 PST
Comment on attachment 5393 [details]
fix.

I've addressed darin's question in the bug.
Comment 8 Darin Adler 2006-01-03 13:51:35 PST
Comment on attachment 5393 [details]
fix.

OK, r=me.