- 25 Nov, 2014 2 commits
-
-
agable@chromium.org authored
R=iannucci@chromium.org, szager@chromium.org, pdr@chromium.org BUG=435729 Review URL: https://codereview.chromium.org/756233003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@293112 0039d316-1c4b-4281-b951-d872f2087c98
-
agable@chromium.org authored
R=kjellander@chromium.org Review URL: https://codereview.chromium.org/753383002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@293110 0039d316-1c4b-4281-b951-d872f2087c98
-
- 24 Nov, 2014 2 commits
-
-
kjellander@chromium.org authored
For Mac, we're depending on having both ios+mac for target_os. R=agable@chromium.org Review URL: https://codereview.chromium.org/753123003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@293085 0039d316-1c4b-4281-b951-d872f2087c98
-
thakis@chromium.org authored
See the bug for how these were built. The linux binaries were built by thestig@chromium.org. The windows binary was built by scottmg@chromium.org. The mac binary was built by thakis@chromium.org BUG=436109 TBR=maruel@chromium.org Review URL: https://codereview.chromium.org/755753002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@293084 0039d316-1c4b-4281-b951-d872f2087c98
-
- 20 Nov, 2014 3 commits
-
-
vapier@chromium.org authored
The reason cited for reverting was invalid, so reland the update. BUG=chromium:431514 Review URL: https://codereview.chromium.org/739393004 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@293049 0039d316-1c4b-4281-b951-d872f2087c98
-
smut@google.com authored
This reverts commit e642ee86. Conflicts: pylintrc Reason: All repos are borked. TBR=vapier Review URL: https://codereview.chromium.org/741023003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@293042 0039d316-1c4b-4281-b951-d872f2087c98
-
kjellander@chromium.org authored
BUG=435091 TESTED=Ran the following: git clone https://chromium.googlesource.com/external/webrtc cd webrtc git auto-svn cat .git/config Verified the URL was using http before this patch, and https after applying it. R=agable@chromium.org, mmoss@chromium.org Review URL: https://codereview.chromium.org/745473003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@293041 0039d316-1c4b-4281-b951-d872f2087c98
-
- 19 Nov, 2014 2 commits
-
-
jochen@chromium.org authored
BUG=none TBR=jrobbins@chromium.org Review URL: https://codereview.chromium.org/733923003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@293010 0039d316-1c4b-4281-b951-d872f2087c98
-
vapier@chromium.org authored
People are just ignoring these and bypassing the presubmit entirely, so disable all the warnings until someone feels like shaving these yaks one by one. BUG=chromium:431514 TEST=ran presubmit on build/ and so no more warnings Review URL: https://codereview.chromium.org/721833003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292997 0039d316-1c4b-4281-b951-d872f2087c98
-
- 18 Nov, 2014 1 commit
-
-
vapier@chromium.org authored
The current pylint is very old at this point. Pull in the latest version as it supports a lot more features. Also need to fix the pylint wrapper to actually update sys.path to use the local modules. We include the six module since not all bots have it, and it's small. This will trigger new warnings in files, but they look like legitimate issues that should be fixed at some point. BUG=chromium:431514 TEST=ran on some code bases and checked output Review URL: https://codereview.chromium.org/741503002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292988 0039d316-1c4b-4281-b951-d872f2087c98
-
- 15 Nov, 2014 2 commits
-
-
kbr@chromium.org authored
This reverts commit 451939d5. Throws an exception with a missing import, breaking Telemetry's presubmit checks and presumably presubmit checks for all Python files in Chromium. See comment on https://codereview.chromium.org/707353002/ . BUG=chromium:431514 Review URL: https://codereview.chromium.org/719313003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292955 0039d316-1c4b-4281-b951-d872f2087c98
-
vapier@chromium.org authored
The current pylint is very old at this point. Pull in the latest version as it supports a lot more features. Also need to fix the pylint wrapper to actually update sys.path to use the local modules. This will trigger new warnings in files, but they look like legitimate issues that should be fixed at some point. BUG=chromium:431514 TEST=ran on some code bases and checked output Review URL: https://codereview.chromium.org/707353002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292954 0039d316-1c4b-4281-b951-d872f2087c98
-
- 14 Nov, 2014 3 commits
-
-
scottmg@chromium.org authored
See also https://codereview.chromium.org/720033003/ BUG=311488,432748 Review URL: https://codereview.chromium.org/719343004 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292950 0039d316-1c4b-4281-b951-d872f2087c98
-
carlosk@chromium.org authored
and a link to it. BUG= Review URL: https://codereview.chromium.org/731473002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292940 0039d316-1c4b-4281-b951-d872f2087c98
-
jorgelo@chromium.org authored
BUG=None TEST=my_activity.py still works. Review URL: https://codereview.chromium.org/714373002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292935 0039d316-1c4b-4281-b951-d872f2087c98
-
- 12 Nov, 2014 2 commits
-
-
ncbray@chromium.org authored
BUG=https://code.google.com/p/chromium/issues/detail?id=430602 R=bradnelson@google.com, dpranke@chromium.org Review URL: https://codereview.chromium.org/718213002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292912 0039d316-1c4b-4281-b951-d872f2087c98
-
machenbach@chromium.org authored
BUG=415464 R=agable@chromium.org, tandrii@chromium.org Review URL: https://codereview.chromium.org/704773002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292909 0039d316-1c4b-4281-b951-d872f2087c98
-
- 11 Nov, 2014 1 commit
-
-
maruel@chromium.org authored
It currently crashes, which is sad. R=pgervais@chromium.org BUG= Review URL: https://codereview.chromium.org/683283006 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292899 0039d316-1c4b-4281-b951-d872f2087c98
-
- 06 Nov, 2014 1 commit
-
-
dnj@chromium.org authored
BUG=chromium:430343 TEST=None Review URL: https://codereview.chromium.org/701843004 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292868 0039d316-1c4b-4281-b951-d872f2087c98
-
- 05 Nov, 2014 2 commits
-
-
szager@chromium.org authored
BUG=430123 R=iannucci@chromium.org,agable@chromium.org Review URL: https://codereview.chromium.org/701893002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292853 0039d316-1c4b-4281-b951-d872f2087c98
-
apavlov@chromium.org authored
The "comments" command is taught to: - accept an arbitrary review issue id; - post comments on the current or specified review issue. This is required for the auto-rebaseline bot to notify a rebaseline requestor of the fact that the rebaseline has happened by means of posting a comment to the review issue that requested the rebaseline, as suggested by iannucci@. R=iannucci, szager, dpranke Review URL: https://codereview.chromium.org/693873002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292844 0039d316-1c4b-4281-b951-d872f2087c98
-
- 04 Nov, 2014 3 commits
-
-
pgervais@chromium.org authored
BUG= Review URL: https://codereview.chromium.org/697283002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292820 0039d316-1c4b-4281-b951-d872f2087c98
-
hinoka@chromium.org authored
BUG= Review URL: https://codereview.chromium.org/696023002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292819 0039d316-1c4b-4281-b951-d872f2087c98
-
thakis@chromium.org authored
Revert of Added orphaned compiled Python file checks. (patchset #2 id:20001 of https://codereview.chromium.org/621533002/) Reason for revert: Makes `git cl upload` very slow in chrome checkouts. Original issue's description: > Added orphaned compiled Python file checks. > > There have been a handful of cases where orphaned compiled Python files (.pyc > files without an accompanying .py file) have been loaded preferentially by > the presubmit scripts, causing weird problems. This adds a check and a mode for > such files, either warning the user or deleting them. > > BUG=None > TEST=local > - Tested against local repository. > > R=iannucci@chromium.org, szager@chromium.org > > Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=292703 TBR=iannucci@chromium.org,szager@chromium.org,dnj@chromium.org NOTREECHECKS=true NOTRY=true BUG=None Review URL: https://codereview.chromium.org/701683002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292818 0039d316-1c4b-4281-b951-d872f2087c98
-
- 02 Nov, 2014 1 commit
-
-
thakis@chromium.org authored
BUG=none Review URL: https://codereview.chromium.org/695113002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292792 0039d316-1c4b-4281-b951-d872f2087c98
-
- 29 Oct, 2014 1 commit
-
-
szager@chromium.org authored
TBR=iannucci@chromium.org BUG= Review URL: https://codereview.chromium.org/686183003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292733 0039d316-1c4b-4281-b951-d872f2087c98
-
- 28 Oct, 2014 1 commit
-
-
dnj@chromium.org authored
There have been a handful of cases where orphaned compiled Python files (.pyc files without an accompanying .py file) have been loaded preferentially by the presubmit scripts, causing weird problems. This adds a check and a mode for such files, either warning the user or deleting them. BUG=None TEST=local - Tested against local repository. R=iannucci@chromium.org, szager@chromium.org Review URL: https://codereview.chromium.org/621533002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292703 0039d316-1c4b-4281-b951-d872f2087c98
-
- 27 Oct, 2014 1 commit
-
-
sbc@chromium.org authored
Previously we just added an assert here but better to simply succeed in this case, while outputting the intial failure message. BUG=425696 Review URL: https://codereview.chromium.org/667793005 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292687 0039d316-1c4b-4281-b951-d872f2087c98
-
- 24 Oct, 2014 1 commit
-
-
agable@chromium.org authored
By directly using git-config rather than git-svn-init, this ensures that crazy values don't get set. It could be "safer" (e.g. check to see if any other git-svn configuration already exists and prompt before overwriting it), but I think that simplicity is better here. R=pgervais@chromium.org BUG=425838 Review URL: https://codereview.chromium.org/675713002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292640 0039d316-1c4b-4281-b951-d872f2087c98
-
- 23 Oct, 2014 1 commit
-
-
agable@chromium.org authored
R=iannucci@chromium.org Review URL: https://codereview.chromium.org/615443002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292613 0039d316-1c4b-4281-b951-d872f2087c98
-
- 22 Oct, 2014 4 commits
-
-
szager@chromium.org authored
R=vadimsh@chromium.org,mmoss@chromium.org BUG= Review URL: https://codereview.chromium.org/671003003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292601 0039d316-1c4b-4281-b951-d872f2087c98
-
dpranke@chromium.org authored
R=jamesr@chromium.org, agable@chromium.org Review URL: https://codereview.chromium.org/666293002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292598 0039d316-1c4b-4281-b951-d872f2087c98
-
agable@chromium.org authored
R=machenbach@chromium.org BUG=415464 Review URL: https://codereview.chromium.org/656623003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292588 0039d316-1c4b-4281-b951-d872f2087c98
-
agable@chromium.org authored
Although many things are private, the urls of the infra_internal and build_internal repositories are not. So let's just add a fetch recipe for them anyway. R=iannucci@chromium.org Review URL: https://codereview.chromium.org/544863006 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292586 0039d316-1c4b-4281-b951-d872f2087c98
-
- 21 Oct, 2014 1 commit
-
-
szager@chromium.org authored
BUG=chromium:425117 R=agable@chromium.org,kjellander@chromium.org Review URL: https://codereview.chromium.org/666713004 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292563 0039d316-1c4b-4281-b951-d872f2087c98
-
- 20 Oct, 2014 1 commit
-
-
szager@chromium.org authored
R=maruel@chromium.org,pgervais@chromium.org BUG= Review URL: https://codereview.chromium.org/661413005 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292537 0039d316-1c4b-4281-b951-d872f2087c98
-
- 17 Oct, 2014 1 commit
-
-
jam@chromium.org authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292513 0039d316-1c4b-4281-b951-d872f2087c98
-
- 16 Oct, 2014 2 commits
-
-
sbc@chromium.org authored
The base url was previously being generated as URL@BRANCH. I'm not sure if this works anywhere but it certainly doesn't on googlesource.com. Here we want URL/+/BRANCH. R=iannucci@chromium.org, sergeyberezin@chromium.org Review URL: https://codereview.chromium.org/652193004 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292493 0039d316-1c4b-4281-b951-d872f2087c98
-
enne@chromium.org authored
input_api.basename is a function, so printing it out yields: "The <function basename at 0x7fccd3735410> directory requires..>" Instead, use it to resolve the local presubmit path that has requested clang formatting. R=dpranke@chromium.org BUG=422332 Review URL: https://codereview.chromium.org/665493002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292491 0039d316-1c4b-4281-b951-d872f2087c98
-
- 13 Oct, 2014 1 commit
-
-
hinoka@chromium.org authored
Git footers doesn't seem to support v8, let alone know that trunk actually maps to the candidate branch. This should fix it. BUG= TBR=agable Review URL: https://codereview.chromium.org/645183002 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@292445 0039d316-1c4b-4281-b951-d872f2087c98
-