- 07 May, 2013 1 commit
-
-
mmoss@chromium.org authored
BUG=238390 Review URL: https://chromiumcodereview.appspot.com/15018003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@198572 0039d316-1c4b-4281-b951-d872f2087c98
-
- 06 May, 2013 1 commit
-
-
agable@chromium.org authored
As per the discussion in https://groups.google.com/a/chromium.org/forum/?fromgroups=#!topic/chromium-dev/Hl2QPNQ9E5o, this CL adds a 'git cl format' command. Users may use it at their own discretion. By default, it generates a diff against the local tracking (upstream) branch and runs clang-format-diff on that diff. When given the '--full' flag, it instead identifies all .cc, .cpp, and .h files in the diff and runs clang-format on the entirety of each of those files. Review URL: https://chromiumcodereview.appspot.com/14629012 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@198477 0039d316-1c4b-4281-b951-d872f2087c98
-
- 04 May, 2013 2 commits
-
-
dbeam@chromium.org authored
Chanot branch[0].isdigit()nging --milestone to use JSON source and adding prompt for branch mismatch. Sample output: $ drover --merge 196360 --milestone 27 Not all platforms have same branch number for M27. Here's a list of platforms on each branch: -(1453): win, ios, cf, mac, linux, android -(1460): cros Which branch? ('q' to cancel) 1453 Review URL: https://chromiumcodereview.appspot.com/14544004 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@198282 0039d316-1c4b-4281-b951-d872f2087c98
-
rcui@google.com authored
BUG=None TEST=Manual Review URL: https://chromiumcodereview.appspot.com/14746006 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@198266 0039d316-1c4b-4281-b951-d872f2087c98
-
- 03 May, 2013 6 commits
-
-
xusydoc@chromium.org authored
Follow-up to ilevy's comments in https://chromiumcodereview.appspot.com/14759006/ Review URL: https://chromiumcodereview.appspot.com/14874005 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@198236 0039d316-1c4b-4281-b951-d872f2087c98
-
szager@chromium.org authored
Add default 30 second nag timer to gclient subprocesses. BUG=227537 Review URL: https://chromiumcodereview.appspot.com/14826003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@198207 0039d316-1c4b-4281-b951-d872f2087c98
-
xusydoc@chromium.org authored
SVN traps SIGINT and attempts to clean itself up, but this results in hangs waiting for TCP. This patch does two things: daemonizes worker threads so they are culled when the main thread dies (is ctrl-C'd) and keeps track of spawned subprocesses to kill any remaining ones when the main program is ctrl-C'd. A user ctrl-C'ing gclient has to manually terminate hung SVN processes, so this introduces no extra data loss or hazard. stracing a hung SVN process shows that it is indeed hanging on TCP reads after receiving a SIGINT, implying there is an underlying but in the SVN binary. Review URL: https://chromiumcodereview.appspot.com/14759006 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@198205 0039d316-1c4b-4281-b951-d872f2087c98
-
rcui@google.com authored
BUG=None TEST=Ran locally Review URL: https://chromiumcodereview.appspot.com/14585006 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@198184 0039d316-1c4b-4281-b951-d872f2087c98
-
jbroman@chromium.org authored
It is somewhat surprising when git-cl, which acts as a git subcommand, launches a different editor. In particular, git has a config option (core.editor) which specifies the editor that should be used. Since we already respect $GIT_EDITOR, it makes sense for git-cl to respect core.editor and $VISUAL as well. R=maruel@chromium.org BUG=237504 Review URL: https://chromiumcodereview.appspot.com/14854003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@198101 0039d316-1c4b-4281-b951-d872f2087c98
-
iannucci@chromium.org authored
R=maruel@chromium.org BUG=166901 Review URL: https://chromiumcodereview.appspot.com/14878007 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@198031 0039d316-1c4b-4281-b951-d872f2087c98
-
- 02 May, 2013 3 commits
-
-
bratell@opera.com authored
BUG=236148 Review URL: https://chromiumcodereview.appspot.com/14104005 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@197872 0039d316-1c4b-4281-b951-d872f2087c98
-
szager@chromium.org authored
BUG=227537 R=maruel@chromium.org,cmp@chromium.org NOTRY=true Review URL: https://chromiumcodereview.appspot.com/14798003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@197790 0039d316-1c4b-4281-b951-d872f2087c98
-
groby@chromium.org authored
Clean up formatting a bit. Review URL: https://chromiumcodereview.appspot.com/14602017 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@197773 0039d316-1c4b-4281-b951-d872f2087c98
-
- 30 Apr, 2013 2 commits
-
-
szager@chromium.org authored
R=maruel@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/14643009 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@197419 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
It is called by logging.foo() so it breaks this use case. TBR=iannucci@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/14569002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@197190 0039d316-1c4b-4281-b951-d872f2087c98
-
- 29 Apr, 2013 2 commits
-
-
maruel@chromium.org authored
Without this change, if an entry is removed from a DEPS file, it won't be effectively deleted. Specify --delete_unversioned_trees so the dependency is properly deleted. TBR=szager@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/14531006 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@197081 0039d316-1c4b-4281-b951-d872f2087c98
-
janx@chromium.org authored
"fetch android" is a basic alias for "fetch chromium --target_os=android", which adds "target_os=['android']" to the gclient specs as described in chromium's AndroidBuildInstructions wiki page. NOTRY=true Review URL: https://chromiumcodereview.appspot.com/14315010 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@197031 0039d316-1c4b-4281-b951-d872f2087c98
-
- 27 Apr, 2013 1 commit
-
-
ilevy@chromium.org authored
Remove 'out' variable reference. If subprocess.communicate() throws an exception, out cannot contain data. BUG=236073 TBR=cmp Review URL: https://chromiumcodereview.appspot.com/14386022 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@196904 0039d316-1c4b-4281-b951-d872f2087c98
-
- 26 Apr, 2013 3 commits
-
-
maruel@chromium.org authored
Fix gcl which was broken in r196786 and wasn't fixed in r196798. TBR=dpranke@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/14314009 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@196808 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
The hostname was used instead of the rpc object. TBR=phajdan.jr@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/14305019 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@196798 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
This makes the commit logs much more useful for a build sheriff. Not only he sees who committed the CL but see who approved it directly at the log. This should help build sheriffs when they fail to contact the author and want to fallback on the reviewer for quick questions. R=dpranke@chromium.org BUG=76730 Review URL: https://chromiumcodereview.appspot.com/13800021 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@196786 0039d316-1c4b-4281-b951-d872f2087c98
-
- 24 Apr, 2013 6 commits
-
-
pkasting@chromium.org authored
This time, do not use the --config-dir hack on svn < 1.7 for the diff command that runs against a remote URL. This seems to trigger auth prompts (see revert comments on https://codereview.chromium.org/14297017/ ). Review URL: https://chromiumcodereview.appspot.com/14084009 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@196263 0039d316-1c4b-4281-b951-d872f2087c98
-
http://crrev.com/195336szager@chromium.org authored
BUG=235215 Review URL: https://chromiumcodereview.appspot.com/13890010 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@196259 0039d316-1c4b-4281-b951-d872f2087c98
-
groby@chromium.org authored
Empty branch output: # Empty branches git branch -D ipc2 # Empty. git branch -D m3 # Empty. Empty branches that have origin/* as upstream will not be considered for deletion. Provide distance from upstream in output, i.e. # Branch gypi-own - [master: ahead 1, behind 281] # Branch m2 - [master: ahead 1] Review URL: https://chromiumcodereview.appspot.com/14446006 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@196249 0039d316-1c4b-4281-b951-d872f2087c98
-
digit@chromium.org authored
The 'RemoveDirectory()' function in gclient_utils is deprecated and rmtree() should be used instead for consistency. This patch modifies all clients in depot_tools to use rmtree() instead and removes the RemoveDirectory function. + The SVNWrapperTestCase.testRevertNoDotSvn() mocking expectation has been slightly changed. This was required because the test invokes code that used to call gclient_utils.RemoveDirectory() directly, while only gclient_utils.rmtree() was mocked. BUG=NONE R=maruel@chromium.org, ilevy@chromium.org TEST=manually run gclient_utils_test / gclient_smoketest / scm_unittest / gclient_scm_test Review URL: https://chromiumcodereview.appspot.com/14134010 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@196133 0039d316-1c4b-4281-b951-d872f2087c98
-
ilevy@chromium.org authored
He has a lot of experience in this directory and we could use the additional reviewer. Also remove nsylvain, who is inactive. R=maruel@chromium.org Review URL: https://chromiumcodereview.appspot.com/13918011 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@196130 0039d316-1c4b-4281-b951-d872f2087c98
-
iannucci@chromium.org authored
R=ilevy@chromium.org, maruel@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/13966016 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@196023 0039d316-1c4b-4281-b951-d872f2087c98
-
- 21 Apr, 2013 3 commits
-
-
jam@chromium.org authored
Reverting r195363, r195328 and r195308. gcl upload is hanging for me unless I revert these changes. f:\src\chrome3\src>gcl upload o0m7 at this point, there's no output. If I press enter, I get: Got an exception Command svn diff --config-dir c:\users\jabdel~1\appdata\local\temp\tmp7dwbyp -r 195324:head svn://chrome-svn/chrome/trun k/src/webkit/glue/dom_operations_unittest.cc returned non-zero exit status 1 in f:\src\chrome3\src Authentication realm: <svn://chrome-svn:3690> 0039d316-1c4b-4281-b951-d872f2087c98 Password for 'jabdelmalek': Authentication realm: <svn://chrome-svn:3690> 0039d316-1c4b-4281-b951-d872f2087c98 > Tweak GenerateDiff: > > * Update comment for accuracy > * Keep the base diff command in a variable; this fixes an oversight where GenerateDiff would run svn diff without the --config-dir hack, and makes later refactoring easy > * Apply review comments from https://codereview.chromium.org/14247007/ to existing code > > Review URL: https://chromiumcodereview.appspot.com/14366029 TBR=pkasting@chromium.org Review URL: https://codereview.chromium.org/14297017 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@195422 0039d316-1c4b-4281-b951-d872f2087c98
-
jam@chromium.org authored
Reverting r195363, r195328 and r195308. gcl upload is hanging for me unless I revert these changes. f:\src\chrome3\src>gcl upload o0m7 at this point, there's no output. If I press enter, I get: Got an exception Command svn diff --config-dir c:\users\jabdel~1\appdata\local\temp\tmp7dwbyp -r 195324:head svn://chrome-svn/chrome/trun k/src/webkit/glue/dom_operations_unittest.cc returned non-zero exit status 1 in f:\src\chrome3\src Authentication realm: <svn://chrome-svn:3690> 0039d316-1c4b-4281-b951-d872f2087c98 Password for 'jabdelmalek': Authentication realm: <svn://chrome-svn:3690> 0039d316-1c4b-4281-b951-d872f2087c98 > Use --internal-diff on svn 1.7+ to slightly reduce disk thrashing. > > This just saves the need to create and remove an empty directory on every call to GenerateDiff. > > Review URL: https://chromiumcodereview.appspot.com/14064017 TBR=pkasting@chromium.org Review URL: https://codereview.chromium.org/14122021 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@195421 0039d316-1c4b-4281-b951-d872f2087c98
-
jam@chromium.org authored
Reverting r195363, r195328 and r195308. gcl upload is hanging for me unless I revert these changes. f:\src\chrome3\src>gcl upload o0m7 at this point, there's no output. If I press enter, I get: Got an exception Command svn diff --config-dir c:\users\jabdel~1\appdata\local\temp\tmp7dwbyp -r 195324:head svn://chrome-svn/chrome/trun k/src/webkit/glue/dom_operations_unittest.cc returned non-zero exit status 1 in f:\src\chrome3\src Authentication realm: <svn://chrome-svn:3690> 0039d316-1c4b-4281-b951-d872f2087c98 Password for 'jabdelmalek': Authentication realm: <svn://chrome-svn:3690> 0039d316-1c4b-4281-b951-d872f2087c98 Username: svn: Can't read stdin: End of file found "Fix typo." > Fix typo. > > Review URL: https://chromiumcodereview.appspot.com/13987010 TBR=pkasting@chromium.org Review URL: https://codereview.chromium.org/14304005 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@195420 0039d316-1c4b-4281-b951-d872f2087c98
-
- 20 Apr, 2013 2 commits
-
-
ilevy@chromium.org authored
Enable parallel tests on depot_tools. On Z620 presubmit times: 3m -> 35s. Review URL: https://codereview.chromium.org/14247012 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@195377 0039d316-1c4b-4281-b951-d872f2087c98
-
pkasting@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/13987010 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@195363 0039d316-1c4b-4281-b951-d872f2087c98
-
- 19 Apr, 2013 5 commits
-
-
szager@chromium.org authored
This is a bit of future-proofing. The git hooks in depot_tools/git-templates/hooks all try to forward to build/git-hooks. Currently, the only actual hook we have is in src/build/git-hooks/pre-commit, which (usefully) makes sure no one inadvertently commits a submodule change. If we want to add more git hooks, we can simply commit them to build/git-hooks, rather than asking everyone to modify .git/hooks in their checkout. NOTRY=true Review URL: https://chromiumcodereview.appspot.com/14230012 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@195336 0039d316-1c4b-4281-b951-d872f2087c98
-
pkasting@chromium.org authored
This just saves the need to create and remove an empty directory on every call to GenerateDiff. Review URL: https://chromiumcodereview.appspot.com/14064017 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@195328 0039d316-1c4b-4281-b951-d872f2087c98
-
pkasting@chromium.org authored
* Update comment for accuracy * Keep the base diff command in a variable; this fixes an oversight where GenerateDiff would run svn diff without the --config-dir hack, and makes later refactoring easy * Apply review comments from https://codereview.chromium.org/14247007/ to existing code Review URL: https://chromiumcodereview.appspot.com/14366029 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@195308 0039d316-1c4b-4281-b951-d872f2087c98
-
pkasting@chromium.org authored
Removes DiffItem, which was unused, and moves _DiffItemInternal below GenerateDiff, now its lone caller. No other code changes, so should be no behavioral change. Review URL: https://chromiumcodereview.appspot.com/14241038 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@195262 0039d316-1c4b-4281-b951-d872f2087c98
-
maruel@chromium.org authored
Make it more consistent across the tool. Using \s also includes \n, which confuses the tool. Add more tests. R=iannucci@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/14265007 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@195214 0039d316-1c4b-4281-b951-d872f2087c98
-
- 18 Apr, 2013 3 commits
-
-
pkasting@chromium.org authored
> Second take at handling users with locally overridden diff-cmds. > > The first take at https://chromiumcodereview.appspot.com/14130006 tried to pass > "--diff-cmd diff", which doesn't work for e.g. Windows cmd users who don't have > a "diff" executable installed. > > This take instead passes the "--internal-diff" switch which tells svn to ignore > any diff-cmd set locally and use its internal diff engine at all times. > > While implementing this I found that the existing code tried to work around this > problem in a different way, by setting up a bogus config dir. Since that > doesn't seem to work for me, and shouldn't be necessary with this patch anyway, > removed that code. > > Review URL: https://chromiumcodereview.appspot.com/14050007 TBR=pkasting@chromium.org Review URL: https://codereview.chromium.org/14188030 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@195009 0039d316-1c4b-4281-b951-d872f2087c98
-
pkasting@chromium.org authored
The first take at https://chromiumcodereview.appspot.com/14130006 tried to pass "--diff-cmd diff", which doesn't work for e.g. Windows cmd users who don't have a "diff" executable installed. This take instead passes the "--internal-diff" switch which tells svn to ignore any diff-cmd set locally and use its internal diff engine at all times. While implementing this I found that the existing code tried to work around this problem in a different way, by setting up a bogus config dir. Since that doesn't seem to work for me, and shouldn't be necessary with this patch anyway, removed that code. Review URL: https://chromiumcodereview.appspot.com/14050007 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@194990 0039d316-1c4b-4281-b951-d872f2087c98
-
bauerb@chromium.org authored
BUG=none Review URL: https://chromiumcodereview.appspot.com/13852011 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@194983 0039d316-1c4b-4281-b951-d872f2087c98
-