Commits · 1f312812293d4947933e3823541203a03838ef67 · Linshizhi / Chromium_Depot_tools

10 Feb, 2011 1 commit

scm.SVN.GenerateDiff() expects an array. · 1f312812

maruel@chromium.org authored Feb 10, 2011

So pass a single item array. The presubmit checks would fail otherwise.

This code path was definitely not exercised by the unit tests. Unit tests are
left as an exercise to the reader.

TBR=vbendeb
BUG=none
TEST=none

Review URL: http://codereview.chromium.org/6479006

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@74396 0039d316-1c4b-4281-b951-d872f2087c98

1f312812

09 Feb, 2011 3 commits

Modify presubmit checks to work on changed lines only. · ab05d582

maruel@chromium.org authored Feb 09, 2011

Unittest is being also updated.

TEST=manual
. run ./presubmit_unittest.py, observe success
.create a CL with code style violations (long lines, traling spaces) and observe the violations reported by 'git cl presubmit'

Patch contributed by Vadim

Review URL: http://codereview.chromium.org/6461011

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@74378 0039d316-1c4b-4281-b951-d872f2087c98

ab05d582

Improve presubmit_support.py handling of git checkout. · 7444c504

maruel@chromium.org authored Feb 09, 2011

It would fail otherwise when called directly from a git checkout subdirectory.

Updated unit test.

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/6459010

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@74281 0039d316-1c4b-4281-b951-d872f2087c98

7444c504

Improve tests to support more git versions and different outputs. · cdbecc49

maruel@chromium.org authored Feb 09, 2011

Disable warning W0404 in pylint.

BUG=none
TEST=should now work on googlers workstations

Review URL: http://codereview.chromium.org/6456010

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@74242 0039d316-1c4b-4281-b951-d872f2087c98

cdbecc49

08 Feb, 2011 3 commits

Update chromite to match the changes in chromiumos's git server. · 00e406b5

dianders@chromium.org authored Feb 08, 2011

Review URL: http://codereview.chromium.org/6286160

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@74182 0039d316-1c4b-4281-b951-d872f2087c98

00e406b5

Move reverting code from gclient_scm to scm to be able to reuse the code. · 07ab60e2

maruel@chromium.org authored Feb 08, 2011

There is 2 changes while moving the code:
- Making externals handling an option.
- Not trapping EnvironmentError.

This code will be reused by the commit queue.

TEST=unit tests
BUG=none

Review URL: http://codereview.chromium.org/6250177

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@74181 0039d316-1c4b-4281-b951-d872f2087c98

07ab60e2

Partial update of upload.py with r660 patch. · 082cccdb

maruel@chromium.org authored Feb 08, 2011

Make json files known as a text mimetype so that they can be seen in reviews.

Original review at http://codereview.appspot.com/4133046/

Also make sure that the copy in third_party and in git_cl are the same. I'll
remove the copy in git_cl in a later change.

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@74116 0039d316-1c4b-4281-b951-d872f2087c98

082cccdb

07 Feb, 2011 1 commit

Add a presubmit test that asserts TODOs have owners. · fdcc9f7f

estade@chromium.org authored Feb 07, 2011

For example, TODO(foo) is ok, but TODO: do something is not.

BUG=none
TEST=included

Review URL: http://codereview.chromium.org/6413004

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@74032 0039d316-1c4b-4281-b951-d872f2087c98

fdcc9f7f

02 Feb, 2011 2 commits

Add comment to clarify SHOULD_LEAK · 17f9e1f7

maruel@chromium.org authored Feb 02, 2011

NO CODE CHANGE.

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/6349053

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@73510 0039d316-1c4b-4281-b951-d872f2087c98

17f9e1f7

Add code to FakeReposBase to revert changes committed by a unit tests. · 26d762d0

maruel@chromium.org authored Feb 02, 2011

This enables seamless unit tests that check-in files without side-effects on the
remaining unit tests.

