- 09 Sep, 2016 5 commits
-
-
scottmg authored
svn://svn.chromium.org/chrome-internal/trunk/src-internal/ doesn't seem to exist any more. d:\src>"d:\src\depot_tools\svn_bin\svn.exe" ls --non-interactive svn://svn.chromium.org/chrome-internal/trunk/src-internal/ svn: Can't connect to host 'svn.chromium.org': A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. R=brucedawson@chromium.org Review-Url: https://codereview.chromium.org/2327213002
-
kjellander authored
A while back, we removed the last pieces of code in src/third_party/libjingle/source/talk and it is no longer synced in Chromium's DEPS file. Clean this up to avoid confusion in the buildbot recipes. BUG=webrtc:4256 Review-Url: https://codereview.chromium.org/2325043002
-
agable authored
R=iannucci@chromium.org BUG=472386 Review-Url: https://codereview.chromium.org/2324073002
-
iannucci authored
R=agable@chromium.org, estaab@chromium.org, hinoka@chromium.org, martiniss@chromium.org BUG=643885 Review-Url: https://codereview.chromium.org/2328883002
-
agable authored
This is a reland of 082267a6 which was reverted in 4dcad385 BUG=472386 Review-Url: https://codereview.chromium.org/2325513002
-
- 08 Sep, 2016 10 commits
-
-
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/227f56ba6353986a7fe6d56fb283ad73c84e94d9 Track step nesting in StreamEngine (Reland). (dnj@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG=chromium:628770 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2325773004
-
iannucci authored
Extra comma changes type from dict to tuple :( TBR=agable@chromium.org, estaab@chromium.org, hinoka@chromium.org, martiniss@chromium.org BUG=643885 NOPRESUBMIT=true NOTRY=true Review-Url: https://codereview.chromium.org/2325763002
-
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/339eace18cba35868b8bbb039100448fbfc58d15 Revert of Track step nesting in StreamEngine. (patchset #5 id:80001 of https://codereview.chromium.org/2245113002/ ) (dnj@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG=chromium:628770 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2328713002
-
iannucci authored
Bot update now always runs regardless of environmental context. R=estaab@chromium.org, hinoka@chromium.org, martiniss@chromium.org, tandrii@chromium.org BUG=643885 Review-Url: https://codereview.chromium.org/2308193002
-
iannucci authored
This is the simplest change I can make here. The bot_update.py script already ignores the value of force (and sets it to True in all cases). This is an expectations-only change. I'll refactor the content of the bot_update.py script to account for this removed option in a followup CL as well. R=agable@chromium.org, estaab@chromium.org, hinoka@chromium.org, martiniss@chromium.org BUG=643885 Review-Url: https://codereview.chromium.org/2316323002
-
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/4b3ff6014bcb018aff9ae2755faf47cc304f47af Track step nesting in StreamEngine. (dnj@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG=chromium:628770 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2321813004
-
agable authored
Currently, if a DEPS file sets use_relative_paths, but *also* sets recursedeps, then the recursed-upon DEPS files still get checked out relative to the .gclient root. This change makes it so that recursed-upon DEPS files check their dependencies out relative to where their parent wants them to be, if that parent sets use_relative_paths=True. R=maruel@chromium.org Review-Url: https://codereview.chromium.org/2324513003
-
agable authored
This makes it easier to reason about tests which care about both the url being fetched and the directory to which it is being fetched, like those concerning relative paths and recursion. R=maruel@chromium.org Review-Url: https://codereview.chromium.org/2316963004
-
dsansome authored
BUG=641838 Review-Url: https://codereview.chromium.org/2318903002
-
hinoka authored
BUG= Review-Url: https://codereview.chromium.org/2316393002
-
- 07 Sep, 2016 8 commits
-
-
bpastene authored
Revert of Delete lots of svn logic from bot_update (patchset #5 id:80001 of https://codereview.chromium.org/2280213002/ ) Reason for revert: Everything's purple :( Original issue's description: > Delete lots of svn logic from bot_update > > R=hinoka@chromium.org > BUG=472386 > > Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/082267a659f94bf7bca5cb144e203ca0913abb3a TBR=hinoka@chromium.org,hinoka@google.com,iannucci@chromium.org,agable@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=472386 Review-Url: https://codereview.chromium.org/2320653003
-
bpastene authored
BUG=644722 Review-Url: https://codereview.chromium.org/2320573005
-
agable authored
R=hinoka@chromium.org BUG=472386 Review-Url: https://codereview.chromium.org/2280213002
-
tandrii authored
R=agable@chromium.org BUG= Review-Url: https://codereview.chromium.org/2319903002
-
martiniss authored
Revert of Require recipe tryjob in CQ (patchset #1 id:1 of https://codereview.chromium.org/2157503003/ ) Reason for revert: Downstream tester messed up now, see https://crbug.com/644609 Original issue's description: > Require recipe tryjob in CQ > > BUG=628801 > > Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/ac74456f1e697f38598cda9b1c1acedb80702c7a TBR=phajdan.jr@chromium.org,estaab@chromium.org,iannucci@chromium.org NOTRY=true (no try because the tester itself is broken) # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=628801, 644609 Review-Url: https://codereview.chromium.org/2322763002
-
andybons authored
BUG=612417 Review-Url: https://codereview.chromium.org/2317213002
-
agable authored
R=martiniss@chromium.org BUG=640251 Review-Url: https://codereview.chromium.org/2318843002
-
iannucci authored
R=hinoka@chromium.org, martiniss@chromium.org, tandrii@chromium.org BUG=643885 Review-Url: https://codereview.chromium.org/2310113003
-
- 06 Sep, 2016 1 commit
-
-
iannucci authored
This makes it so that regardless of what force is set to in the recipe, if you invoke bot_update.ensure_checkout, it will actually run. Removal of the force option (and related options) to come shortly. R=estaab@chromium.org, hinoka@chromium.org, martiniss@chromium.org, tandrii@chromium.org BUG=643885 Review-Url: https://codereview.chromium.org/2311673002
-
- 05 Sep, 2016 1 commit
-
-
qyearsley authored
This CL just contains spelling changes, a minor rewording, and addition of backticks for commands. Review-Url: https://codereview.chromium.org/2296393003
-
- 04 Sep, 2016 1 commit
-
-
estaab authored
BUG= Review-Url: https://codereview.chromium.org/2304393002
-
- 02 Sep, 2016 3 commits
-
-
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/b79776cf2fbb4822372bbc03ee9a44318c23af8c Add more information to the README.md (martiniss@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2308003003
-
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/11cda51b49aeaa3d56af70eb138ea28eabdd16bd Implement simple step whitelisting in recipe_engine. (iannucci@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG=459361 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2308883002
-
dnj authored
Currently, if one edits one's local CL description (e.g., `git commit --amend`), there is no path for that update to get pushed to Rietveld. This adds a "+" sentinel to the "-n" flag for `git cl description` that tells it to load description content from the local commit. The description should match what would be generated by "git cl upload". BUG=None TEST=local Review-Url: https://codereview.chromium.org/2307463002
-
- 01 Sep, 2016 4 commits
-
-
tandrii authored
R=machenbach@chromium.org,andybons@chromium.org BUG=602906 TEST=manual run of Angle recipe locally Review-Url: https://codereview.chromium.org/2299693003
-
qyearsley authored
For use in scripts that check try job results, this is currently practically the same as making requests to https://codereview.chromium.org/api/<cl-number>/<patchset-number>. However, I think that this might be more future-proof, since it fetches try job information from Buildbucket, and I expect to still work after migration from Rietveld to Gerrit. BUG=640354 Review-Url: https://codereview.chromium.org/2274743003
-
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/6ac56ce6c1693f924a5e1efaef28c792c3ce1052 Warn if no valid SSL python libraries installed (martiniss@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2299233002
-
scottmg authored
Can't just let the GC collect the pool on Windows. R=tandrii@chromium.org BUG=618726 Review-Url: https://codereview.chromium.org/2296723006
-
- 31 Aug, 2016 5 commits
-
-
andybons authored
BUG=602906 R=tandrii@chromium.org Recipe-Tryjob-Bypass-Reason: tandrii@ takes responsibility. Review-Url: https://codereview.chromium.org/2298883002
-
iannucci authored
Additions: * Remove PYTHONUNBUFFERED for svn tests This obviates https://codereview.chromium.org/2298863002/. recipe_engine: https://crrev.com/09345f9dcf4d2744f66d64766fd349ddf04f04ce Cancel PYTHONUNBUFFERED when unbuffered is explicitly set to False (iannucci@chromium.org) TBR=martiniss@chromium.org, phajdan.jr@chromium.org BUG=635641 Review-Url: https://codereview.chromium.org/2298893002
-
tandrii authored
Revert of It's time to bring subprocess42 to depot_tools. (patchset #1 id:1 of https://codereview.chromium.org/2291223003/ ) Reason for revert: Speculative revert because it might be breaking presubmit in depot_tools. Original issue's description: > It's time to bring subprocess42 to depot_tools. > > The immediate need is to properly kill git fetch and its sub-spawned > process from gclient. > > R=maruel@chromium.org, hinoka@chromium.org > BUG=635641 > > Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/6d3c290164360907fe08784093e36314069e689f TBR=hinoka@chromium.org,maruel@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=635641 Review-Url: https://codereview.chromium.org/2295043003
-
Andrii Shyshkalov authored
The immediate need is to properly kill git fetch and its sub-spawned process from gclient. R=maruel@chromium.org, hinoka@chromium.org BUG=635641 Review URL: https://codereview.chromium.org/2291223003 .
-
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/a93ebc8b69c18bce873f4df1979b6534f3c13e68 Organize environment whitelist, add PYTHONUNBUFFERED. (iannucci@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG=635641 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2290293004
-
- 30 Aug, 2016 2 commits
-
-
tandrii authored
Also spits out whatever output was produced by git fetch for debugging. BUG=635641 R=machenbach@chromium.org,hinoka@chromium.org Review-Url: https://codereview.chromium.org/2293013002
-
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
-