Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(375)

Issue 2868045: gyp: update to r835 (Closed)

Created:
10 years, 5 months ago by Evan Martin
Modified:
9 years, 7 months ago
Reviewers:
CC:
chromium-reviews
Visibility:
Public.

Description

gyp: update to r835 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 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=51550

Patch Set 1 #

Patch Set 2 : 836 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 1 chunk +1 line, -1 line 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698