BUG=none
TEST=none
TBR=dpranke

Review URL: http://codereview.chromium.org/6250073

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@73456 0039d316-1c4b-4281-b951-d872f2087c98

26d762d0

01 Feb, 2011 1 commit

Update git_cl to look at right Tree Status URL for Chromium OS. · bf1a7ba9

msb@chromium.org authored Feb 01, 2011

Improves output of git cl tree to use Chromium OS status, not Chromium Status.

BUG=chromium-os:11449
TEST=Output of 'git cl tree' for Chromium OS includes output from Chromium OS instead of from Chromium.

Review URL: http://codereview.chromium.org/6312047

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@73292 0039d316-1c4b-4281-b951-d872f2087c98

bf1a7ba9

31 Jan, 2011 2 commits

Set chromite as executable. · 2512c172

maruel@chromium.org authored Jan 31, 2011


git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@73174 0039d316-1c4b-4281-b951-d872f2087c98

2512c172

Enforce inserting the depot_tools path as the first path in sys.path. · 075c639a

maruel@chromium.org authored Jan 31, 2011

Otherwise if there is a file named breakpad.py for instance in the current
directory, it will be loaded instead of the one in depot_tools.

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/6312034

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@73165 0039d316-1c4b-4281-b951-d872f2087c98

075c639a

28 Jan, 2011 1 commit

Add chromite wrapper to depot_tools. · e92854e4

maruel@chromium.org authored Jan 28, 2011

This file is a copy of the chromite wrapper that lives here:
http://git.chromium.org/gitweb/?p=chromite.git;a=blob;f=bin/chromite

Anush requested that this wrapper be placed into depot_tools so
that users will get the wrapper in their path. The wrapper will go
and find the real version of chromite based on the CWD.

Patch contributed by dianders@chromium.org.

Review URL: http://codereview.chromium.org/6299018

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@73011 0039d316-1c4b-4281-b951-d872f2087c98

e92854e4

26 Jan, 2011 1 commit

Unconditionally run gclient.bat on cygwin, as gyp doesn't work right when · cca55826

bradnelson@google.com authored Jan 26, 2011

run through cygwin python anymore (and fails subtlety).

BUG=None
TEST=None
RS=pkasting

Review URL: http://codereview.chromium.org/6400002

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@72718 0039d316-1c4b-4281-b951-d872f2087c98

cca55826

25 Jan, 2011 1 commit

Change the fake repo svn users to be emails. · 21548eb0

maruel@chromium.org authored Jan 25, 2011

Also remove the harcoding of the names by putting them in a variable.

This will help testing of the commit queue.

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/6350013

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@72454 0039d316-1c4b-4281-b951-d872f2087c98

21548eb0

24 Jan, 2011 1 commit

Don't prompt developers to add blank test= or bug= lines. · 5c76de94

jam@chromium.org authored Jan 24, 2011

Review URL: http://codereview.chromium.org/6391001

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@72349 0039d316-1c4b-4281-b951-d872f2087c98

5c76de94

20 Jan, 2011 1 commit

Tweak an informative message that has mislead at least 3 people so far. · cde3bb69

thakis@chromium.org authored Jan 20, 2011

Review URL: http://codereview.chromium.org/6331008

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@71893 0039d316-1c4b-4281-b951-d872f2087c98

cde3bb69

18 Jan, 2011 1 commit

Second attempt to work around App Engine flakiness by retrying upon URLError · 6c96e493

kbr@chromium.org authored Jan 18, 2011

("Connection timed out"). This allowed me to successfully upload a CL I had
been stuck on for a couple of hours.

Manually tested by raising both HTTPError and URLError within this code block
and ensuring both are caught correctly.

BUG=none
TEST=none


Review URL: http://codereview.chromium.org/6304006

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@71702 0039d316-1c4b-4281-b951-d872f2087c98

6c96e493

