Changeset 40556 in webkit


Ignore:
Timestamp:
Feb 3, 2009 4:39:11 PM (15 years ago)
Author:
kevino@webkit.org
Message:

Reviewed by Darin Adler.

Change the way wxWidgets build gets arguments.
Change "checkForArgumentAndRemoveFromARGV" in build-webkit to check if the
argument passed matches one in ARGV exactly (allows, ie, --wx-args not to be
removed when --wx is checked for).

https://bugs.webkit.org/show_bug.cgi?id=23701

Location:
trunk/WebKitTools
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/WebKitTools/ChangeLog

    r40430 r40556  
     12009-02-02  miggilin  <mr.diggilin@gmail.com>
     2
     3        Reviewed by Darin Adler.
     4
     5        Change the way wxWidgets build gets arguments.
     6        Change "checkForArgumentAndRemoveFromARGV" in build-webkit to check if the
     7        argument passed matches one in ARGV exactly (allows, ie, --wx-args not to be
     8        removed when --wx is checked for).
     9
     10        https://bugs.webkit.org/show_bug.cgi?id=23701
     11
     12        * Scripts/build-webkit:
     13        * Scripts/webkitdirs.pm:
     14
    1152009-01-30  Chris Fleizach  <cfleizach@apple.com>
    216
  • trunk/WebKitTools/Scripts/build-webkit

    r38965 r40556  
    124124chdirWebKit();
    125125
    126 # FIXME: Migrate build-wxwebkit commands into build-webkit.
    127126if (isWx()) {
    128     my @opts = ();
    129127    $ENV{"WEBKITOUTPUTDIR"} = productDir();
    130     foreach (@ARGV) {
    131         if ($_ eq "wxgc" || $_ eq "wxpython") {
    132             push(@opts, $_);
    133         }
    134     }
     128
     129    my @opts = getWxArgs();
     130
    135131    if ($clean) {
    136132        push(@opts, "clean");
  • trunk/WebKitTools/Scripts/webkitdirs.pm

    r40073 r40556  
    5757my $isGtk;
    5858my $isWx;
     59my @wxArgs;
    5960my $isChromium;
    6061my $forceRun64Bit;
     
    526527    my $argToCheck = shift;
    527528    foreach my $opt (@ARGV) {
    528         if ($opt =~ /^$argToCheck/i ) {
    529             @ARGV = grep(!/^$argToCheck/i, @ARGV);
     529        if ($opt =~ /^$argToCheck$/i ) {
     530            @ARGV = grep(!/^$argToCheck$/i, @ARGV);
    530531            return 1;
    531532        }
     
    575576    return if defined($isWx);
    576577    $isWx = checkForArgumentAndRemoveFromARGV("--wx");
     578}
     579
     580sub getWxArgs()
     581{
     582    if (!@wxArgs) {
     583        @wxArgs = ("");
     584        my $rawWxArgs = "";
     585        foreach my $opt (@ARGV) {
     586            if ($opt =~ /^--wx-args/i ) {
     587                @ARGV = grep(!/^--wx-args/i, @ARGV);
     588                $rawWxArgs = $opt;
     589                $rawWxArgs =~ s/--wx-args=//i;
     590            }
     591        }
     592        @wxArgs = split(/,/, $rawWxArgs);
     593    }
     594    return @wxArgs;
    577595}
    578596
Note: See TracChangeset for help on using the changeset viewer.