- 14 Sep, 2012 2 commits
-
-
binji@chromium.org authored
BUG=none Review URL: https://chromiumcodereview.appspot.com/10907229 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@156796 0039d316-1c4b-4281-b951-d872f2087c98
-
thakis@chromium.org authored
On both mac and linux, I ran: git checkout v1.0.0 ninja ./ninja ninja_test ./ninja_test # all passed strip ninja Changes since the last push: * NINJA_STATUS can now print rates via the o and c format chars * The version switch is now --version, -V no longer works * -t list now works without a build.ninja * Old .ninja_log files now trigger a rebuild instead of an error * Missing .d files make the corresponding .o file dirty * Allow @ and = in paths in depfiles * Ninja's output is now lower-case ("error:" instead of "ERROR:" etc) * Lots of Windows features (which doesn't affect Mac and Linux) On Linux, I switched from gcc4.4.3 to clang r163126. The clang-built binary is 125856 bytes (compared to 129944 bytes previously), and it does an empty 'chrome' build about 70ms faster (0.94s instead of 1.01s). The speed difference is almost completely in faster depfile parsing: clang-built (0.943s): $ time ../../ninja/ninja -C out/Release chrome -d stats ninja: Entering directory `out/Release' ninja: no work to do. metric count avg (us) total (ms) .ninja parse 716 507.8 363.6 canonicalize str 113486 0.2 27.7 canonicalize path 1305641 0.1 184.1 lookup node 1305641 0.2 200.9 .ninja_log load 1 11144.0 11.1 node stat 37828 2.6 98.0 depfile load 9266 80.9 749.3 path->node hash load 0.98 (48241 entries / 49157 buckets) real 0m1.162s user 0m0.980s sys 0m0.170s gcc-built (1.013s): $ time ninja -C out/Release chrome -d stats ninja: Entering directory `out/Release' ninja: no work to do. metric count avg (us) total (ms) .ninja parse 716 516.4 369.7 canonicalize str 113486 0.2 27.6 canonicalize path 1305641 0.1 181.0 lookup node 1305641 0.2 260.0 .ninja_log load 1 11012.0 11.0 node stat 37828 2.6 98.2 depfile load 9266 86.3 799.3 path->node hash load 0.98 (48241 entries / 49157 buckets) real 0m1.220s user 0m0.970s sys 0m0.240s On Mac, I alsow used clang r163126. The binary size stayed at 125104 bytes, performance didn't change. An empty build of chrome takes 1.26s. While node stat is a bit slower, and there are more defiles to parse compared to linux, most of the time difference is in slower .ninja parsing: mac (1.263s): $ time ../../ninja/ninja -C out/Release/ chrome -d stats ninja: Entering directory `out/Release/' ninja: no work to do. metric count avg (us) total (ms) .ninja parse 687 823.4 565.7 canonicalize str 115002 0.3 31.1 canonicalize path 1120853 0.2 175.6 lookup node 1120853 0.2 202.9 .ninja_log load 1 54126.0 54.1 node stat 38860 4.1 160.4 depfile load 9649 86.1 830.8 path->node hash load 0.99 (48522 entries / 49157 buckets) real 0m1.478s user 0m1.121s sys 0m0.351s TBR=scottmg Review URL: https://codereview.chromium.org/10918242 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@156751 0039d316-1c4b-4281-b951-d872f2087c98
-
- 12 Sep, 2012 3 commits
-
-
sbc@chromium.org authored
There are several places where we advise people to use custom_deps with { "foo" : None } to avoid checking out certain sub-repos. This caused wtf to crash on startup. BUG= Review URL: https://chromiumcodereview.appspot.com/10917220 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@156373 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
It relied on a behavior that doesn't hold on python 2.7 anymore w.r.t. tuple comparison to non-tuple objects. Now be explicit about having source files to be first. R=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10915240 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@156355 0039d316-1c4b-4281-b951-d872f2087c98
-
https://codereview.chromium.orgjam@chromium.org authored
Switch gcl/git upload to use https://codereview.chromium.org instead of https://chromiumcodereview.appspot.com. It's been a week since the SSL certificate got added, and the old DNS records had a TTL of 7 days. Review URL: https://chromiumcodereview.appspot.com/10910225 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@156316 0039d316-1c4b-4281-b951-d872f2087c98
-
- 11 Sep, 2012 4 commits
-
-
maruel@chromium.org authored
TBR=petermayo@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10910201 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@156105 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
Not sure this code worked. R=cmp@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10913193 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@156059 0039d316-1c4b-4281-b951-d872f2087c98
-
iannucci@chromium.org authored
Enables git to correctly detect copies and renames simultaneously. BUG=144408 Review URL: https://chromiumcodereview.appspot.com/10915190 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@155899 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
Fix issue with invalid password in keyring and git renames. R=petermayo@chromium.org BUG=144408 Review URL: https://chromiumcodereview.appspot.com/10907089 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@155898 0039d316-1c4b-4281-b951-d872f2087c98
-
- 07 Sep, 2012 1 commit
-
-
maruel@chromium.org authored
R=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10905160 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@155506 0039d316-1c4b-4281-b951-d872f2087c98
-
- 06 Sep, 2012 2 commits
-
-
maruel@chromium.org authored
Thin wrapper to process --bot and --testfilter to keep compability with gcl try and git-try and pass the data directly to Rietveld. R=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10916118 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@155176 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
This is mainly to reduce the spam that the commit queue adds to commit-bot@chromium.org as all follow up emails on code reviews are sent to it because it was adding itself as a reviewer, even if it is not. R=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10918075 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@155141 0039d316-1c4b-4281-b951-d872f2087c98
-
- 05 Sep, 2012 2 commits
-
-
maruel@chromium.org authored
Technically, all of these should be generated in /tmp but let's punt that for now. TBR=petermayo@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10915093 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@154965 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
R=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10920061 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@154955 0039d316-1c4b-4281-b951-d872f2087c98
-
- 04 Sep, 2012 1 commit
-
-
sbc@chromium.org authored
allows one to run the presubmit script without submitting locally (alternative would be to make this a warning I guess). Also, pass option.verbose through to DoGetTrySlaves() so it will log which PRESUBMIT files are taking part in the decision. BUG= Review URL: https://chromiumcodereview.appspot.com/10915072 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@154838 0039d316-1c4b-4281-b951-d872f2087c98
-
- 03 Sep, 2012 2 commits
-
-
maruel@chromium.org authored
Fix RawCheckout support in apply_issue.py. TBR=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10908063 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@154703 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
Patch in upload.py fix for python 2.7. TBR=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10914068 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@154696 0039d316-1c4b-4281-b951-d872f2087c98
-
- 31 Aug, 2012 1 commit
-
-
maruel@chromium.org authored
Print a meaningful error message so the user can try diagnosing himself. R=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10913026 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@154436 0039d316-1c4b-4281-b951-d872f2087c98
-
- 29 Aug, 2012 8 commits
-
-
maruel@chromium.org authored
R=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10887035 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@153976 0039d316-1c4b-4281-b951-d872f2087c98
-
erg@chromium.org authored
(Adding this after the pretty bad build bustage today.) Review URL: https://chromiumcodereview.appspot.com/10891039 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@153966 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
apply_issue.py didn't have output, making it a bit boring. Now it's super cute. R=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10889037 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@153948 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
R=nsylvain@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10885035 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@153940 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
This CL shall be reverted once the try server is fixed. R=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10900029 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@153916 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
R=petermayo@chromium.org BUG=124817 Review URL: https://chromiumcodereview.appspot.com/10890039 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@153908 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
Add new test cases to verify it properly handles "-1 +1, 2" and "-1 +1" for short files. R=petermayo@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10894036 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@153906 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
Issue and patchset can only be int, so store them as ints. This fixes git cl comment, which was broken by r152756. R=petermayo@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10894019 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@153885 0039d316-1c4b-4281-b951-d872f2087c98
-
- 22 Aug, 2012 3 commits
-
-
maruel@chromium.org authored
This ensures we don't send incorrect values like None or random strings. R=cmp@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10836348 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@152756 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
R=cmp@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10828393 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@152752 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
That's the language all the kool kids are using. TBR=xusydoc@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10874003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@152711 0039d316-1c4b-4281-b951-d872f2087c98
-
- 13 Aug, 2012 1 commit
-
-
gavinp@chromium.org authored
I recently needed to run presubmit tests in depot_tools, and I was helpfully told: $ gcl presubmit mychange ... eliding stuff ... Install google_appengine sdk in /scratch/scratch/chrome/depot_tools_git/testing_support or higher up This was a lie. If you put google_appengine there, the same presubmit tools that told you to install it there will fail you for numerous style violations contained within appengine itself. This change makes us ignore those by blacklisting google_appengine inside of testing_support. Review URL: https://chromiumcodereview.appspot.com/10829293 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@151210 0039d316-1c4b-4281-b951-d872f2087c98
-
- 12 Aug, 2012 1 commit
-
-
gavinp@chromium.org authored
Many, many times this has burned me badly; if you have a dirty index, git try will just upload your commits, and not the index. R=nsylvain@chromium.org BUG=None Review URL: https://chromiumcodereview.appspot.com/10836180 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@151202 0039d316-1c4b-4281-b951-d872f2087c98
-
- 10 Aug, 2012 1 commit
-
-
wittman@chromium.org authored
BUG=138104 TEST= Review URL: https://chromiumcodereview.appspot.com/10826035 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@150952 0039d316-1c4b-4281-b951-d872f2087c98
-
- 09 Aug, 2012 1 commit
-
-
brettw@chromium.org authored
I can never remember the syntax (and always forget I have to specify the branch to set the upstream branch on) when using "git branch". This patch just allows git-cl upstream to set the branch if an argument is provided. With no arg it prints the current one as it does today. Review URL: https://chromiumcodereview.appspot.com/10825285 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@150945 0039d316-1c4b-4281-b951-d872f2087c98
-
- 07 Aug, 2012 1 commit
-
-
mark@chromium.org authored
on all modified files, and don't require (c). BUG=140977 Review URL: https://chromiumcodereview.appspot.com/10824191 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@150417 0039d316-1c4b-4281-b951-d872f2087c98
-
- 02 Aug, 2012 2 commits
-
-
joshua.lock@intel.com authored
Using urllib for SSL connections when behind a proxy is known to be broken. Upstream has fixed this in urllib2 in Python 2.6.3 and newer so replace uses of urllib for SSL connections with urllib2 methods. R=maruel@chromium.org BUG=134165 TEST=gclient sync behind corporate proxy. Submitting this CL with git_cl. Review URL: https://chromiumcodereview.appspot.com/10825107 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@149742 0039d316-1c4b-4281-b951-d872f2087c98
-
szager@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/10830108 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@149649 0039d316-1c4b-4281-b951-d872f2087c98
-
- 01 Aug, 2012 1 commit
-
-
scottmg@google.com authored
Make 'deplist' and 'depfile' synonyms temporarily during transition of name in gyp. BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/10823120 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@149425 0039d316-1c4b-4281-b951-d872f2087c98
-
- 31 Jul, 2012 3 commits
-
-
cmp@chromium.org authored
TBR=szager@chromium.org Review URL: https://chromiumcodereview.appspot.com/10825122 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@149322 0039d316-1c4b-4281-b951-d872f2087c98
-
szager@chromium.org authored
TBR=cmp@chromium.org Review URL: https://chromiumcodereview.appspot.com/10826064 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@149319 0039d316-1c4b-4281-b951-d872f2087c98
-
szager@chromium.org authored
This simplifies some of the git-submodule stuff. For example, deps2hooks.bat can be changed from this: @echo off setlocal set TMP_GCLIENT_FILE=.tmp.gclient echo solutions=[{"name":".","url":None,"deps_file":".DEPS.git"}] > %TMP_GCLIENT_FILE% FOR /F "delims=" %%a in ('call gclient hookinfo --gclientfile=%TMP_GCLIENT_FILE%') do set HOOKINFO=%%a set HOOKINFO="call %HOOKINFO:;= & call%" cd .. cmd /c %HOOKINFO% del /F src\%TMP_GCLIENT_FILE% ... to this: @echo off setlocal call gclient runhooks --spec="solutions=[{'name':'src','url':None,'deps_file':'.DEPS.git'}]" Review URL: https://chromiumcodereview.appspot.com/10835045 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@149223 0039d316-1c4b-4281-b951-d872f2087c98
-