17 Jan, 2011 2 commits

Add --ignore-externals to svn update and svn checkout invocations. · ce117f61

maruel@chromium.org authored Jan 17, 2011

svn:externals are slow to update and are unnecessary in a gclient environment.

TEST=none
BUG=55904

Review URL: http://codereview.chromium.org/3421013

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@71604 0039d316-1c4b-4281-b951-d872f2087c98

ce117f61

Add more logging. Useful to understand `git try` slowness. · 3487f7f4

thakis@chromium.org authored Jan 17, 2011

Review URL: http://codereview.chromium.org/6247008

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@71602 0039d316-1c4b-4281-b951-d872f2087c98

3487f7f4

15 Jan, 2011 1 commit

Revert http://codereview.chromium.org/6345001/ because it apparently · e0c16f9f

kbr@chromium.org authored Jan 15, 2011

introduced a bug which I didn't encounter during testing.

BUG=none
TEST=none
TBR=maruel

Review URL: http://codereview.chromium.org/6340006

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@71520 0039d316-1c4b-4281-b951-d872f2087c98

e0c16f9f

14 Jan, 2011 1 commit

Work around App Engine flakiness by retrying upon URLError · 02a89f56

kbr@chromium.org authored Jan 14, 2011

("Connection timed out"). This allowed me to successfully upload a CL
I have been stuck on for a couple of hours.

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/6345001

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@71513 0039d316-1c4b-4281-b951-d872f2087c98

02a89f56

12 Jan, 2011 1 commit

Improve parsing of committed hash to be more resilient. · df947ea8

maruel@chromium.org authored Jan 12, 2011

TEST=Verified that if this code breaks, the presubmit checks fail.
BUG=none

Review URL: http://codereview.chromium.org/6079003

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@71229 0039d316-1c4b-4281-b951-d872f2087c98

df947ea8

11 Jan, 2011 2 commits

Update the repo wrapper · 2c64b712

msb@chromium.org authored Jan 11, 2011

BUG=
TEST=

Review URL: http://codereview.chromium.org/6179004

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@71101 0039d316-1c4b-4281-b951-d872f2087c98

2c64b712

Add support for post-dcommit/post-push hooks. · 0ba7f967

maruel@chromium.org authored Jan 11, 2011

Patch contributed by sadrul@chromium.org

Review URL: http://codereview.chromium.org/5972005

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@71097 0039d316-1c4b-4281-b951-d872f2087c98

0ba7f967

10 Jan, 2011 1 commit

Update the index in SendUpstream before tree check. · c76e675f

chase@chromium.org authored Jan 10, 2011

BUG=none
TEST=git cl dcommit/push performs an index update before
checking if the tree is dirty
Review URL: http://codereview.chromium.org/6172003

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@70908 0039d316-1c4b-4281-b951-d872f2087c98

c76e675f

03 Jan, 2011 1 commit

Fix path filtering to be on the relative path and no on the absolute path. · 5d0dc43e

maruel@chromium.org authored Jan 03, 2011

Add logging to presubmit_support.py.

BUG=
TEST=

Review URL: http://codereview.chromium.org/6004006

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@70378 0039d316-1c4b-4281-b951-d872f2087c98

5d0dc43e

23 Dec, 2010 2 commits

Import from git_cl instead of git_cl_repo. · b9a98324

chase@chromium.org authored Dec 23, 2010

Also update the PRESUBMIT check to exclude git_cl/.

BUG=none
TEST=git-cl hooks run
TBR=maruel@chromium.org
Review URL: http://codereview.chromium.org/5998008

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@70108 0039d316-1c4b-4281-b951-d872f2087c98

b9a98324

Move git-cl into depot_tools. · cc51cd03

chase@chromium.org authored Dec 23, 2010

BUG=none
TEST=git-cl works after move
Review URL: http://codereview.chromium.org/5012006

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@70011 0039d316-1c4b-4281-b951-d872f2087c98

