- 09 Oct, 2012 2 commits
-
-
cmp@chromium.org authored
R=maruel@chromium.org BUG=153446 Review URL: https://chromiumcodereview.appspot.com/11094033 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@160960 0039d316-1c4b-4281-b951-d872f2087c98
-
sivachandra@chromium.org authored
BUG=152209 Review URL: https://chromiumcodereview.appspot.com/11043031 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@160956 0039d316-1c4b-4281-b951-d872f2087c98
-
- 04 Oct, 2012 1 commit
-
-
maruel@chromium.org authored
TBR=csharp@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/11065002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@160027 0039d316-1c4b-4281-b951-d872f2087c98
-
- 03 Oct, 2012 2 commits
-
-
maruel@chromium.org authored
If a file is empty, it's not a binary file in the first place. This is to protect against improper uploads. From light testing, it seems binary file support is totally broken. This needs to be verified throughout before being enabled back. TBR=rogerta@chromium.org BUG=23608 Review URL: https://chromiumcodereview.appspot.com/11016032 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@159941 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
R=nsylvain@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/11046016 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@159909 0039d316-1c4b-4281-b951-d872f2087c98
-
- 02 Oct, 2012 3 commits
-
-
hinoka@chromium.org authored
BUG= Review URL: https://chromiumcodereview.appspot.com/11046004 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@159765 0039d316-1c4b-4281-b951-d872f2087c98
-
iannucci@chromium.org authored
BUG=125983 Review URL: https://chromiumcodereview.appspot.com/10945002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@159732 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
TBR=iannucci@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/11032023 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@159715 0039d316-1c4b-4281-b951-d872f2087c98
-
- 01 Oct, 2012 4 commits
-
-
maruel@chromium.org authored
The object creation order was wrong TBR=rogerta@chromium.org BUG=149941 Review URL: https://chromiumcodereview.appspot.com/11024009 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@159524 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
Stop using stderr so the output is more readable on buildbot. Otherwise stderr arrives before stdout in apply_issue step. TBR=rogerta@chromium.org BUG=153284 Review URL: https://chromiumcodereview.appspot.com/11038002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@159508 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
It's to help users figuring out what is happening. TBR=rogerta@chromium.org BUG=153284 Review URL: https://chromiumcodereview.appspot.com/11028002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@159504 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
TBR=rogerta@chromium.org BUG=153284 Review URL: https://chromiumcodereview.appspot.com/11019015 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@159488 0039d316-1c4b-4281-b951-d872f2087c98
-
- 28 Sep, 2012 1 commit
-
-
maruel@chromium.org authored
Nobody cares about those .orig files and they get in the way on the try server for some specific license check. R=cmp@chromium.org BUG=152507 Review URL: https://chromiumcodereview.appspot.com/10983092 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@159329 0039d316-1c4b-4281-b951-d872f2087c98
-
- 27 Sep, 2012 1 commit
-
-
lliabraa@chromium.org authored
The try slaves run 'git revert' to clean up from the previous run. This only runs 'git reset --hard', which does not remove any untracked files, so if a CL adds a new file it will still be present after 'gclient revert'. Adding git clean -f -d removes untracked files (-f) and directories (-d). Review URL: https://chromiumcodereview.appspot.com/10986032 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@159020 0039d316-1c4b-4281-b951-d872f2087c98
-
- 26 Sep, 2012 1 commit
-
-
szager@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/10958056 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@158892 0039d316-1c4b-4281-b951-d872f2087c98
-
- 24 Sep, 2012 2 commits
-
-
ilevy@chromium.org authored
- and display a warning message BUG=146713 Review URL: https://chromiumcodereview.appspot.com/10956062 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@158377 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
For an unknown reason, 'svn propset svn:executable * foo.sh' doesn't work on Windows. It is even more awkward that 'svn propset svn:executable . foo.sh' works just fine, in particular, subversion replaces the value, as long as it's not an empty string, back to '*'. R=petermayo@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10967071 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@158281 0039d316-1c4b-4281-b951-d872f2087c98
-
- 21 Sep, 2012 5 commits
-
-
rogerta@chromium.org authored
BUG=149941 Review URL: https://chromiumcodereview.appspot.com/10958049 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@158068 0039d316-1c4b-4281-b951-d872f2087c98
-
sbc@chromium.org authored
BUG=http://code.google.com/p/chromium/issues/detail?id=151110 Review URL: https://chromiumcodereview.appspot.com/10956047 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@158064 0039d316-1c4b-4281-b951-d872f2087c98
-
rogerta@chromium.org authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@158049 0039d316-1c4b-4281-b951-d872f2087c98
-
rogerta@chromium.org authored
if depot tools is in the path. BUG=149941 Review URL: https://chromiumcodereview.appspot.com/10966042 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@158039 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
svn propdel svn:executable shouldn't be called on it. R=petermayo@chromium.org BUG=150960 Review URL: https://chromiumcodereview.appspot.com/10962038 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@158038 0039d316-1c4b-4281-b951-d872f2087c98
-
- 20 Sep, 2012 1 commit
-
-
maruel@chromium.org authored
This brings back git cl try to try at LKGR instead of HEAD, which is coherent with git-try and gcl try. R=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10963013 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@157810 0039d316-1c4b-4281-b951-d872f2087c98
-
- 18 Sep, 2012 3 commits
-
-
scottmg@google.com authored
Note that this unforks, windows ninja is now built from martine/ninja. TBR=iannucci@chromium.org Review URL: https://codereview.chromium.org/10947018 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@157451 0039d316-1c4b-4281-b951-d872f2087c98
-
rogerta@chromium.org authored
can be applied. This is the first part for this bug; the second part is in commands.py where this is called by the TS. BUG=149941 Review URL: https://chromiumcodereview.appspot.com/10917306 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@157355 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
Also specify the batch file on Windows. Python's subprocess is picky about that. TBR=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10939008 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@157337 0039d316-1c4b-4281-b951-d872f2087c98
-
- 17 Sep, 2012 1 commit
-
-
maruel@chromium.org authored
It runs it by specifying BASE/HEAD to stay at the current revision. This works around the lack of gclient sync when a DEPS file is applied on the Try Server when triggered from Rietveld. This was handled my the Try Server inside scripts/slave/chromium_command.py manually in the case a 'patch' is specified. TBR=rogerta@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10918264 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@157110 0039d316-1c4b-4281-b951-d872f2087c98
-
- 15 Sep, 2012 1 commit
-
-
ilevy@chromium.org authored
_check_output_svn sets the cwd to project_path therefore files should not have project_path. R=maruel@chromium.org,cmp@chromium.org BUG=149702 Review URL: https://chromiumcodereview.appspot.com/10909254 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@156953 0039d316-1c4b-4281-b951-d872f2087c98
-
- 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
-