- 30 Aug, 2016 1 commit
-
-
https://codereview.chromium.org/2282223002/machenbach authored
Reason for revert: Still in use. Breaks the v8 auto-rollers. https://cs.chromium.org/search/?q=roll-dep-svn&sq=package:chromium&type=cs Original issue's description: > Delete roll-dep-svn > > R=dnj@chromium.org > BUG=475320 > > Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/96ebc1d0e100021990312072e6eda6b8a0a81ee5 TBR=dnj@chromium.org,agable@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=475320 Review-Url: https://codereview.chromium.org/2297513002
-
- 29 Aug, 2016 7 commits
-
-
agable authored
R=martiniss@google.com BUG=640251 Recipe-Tryjob-Bypass-Reason: can't mock expectation changes downstream Review-Url: https://codereview.chromium.org/2285213004
-
agable authored
The URL helper functions were only used by this file, the svn custom_deps urls aren't referenced in any DEPS files I can find anymore, and the oilpan and buildspec configs aren't used by any recipes. R=hinoka@chromium.org BUG=640251 Review-Url: https://codereview.chromium.org/2291583003
-
thomasanderson authored
Review-Url: https://codereview.chromium.org/2285393002
-
agable authored
R=dnj@chromium.org BUG=475320 Review-Url: https://codereview.chromium.org/2282223002
-
tandrii authored
BUG=641384 R=phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2293523002
-
Paweł Hajdan, Jr authored
BUG=635421, 629679 R=tandrii@chromium.org Review URL: https://codereview.chromium.org/2286333002 .
-
tandrii authored
R=andybons@chromium.org BUG=641127 Review-Url: https://codereview.chromium.org/2280563003
-
- 26 Aug, 2016 3 commits
-
-
agable authored
BUG=640251,472386 Review-Url: https://codereview.chromium.org/2286793003
-
hinoka authored
BUG= Review-Url: https://codereview.chromium.org/2283043002
-
phajdan.jr authored
BUG=635421, 629679 Review-Url: https://codereview.chromium.org/2280973002
-
- 25 Aug, 2016 2 commits
-
-
tandrii authored
This provides env variable GCLIENT_KILL_GIT_FETCH_AFTER that kills git fetch if it produces no output for that many seconds. Note that this is not final patch, but an experiment. See http://crbug.com/635641#c24 for the deployment plan. BUG=635641 R=hinoka@chromium.org Review-Url: https://codereview.chromium.org/2241843002
-
agable authored
Also contains some other changes (to the example, and to safesync url) in order for test coverage to remain complete. Will be followed shortly by a wholesale removal of svn support from the gclient module. R=iannucci@chromium.org BUG=640251 Review-Url: https://codereview.chromium.org/2270083004
-
- 24 Aug, 2016 5 commits
-
-
agable authored
R=tandrii@chromium.org BUG=638750 Review-Url: https://codereview.chromium.org/2271873003
-
agable authored
These tools are relatively standalone, and only ever worked for SVN. Removing these is a good start to removing other SVN support code. R=maruel@chromium.org BUG=475321 Review-Url: https://codereview.chromium.org/2269413002
-
qyearsley authored
This should only change spelling in: - messages that are printed - comments - docstrings Review-Url: https://codereview.chromium.org/2277513002
-
qyearsley authored
I know this is an unimportant change, and that across Chromium, "try job" and "tryjob", and "try bot" and "trybot" are basically interchangeable, still I think it's nice to have the spelling consistent within one module. I prefer the version with a space just because that's what's used in the documentation: https://www.chromium.org/developers/testing/try-server-usage Review-Url: https://codereview.chromium.org/2269283002
-
recipe-roller authored
This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) recipe_engine: https://crrev.com/b9c3996241af9c5b0d60c0e7670897ecc3894e46 Fix recipes.py remote on Windows (phajdan.jr@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG=chromium:639891,chromium:584079 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2271803003
-
- 23 Aug, 2016 4 commits
-
-
agable authored
R=iannucci@chromium.org, martiniss@chromium.org BUG=640251 Recipe-Tryjob-Bypass-Reason: None of the affected recipes need safesync anyway, will manually roll Review-Url: https://codereview.chromium.org/2268893004
-
recipe-roller authored
This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) recipe_engine: https://crrev.com/772978c9b350224f62085c258408a02dcd97986c Remove preloaded 'google' module (tikuta@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG=638824 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2274673002
-
kmarshall authored
This CL adds a couple flags to "git cl archive". dry-run: Lists the cleanup tasks, but exits before any changes are made to Git. notags: Deletes branches only; does not create archival tags. R=tandrii@chromium.org,groby@chromium.org BUG= Review-Url: https://codereview.chromium.org/2276663002
-
tandrii authored
R=maruel@chromium.org,phajdan.jr@chromium.org BUG=640115 Review-Url: https://codereview.chromium.org/2272613002
-
- 22 Aug, 2016 5 commits
-
-
tandrii authored
R=maruel@chromium.org,emso@chromium.org BUG= Review-Url: https://codereview.chromium.org/2269593002
-
borenet authored
TBR=maruel BUG=skia:5660 Review-Url: https://codereview.chromium.org/2262283002
-
recipe-roller authored
This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) recipe_engine: https://crrev.com/6769b4ea18de83ed9676b90a0893417a47be62cc Fix autoroller for gitiles (support commit metadata) (phajdan.jr@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG=chromium:584079 Recipe-Tryjob-Bypass-Reason: Autoroller Review-Url: https://codereview.chromium.org/2266003002
-
phajdan.jr authored
This was requested in https://chromium-review.googlesource.com/c/373499 BUG=none Review-Url: https://codereview.chromium.org/2263103002
-
recipe-roller authored
This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) recipe_engine: https://crrev.com/990f604880743564b4d20029026487d7477c3ed3 fetch: set remote.origin.url to correct value rather than failing (phajdan.jr@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG=chromium:584079 Recipe-Tryjob-Bypass-Reason: Autoroller Review-Url: https://codereview.chromium.org/2263093002
-
- 19 Aug, 2016 2 commits
-
-
rmistry authored
BUG=skia:5627 Review-Url: https://codereview.chromium.org/2249983004
-
phajdan.jr authored
BUG=635421, 629679 Recipe-Tryjob-Bypass-Reason: downstream changes expected Review-Url: https://codereview.chromium.org/2264583002
-
- 18 Aug, 2016 5 commits
-
-
tandrii authored
This is because later calls to SetIssue() would fail. R=michaelpg@chromium.org BUG=611020 Review-Url: https://codereview.chromium.org/2259993002
-
tandrii authored
Previously there was a soup with add-hoc formatting with current branch name, which wasn't always set (see bug 611020). This CL makes sure all such operations now: * properly use types --int and --bool * go through the *only* appropriate get/set/unset function. Furthermore, tests were a mess wrt to raising exceptions when git processes terminated with an exception. This CL cleaned up, though I didn't go through all expectations, so some returns of empty stdout instead of raising CalledProcess error are likely remaining. Disclaimer: this CL is not necessarily fixing the referenced bug below, but it should at least provide better stacktrace when the bug manifestst itself. BUG=611020 R=agable@chromium.org Review-Url: https://codereview.chromium.org/2259043002
-
phajdan.jr authored
It's backwards compatible with the logic in win_tools.bat and is intended to eventually replace it. BUG=635421, 629679 Review-Url: https://codereview.chromium.org/2254843004
-
martiniss authored
BUG=623207 Review-Url: https://codereview.chromium.org/2236883003
-
tandrii authored
R=andybons@chromium.org BUG= Review-Url: https://codereview.chromium.org/2260453002
-
- 17 Aug, 2016 6 commits
-
-
hinoka authored
BUG= Review-Url: https://codereview.chromium.org/2253013004
-
iannucci authored
R=phajdan.jr@chromium.org, tandrii@chromium.org, vadimsh@chromium.org BUG= Review-Url: https://codereview.chromium.org/2245263007
-
tandrii authored
BUG=626364 R=andybons@chromium.org TEST=manual: https://chromium-review.googlesource.com/#/c/372138 Review-Url: https://codereview.chromium.org/2259583002
-
iannucci authored
R=phajdan.jr@chromium.org, vadimsh@chromium.org BUG= Review-Url: https://codereview.chromium.org/2250323003
-
tandrii authored
BUG=629863 R=seanmccullough@chromium.org,andybons@chromium.org Review-Url: https://codereview.chromium.org/2249303003
-
tandrii authored
Without it, upload fails because ref is invalid. BUG=622062 R=andybons@chromium.org Review-Url: https://codereview.chromium.org/2252043002
-