cc51cd03

21 Dec, 2010 4 commits

patch from issue 5575008 · dcd1522a

evan@chromium.org authored Dec 21, 2010

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@69888 0039d316-1c4b-4281-b951-d872f2087c98

dcd1522a

Silence pylint warnings. · 4c6f18ca

maruel@chromium.org authored Dec 21, 2010

TBR=dpranke
BUG=
TEST=

Review URL: http://codereview.chromium.org/6051004

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@69848 0039d316-1c4b-4281-b951-d872f2087c98

4c6f18ca

Split FakeRepos class in two so it can be reused for commti-queue unit tests. · e0f81697

maruel@chromium.org authored Dec 21, 2010

FakeRepos is useful for offline SCM tests but it was hardcoded for depot_tools
requirements. Luckily, it was already modular enought that it merely required to
shuffle code around to split it in two classes to be able to reuse the code
as-is.

BUG=
TEST=

Review URL: http://codereview.chromium.org/6029004

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@69845 0039d316-1c4b-4281-b951-d872f2087c98

e0f81697

Workaround the fact that pylint calls sys.exit(). · fca53394

maruel@chromium.org authored Dec 21, 2010

Otherwise the following presubmit checks where never called.

TBR=nsylvain
BUG=
TEST=fixes unit tests

Review URL: http://codereview.chromium.org/5995003

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@69844 0039d316-1c4b-4281-b951-d872f2087c98

fca53394

15 Dec, 2010 1 commit

Add support for an --upstream_branch argument to make diffing against · 2e9ba2ac

dpranke@google.com authored Dec 15, 2010

things other than trunk easier.

    BUG=none
    R=maruel@chromium.org
    TEST=none

Review URL: http://codereview.chromium.org/5680006

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@69307 0039d316-1c4b-4281-b951-d872f2087c98

2e9ba2ac

14 Dec, 2010 1 commit

Enhance RunPylint to use white_list and black_list arguments. · bf38a7ed

maruel@chromium.org authored Dec 14, 2010

Change RunPylint to parse all .py files when one is modified.

Make all depot_tools/tests pass on pylint. That mostly meant fixing some
builtins aliasing, wrong alignment and turning off most remaining warnings.

BUG=none
TEST=unit tests

Review URL: http://codereview.chromium.org/5695007

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@69159 0039d316-1c4b-4281-b951-d872f2087c98

bf38a7ed

13 Dec, 2010 1 commit

Add RunPylint as a canned presubmit check. · e94aedc2

maruel@chromium.org authored Dec 13, 2010

Adding it as I figured out how to make it run correctly on ubuntu 10.4 and it's
used in enough places it warrants a canned check.

BUG=none
TEST=it self tests itself.

Review URL: http://codereview.chromium.org/5682011

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@69051 0039d316-1c4b-4281-b951-d872f2087c98

e94aedc2

02 Dec, 2010 1 commit

Make max len parametrable so it can be modified for users. · f31294b4

maruel@chromium.org authored Dec 02, 2010

Mostly useful for commit-queue as I want to both send the trace and be able to
read it locally.

TBR=jam

Review URL: http://codereview.chromium.org/5608001

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@68055 0039d316-1c4b-4281-b951-d872f2087c98

f31294b4

25 Nov, 2010 2 commits

Increment tries so it doesn't loop forever. · b7d3b24d

maruel@chromium.org authored Nov 25, 2010

TEST=none
BUG=none

Review URL: http://codereview.chromium.org/5257002

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@67359 0039d316-1c4b-4281-b951-d872f2087c98

b7d3b24d

Propagate host to _GetAuthToken() to make it coherent. · d528cb88

maruel@chromium.org authored Nov 25, 2010

TEST=none
BUG=none

Review URL: http://codereview.chromium.org/5342001

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@67358 0039d316-1c4b-4281-b951-d872f2087c98

d528cb88