Skip to content
Snippets Groups Projects
  • evan@chromium.org's avatar
    fd2a308a
    gyp: update to r835 · fd2a308a
    evan@chromium.org authored
    Includes following changes.
    
    ------------------------------------------------------------------------
    r834 | evan@chromium.org | 2010-06-29 13:58:08 -0700 (Tue, 29 Jun 2010) | 6 lines
    
    Don't use ranlib, use ar's "s" flag instead.
    
    Patch by Shinichiro Hamaji <hamaji@chromium.org>.
    Review URL: http://codereview.chromium.org/2800033
    
    ------------------------------------------------------------------------
    r833 | mark@chromium.org | 2010-06-28 13:15:06 -0700 (Mon, 28 Jun 2010) | 5 lines
    
    Xcode support for already-assembled .o (object) files as sources.
    
    BUG=gyp:157
    TEST=none
    Review URL: http://codereview.chromium.org/2878002
    
    ------------------------------------------------------------------------
    r832 | evan@chromium.org | 2010-06-25 10:46:11 -0700 (Fri, 25 Jun 2010) | 30 lines
    
    Use GNU ar's thin archive if this feature is available.
    
    The thin archives don't have actual copy of object files. Instead, they have
    the paths to object files like symbolic links. With this option, ar and ranlib
    in chromium build become much faster on linux.
    
    Before this change:
    
    Total size of .a files: 1726537946 (1726M)
    ./ar.sh 8.73s user 12.28s system 26% cpu 1:19.24 total
    ./link.sh 9.14s user 4.54s system 14% cpu 1:36.01 total
    
    After this change:
    
    Total size of .a files: 43157870 (43M)
    ./ar.sh 7.77s user 1.75s system 97% cpu 9.797 total
    ./link.sh 9.10s user 5.26s system 21% cpu 1:06.18 total
    
    I'm not sure why link also becomes faster. Maybe other processes were running
    when I ran link.sh. The link.sh just links chrome and the ar.sh is generated by
    grepping our build process. The actual file is here:
    http://shinh.skr.jp/t/ar.sh.txt
    
    Unfortunately, this feature is relatively new (2008-03-27,
    http://sourceware.org/ml/binutils/2008-03/msg00150.html), so I think we cannot
    expect everyone have this feature.  So we autodetect it at build time.
    
    Code review: http://codereview.chromium.org/2841004
    
    ------------------------------------------------------------------------
    r831 | mark@chromium.org | 2010-06-24 09:44:51 -0700 (Thu, 24 Jun 2010) | 5 lines
    
    Make loadable_modules without mac_bundle buildable.
    
    BUG=none
    TEST=none
    Review URL: http://codereview.chromium.org/2859021
    ------------------------------------------------------------------------
    r830 | stuartmorgan@chromium.org | 2010-06-23 08:01:31 -0700 (Wed, 23 Jun 2010) | 1 line
    
    Add support for target-level Xcode config files. Review URL: http://codereview.appspot.com/1687044/show
    ------------------------------------------------------------------------
    r827 | thestig@chromium.org | 2010-05-21 18:44:01 -0700 (Fri, 21 May 2010) | 5 lines
    
    Raise an error when a configuration dictionary contains invalid keys.
    
    BUG=none
    TEST=included
    Review URL: http://codereview.chromium.org/2094021
    
    Review URL: http://codereview.chromium.org/2806038
    
    Review URL: http://codereview.chromium.org/2868045
    
    git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51550 0039d316-1c4b-4281-b951-d872f2087c98
    fd2a308a
    History
    gyp: update to r835
    evan@chromium.org authored
    Includes following changes.
    
    ------------------------------------------------------------------------
    r834 | evan@chromium.org | 2010-06-29 13:58:08 -0700 (Tue, 29 Jun 2010) | 6 lines
    
    Don't use ranlib, use ar's "s" flag instead.
    
    Patch by Shinichiro Hamaji <hamaji@chromium.org>.
    Review URL: http://codereview.chromium.org/2800033
    
    ------------------------------------------------------------------------
    r833 | mark@chromium.org | 2010-06-28 13:15:06 -0700 (Mon, 28 Jun 2010) | 5 lines
    
    Xcode support for already-assembled .o (object) files as sources.
    
    BUG=gyp:157
    TEST=none
    Review URL: http://codereview.chromium.org/2878002
    
    ------------------------------------------------------------------------
    r832 | evan@chromium.org | 2010-06-25 10:46:11 -0700 (Fri, 25 Jun 2010) | 30 lines
    
    Use GNU ar's thin archive if this feature is available.
    
    The thin archives don't have actual copy of object files. Instead, they have
    the paths to object files like symbolic links. With this option, ar and ranlib
    in chromium build become much faster on linux.
    
    Before this change:
    
    Total size of .a files: 1726537946 (1726M)
    ./ar.sh 8.73s user 12.28s system 26% cpu 1:19.24 total
    ./link.sh 9.14s user 4.54s system 14% cpu 1:36.01 total
    
    After this change:
    
    Total size of .a files: 43157870 (43M)
    ./ar.sh 7.77s user 1.75s system 97% cpu 9.797 total
    ./link.sh 9.10s user 5.26s system 21% cpu 1:06.18 total
    
    I'm not sure why link also becomes faster. Maybe other processes were running
    when I ran link.sh. The link.sh just links chrome and the ar.sh is generated by
    grepping our build process. The actual file is here:
    http://shinh.skr.jp/t/ar.sh.txt
    
    Unfortunately, this feature is relatively new (2008-03-27,
    http://sourceware.org/ml/binutils/2008-03/msg00150.html), so I think we cannot
    expect everyone have this feature.  So we autodetect it at build time.
    
    Code review: http://codereview.chromium.org/2841004
    
    ------------------------------------------------------------------------
    r831 | mark@chromium.org | 2010-06-24 09:44:51 -0700 (Thu, 24 Jun 2010) | 5 lines
    
    Make loadable_modules without mac_bundle buildable.
    
    BUG=none
    TEST=none
    Review URL: http://codereview.chromium.org/2859021
    ------------------------------------------------------------------------
    r830 | stuartmorgan@chromium.org | 2010-06-23 08:01:31 -0700 (Wed, 23 Jun 2010) | 1 line
    
    Add support for target-level Xcode config files. Review URL: http://codereview.appspot.com/1687044/show
    ------------------------------------------------------------------------
    r827 | thestig@chromium.org | 2010-05-21 18:44:01 -0700 (Fri, 21 May 2010) | 5 lines
    
    Raise an error when a configuration dictionary contains invalid keys.
    
    BUG=none
    TEST=included
    Review URL: http://codereview.chromium.org/2094021
    
    Review URL: http://codereview.chromium.org/2806038
    
    Review URL: http://codereview.chromium.org/2868045
    
    git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51550 0039d316-1c4b-4281-b951-d872f2087c98