TBR=mmoss@chromium.org

BUG=


Review URL: https://chromiumcodereview.appspot.com/12330179

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

BUG=none


Review URL: https://chromiumcodereview.appspot.com/12330131

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

Currently, when we run the OWNERS check, we print the list of directories
that contain the relevant OWNERS files for any modified files in a change
still needing approval. 

This has two problems:

1) if we bubble all the way up to the top level OWNERS, we print "" instead of
"src/" or something more useful (bug 157191)

2) for OWNERS files that contain per-file set-noparent entries (like changes to IPC messages), this can be really confusing because an owner of other stuff in the directory might've approved things already.

This change will now print the list of files in the CL that are still unapproved.
This might be a lot more verbose (since you get N lines rather than 1 for N files in a given directory), but hopefully it'll be clearer in the two cases above.

Also, this change takes care of some lingering clean-up in the code to rename some methods to be clearer.

R=maruel@chromium.org
BUG=157191




Review URL: https://chromiumcodereview.appspot.com/12314044

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

Review URL: https://chromiumcodereview.appspot.com/12320043

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

Printing the list of all the files examined can produce a lot
out of output and bury the interesting output.


BUG=


Review URL: https://chromiumcodereview.appspot.com/12315031

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

We were passing a unicode string in the env block to subprocess.Popen which makes it unhappy. This forces the string to ascii first.

BUG=NONE
TEST=NONE


Review URL: https://chromiumcodereview.appspot.com/12255083

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

TBR=cmp@chromium.org


Review URL: https://chromiumcodereview.appspot.com/12218178

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

R=maruel@chromium.org, szager@chromium.org
BUG=


Review URL: https://chromiumcodereview.appspot.com/12301002

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

BUG='git cl dcommit' fails


Review URL: https://chromiumcodereview.appspot.com/12300006

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

R=maruel@chromium.org
BUG=


Review URL: https://chromiumcodereview.appspot.com/12300002

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

TBR=maruel@chromium.org
BUG=


Review URL: https://chromiumcodereview.appspot.com/12300005

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

R=cmp@chromium.org, maruel@chromium.org


Review URL: https://chromiumcodereview.appspot.com/12276007

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

Also add git-crsync as shorthand for `git crup --sync`.

TBR=iannucci@chromium.org

BUG=

Review URL: https://codereview.chromium.org/12279006

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

The Android WebView code includes several manually-maintained .mk files
that are included by the Android build system. These contain some fairly
long lines as they need to refer to deep pathnames, which means the CQ
cannot be used as it always triggers a presubmit warning.

Allow .mk files to have 200 columns per line instead, which should be
enough for the long paths that are used. Add a test case for this.

BUG=


Review URL: https://chromiumcodereview.appspot.com/12252067

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

gpylint doesn't like the use of "type" as a variable name (redefining a
builtin) - change the signature of the method to use the exc_ prefix
like sys.exc_*. The variables are not actually used anyway.

BUG=


Review URL: https://chromiumcodereview.appspot.com/12255048

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

This fix adds the crosbug.com shorturl scheme for the chromium-os project.

BUG=None
TEST=manual run of ./my_activity.py


Review URL: https://chromiumcodereview.appspot.com/12226135

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

This is basically a wrapper script for running a WebKit change against
the try bots.  It can be run from within third_party/WebKit and will try to apply the patch against ToT WebKit.

This tells the try server we want to use ToT WebKit by passing third_party/WebKit@HEAD as the first line of the patch file.

BUG=None
Review URL: https://codereview.chromium.org/12226013

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

BUG=


Review URL: https://chromiumcodereview.appspot.com/12035089

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

TBR=laforge@chromium.org
BUG=


Review URL: https://chromiumcodereview.appspot.com/12225116

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

Chromite will be added to the Chrome checkout at third_party/chromite -
detect when the checkout is a gclient checkout, and look for chromite at
that path.

BUG=None
TEST=ran 'cbuildbot' in both a repo and gclient checkout.


Review URL: https://chromiumcodereview.appspot.com/12094111

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

BUG=none


Review URL: https://chromiumcodereview.appspot.com/12094102

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

BUG=172935
Review URL: https://codereview.chromium.org/12089080

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

Review URL: https://codereview.chromium.org/12111002

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

Use closest git commit to the target svn revision.
Otherwise --checkout may be required to create
a git commit for a specific svn revision.

NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/11886086

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

R=maruel@chromium.org
BUG=163426

Review URL: https://chromiumcodereview.appspot.com/12047089

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

BUG=169945


Review URL: https://chromiumcodereview.appspot.com/12047061

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

BUG=171501
R=szager@chromium.org


Review URL: https://chromiumcodereview.appspot.com/12040027

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

The old code would always exit 0, because it returned the exit status of the if command

BUG=none
TEST=try to run git crup in src/ with unstaged changes. Should bail out and not run hooks
R=szager@chromium.org


Review URL: https://chromiumcodereview.appspot.com/12040046

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

BUG=171500
R=szager@chromium.org


Review URL: https://chromiumcodereview.appspot.com/12051022

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

Very long comments, like when posting a diff that failed to apply, cause extra
burden on the Commit Queue logs.

TBR=csharp@chromium.org
BUG=


Review URL: https://chromiumcodereview.appspot.com/11929027

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

It's done for the Commit Queue.

TBR=csharp@chromium.org
BUG=


Review URL: https://chromiumcodereview.appspot.com/11941019

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

r176777 introduced a change that relied on os.path.samepath, which does not exist on Windows.

BUG=


Review URL: https://chromiumcodereview.appspot.com/12021013

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

previously we would return "*" as one of the suggested owners when
a CL included a file that anyone could approve. If the change had
other owners, the "*" was unnecessary, and if the change only included
wildcard-owned files, "*" isn't very helpful, so I've changed the text slightly.

R=maruel@chromium.org
BUG=169168


Review URL: https://chromiumcodereview.appspot.com/11867016

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

The previous --force-branch option is renamed --checkout (the old name preserved for backwards compatibility).

Usage:
  --checkout         Create a branch and check it out.
  --create           Create a branch.
  -n, --name <name>  Specify the name of branch to create or reset.
                       This will force the branch using 'git branch -f '.
  -q, --quiet        Quiet.

NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/11962003

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

NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/11987002

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

TBR=scheib@chromium.org
NOTRY=true



Review URL: https://chromiumcodereview.appspot.com/11959013

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

pylint checks were erronously excluded for PRESUBMIT.py files in
root directory of repos because the regex was getting a './' added.

R=iannucci@chromium.org
TBR=M-A

Review URL: https://chromiumcodereview.appspot.com/11880040

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

This fixes a potential issue with RunPylint in windows where paths
can contain special characters.

TBR=iannucci@chromium.org


Review URL: https://chromiumcodereview.appspot.com/11786007

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

Review URL: https://codereview.chromium.org/11778024

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

It's bad python practice to use a class-level member that is mutable. self.calls
is already initialized in __init__ so it's not useful (and dangerous) to have it
as a class member.

R=cmp@chromium.org
BUG=


Review URL: https://chromiumcodereview.appspot.com/11801018

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