- 04 Mar, 2019 4 commits
-
-
Robert Iannucci authored
Downstream counterpart to https://chromium-review.googlesource.com/1481660 R=martiniss@chromium.org, nodir@chromium.org Bug: 930761, 927716 Change-Id: Ib6390b3d27bed79a01eefa966084233e71aa8235 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1496049 Commit-Queue: Robbie Iannucci <iannucci@chromium.org> Reviewed-by:
Nodir Turakulov <nodir@chromium.org> Reviewed-by:
Stephen Martinis <martiniss@chromium.org>
-
Alan Cutter authored
Change-Id: Ieebdb32ac316a443789e26f3be8d5dcb4f94c4b6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1470404Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Auto-Submit: Alan Cutter <alancutter@chromium.org>
-
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. recipe_engine: https://crrev.com/ae3604f2d2ae0291ecdc690672cc173e06a965ba [proto_support] Handle unicode filesystem walks. (iannucci@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: I2ab9ccba317f6a74d1c05748ca7d4cceddd0be28 Reviewed-on: https://chromium-review.googlesource.com/c/1498126Reviewed-by:
Recipe Roller <recipe-roller@chromium.org> Commit-Queue: Recipe Roller <recipe-roller@chromium.org>
-
Takuto Ikuta authored
Large parallelism causes too many open files error. Let me limit to 500 for Mac now. Bug: 936864 Change-Id: I2fccc8cf14483c6f34d84c84d82c44df6e4f3177 Reviewed-on: https://chromium-review.googlesource.com/c/1496675Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
-
- 03 Mar, 2019 1 commit
-
-
William Hesse authored
Bug: Change-Id: I058fcd85548c6e17f33de71ed2ad04549294e13d R: dpranke@chromium.org, iannucci@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/558250Reviewed-by:
Robbie Iannucci <iannucci@chromium.org> Reviewed-by:
Andrii Shyshkalov <tandrii@chromium.org> Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
-
- 02 Mar, 2019 2 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. recipe_engine: https://crrev.com/1a86f54ceba35d31196e8664958e53ba8f64cbd2 cq: fix a comment. (tandrii@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: I6b5d17b19a7f0c9d5802ba9959abc064f1ad98a3 Reviewed-on: https://chromium-review.googlesource.com/c/1497658Reviewed-by:
-
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. recipe_engine: https://crrev.com/0c4c8afe0d49874f5d596697b09057aa3be0a64c [source_manifest] Use PB for source manifest proto. (iannucci@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: I8e0942b4bd9bbe4b32b239aee88ab6c234f1cf07 Reviewed-on: https://chromium-review.googlesource.com/c/1496053Reviewed-by:
-
- 01 Mar, 2019 2 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. recipe_engine: https://crrev.com/921a0d25cc4e1ccc382944bcfadda66e4f04948a Add proper protobuf support to recipe engine. (iannucci@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: I0638634da889a2a3413bbf770bbb3caaa9dc5a47 Reviewed-on: https://chromium-review.googlesource.com/c/1496042Reviewed-by:
-
Vadim Shtayura authored
Includes bug fix for crbug.com/936911 and no other changes. R=tandrii@chromium.org, iannucci@chromium.org BUG=936911 Change-Id: I92e6725dc16b513579db87b81e389f9d61bdf62b Reviewed-on: https://chromium-review.googlesource.com/c/1496034Reviewed-by:
-
- 28 Feb, 2019 2 commits
-
-
Robert Iannucci authored
The newest version isn't quite ready for prime-time (it drops support for the `-repository` kitchen flag, which is still passed by buildbucket). I should be able to reland this later today. TBR=nodir@chromium.org, tandrii@chromium.org, vadimsh@chromium.org Bug: 936254 Change-Id: I18d25886cb008dc6f1a32a7ff85ec95b4a055b02 Reviewed-on: https://chromium-review.googlesource.com/c/1495205Reviewed-by:
Vadim Shtayura <vadimsh@chromium.org> Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
-
Henrique Ferreiro authored
Add a shortcut to extend the branches to be rebased with the current branch without having to type its name. Change-Id: If2fa117a6418590cdec5e65430f65dcdc825b611 Reviewed-on: https://chromium-review.googlesource.com/c/1483030Reviewed-by:
-
- 27 Feb, 2019 1 commit
-
-
Sergiy Belozorov authored
R=tandrii@chromium.org Bug: 935715 Change-Id: Ia7b8e2b187b59680270d694b24edf82493d1ed5c Reviewed-on: https://chromium-review.googlesource.com/c/1487212Reviewed-by:
-
- 26 Feb, 2019 2 commits
-
-
Vadim Shtayura authored
Hopefully it works better on OSX 10.14. R=tandrii@chromium.org BUG=936067 Change-Id: I097802a053b7f17b81f13a69c1f0c153dc327b10 Reviewed-on: https://chromium-review.googlesource.com/c/1489874Reviewed-by:
-
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. recipe_engine: https://crrev.com/81bee2d517dae82881c1193fe9e4860fc7355f5b Implement api.buildbucket.build_url helper property (sergiyb@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: I6f1e28a0aee6e4bf94ddaf49c30306098e0b46a5 Reviewed-on: https://chromium-review.googlesource.com/c/1489553Reviewed-by:
-
- 25 Feb, 2019 3 commits
-
-
Bruce Dawson authored
If you specify the wrong year to the -b flag then you may end up with an inverted time range that is entirely in the future, and this causes odd failures. Or at least that's what I've heard. This detects some of these error cases and halts. Also a one-line git cl format cleanup. Change-Id: Iede80faed00d4857443b3a1d853fa2ba69f47023 Reviewed-on: https://chromium-review.googlesource.com/c/1487744 Commit-Queue: Bruce Dawson <brucedawson@chromium.org> Auto-Submit: Bruce Dawson <brucedawson@chromium.org> Reviewed-by:
-
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. recipe_engine: https://crrev.com/03539506d1955ecf403784d0e900dd4d71011d97 scheduler: allow to opt-out of default tags in BuildbucketTrigger. (tandrii@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: Ic82acdf0eb4d5972ec6f96a93e64aff0a227160c Reviewed-on: https://chromium-review.googlesource.com/c/1487334Reviewed-by:
-
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. recipe_engine: https://crrev.com/79f79017de52c70f04314d85e2ed4d8b89546a17 cq: don't emit property when no new builds have been triggered. (tandrii@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: I159fba67df2724083d737ea369144243d1b7e03b Reviewed-on: https://chromium-review.googlesource.com/c/1485296Reviewed-by:
-
- 23 Feb, 2019 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. recipe_engine: https://crrev.com/2a09b5794c3305f9ec315ac948ee097095d17d90 [doc] Change default to "gen". (iannucci@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: I1d56d79c6a5523555d559d209336a31015cc0502 Reviewed-on: https://chromium-review.googlesource.com/c/1484465Reviewed-by:
-
Andrii Shyshkalov authored
This reverts commit 245337d6. Reason for revert: broke presubmit builders running run_presubmit.py recipe. Original change's description: > Pass target branch from tryserver in bot_update module > > Change-Id: I9d250c87355833ddc6c7d8f41e9e43575713925f > > Recipe-Nontrivial-Roll: build_limited_scripts_slave > Recipe-Nontrivial-Roll: skia > Recipe-Nontrivial-Roll: infra > Recipe-Nontrivial-Roll: build > Recipe-Nontrivial-Roll: release_scripts > Change-Id: I9d250c87355833ddc6c7d8f41e9e43575713925f > Reviewed-on: https://chromium-review.googlesource.com/c/1420977 > Commit-Queue: William Hesse <whesse@google.com> > Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org> TBR=whesse@google.com,ehmaldonado@chromium.org Change-Id: I89513b5bb053788291e79340c1685189adc794ec No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/1484655Reviewed-by:
-
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. recipe_engine: https://crrev.com/16dfe87b81be9acfd9e1763b20bd028f81d28e13 cq: add API to track triggered_build_ids. (tandrii@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: If06ae07e84bc2e729fc22adc0a08c5cba5a5abfa Reviewed-on: https://chromium-review.googlesource.com/c/1484453Reviewed-by:
-
- 22 Feb, 2019 13 commits
-
-
Vadim Shtayura authored
Change log: https://chromium.googlesource.com/infra/luci/luci-go/+log/ea41200b..1d348dee/lucicfg R=tandrii@chromium.org CC=andrewlamb@google.com Change-Id: I37b8a33971348b24db53ddf384ec3f5d7088e283 Reviewed-on: https://chromium-review.googlesource.com/c/1483872Reviewed-by:
-
Edward Lemur authored
Bug: 914655 Change-Id: I06a3ff39d884327fd407a5c50cc9730a375bef0b Reviewed-on: https://chromium-review.googlesource.com/c/1382974 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
-
Jordan Bayles authored
This patch makes some changes to the cpplint.py script for integration with the VSCode cpplint plugin, available here: https://marketplace.visualstudio.com/items?itemName=mine.cpplint Specifically, the "--repository" and "--headers" options currently cause our version of cpplint to crash, when we should at worst ignore them. A minor messaging tweak outputting the contents of the GetoptError, to make debugging input argument bugs easier, is included Change-Id: Id78616f5e2fdecb7da650042f0658623477ef9cb Reviewed-on: https://chromium-review.googlesource.com/c/1481938Reviewed-by:
-
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. recipe_engine: https://crrev.com/6605f1b0db4ddee8e7f40b73fe60cf5f8cbe794f Make StatusFailure work for non-exceptional failures only (sergiyb@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: Ia5515d64ea9f9f1d0c40f6b59a7f95bab774c625 Reviewed-on: https://chromium-review.googlesource.com/c/1483759Reviewed-by:
-
Quinten Yearsley authored
In the cpplint fn_size check, there's a part where it explicitly says to ignore macros, BUT the check for macro names didn't allow for digits, which are present in the IPC_* macro names, which look like "IPC_SYNC_MESSAGE_CONTROL4_1". There are two interesting things about this: 1. This bug was filed in 2011! Yay for fixing a 5-digit crbug with one-line fix! 2. This is a great example of how cpplint's regex-based checking is error-prone compared to something that actually has access to an abstract syntax tree. Bug: 83036 Change-Id: I49acd80c265d7f7bc633f2092388525b4518ae23 Reviewed-on: https://chromium-review.googlesource.com/c/1483283Reviewed-by:
Marc-Antoine Ruel <maruel@chromium.org> Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>
-
Sergiy Belozorov authored
R=iannucci@chromium.org Bug: 899266 Change-Id: Ia9b1f38590d636fa2858a2bd0bbf75d6b2cfe8fa Reviewed-on: https://chromium-review.googlesource.com/c/1483033Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Sergiy Belozorov <sergiyb@chromium.org>
-
William Hesse authored
Change-Id: I9d250c87355833ddc6c7d8f41e9e43575713925f Recipe-Nontrivial-Roll: build_limited_scripts_slave Recipe-Nontrivial-Roll: skia Recipe-Nontrivial-Roll: infra Recipe-Nontrivial-Roll: build Recipe-Nontrivial-Roll: release_scripts Change-Id: I9d250c87355833ddc6c7d8f41e9e43575713925f Reviewed-on: https://chromium-review.googlesource.com/c/1420977 Commit-Queue: William Hesse <whesse@google.com> Reviewed-by:
-
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. recipe_engine: https://crrev.com/55eb941710a0632db6973ddda5f53186c75b58fe [swarming] update client version (tikuta@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: I75559949eb574a047dc3f70f30abf08efba8f4be Reviewed-on: https://chromium-review.googlesource.com/c/1482714Reviewed-by:
-
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. recipe_engine: https://crrev.com/817452a9699e42bede7afbf465baad7e46e48c98 Add StatusAnyFailure post-processing step (sergiyb@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: Ic8b73674729c1b1b5832b98850a8cf07e8e0eb2f Reviewed-on: https://chromium-review.googlesource.com/c/1482712Reviewed-by:
-
Henrique Ferreiro authored
Previously, `git reparent-branch` would call `git rebase-update --no-fetch` effectively rebasing all branches. Limit the branches to update to the one being moved and its dependants. Change-Id: Ifb9f5b0ab4afe18f9c642ca47d683d9f3745bede Reviewed-on: https://chromium-review.googlesource.com/c/1476393 Commit-Queue: Robbie Iannucci <iannucci@chromium.org> Reviewed-by:
-
Henrique Ferreiro authored
When calling `git rebase-update <branches>`, don't emit messages about skipping branches which are not in the list of branches to be rebased. R=iannucci@chromium.org Change-Id: Ia5135d2e794646aaec68bca1aa64f4a73c98c186 Reviewed-on: https://chromium-review.googlesource.com/c/1476394 Commit-Queue: Robbie Iannucci <iannucci@chromium.org> Reviewed-by:
-
Caleb Rouleau authored
The current BugsFromDescription code returns ['1234, 5678'] but I think it should return ['1234', '5678']. For 'BUG=1234, 5678' this already works. This was preventing me from uploading https://chromium-review.googlesource.com/c/catapult/+/1481239 because of https://cs.chromium.org/chromium/src/third_party/catapult/PRESUBMIT.py?q=catapult/PRESUBMIT.py&sq=package:chromium&g=0&l=52 See https://logs.chromium.org/logs/catapult/buildbucket/cr-buildbucket.appspot.com/8920885706080647120/+/steps/presubmit/0/stdout Change-Id: Ibb3fca8c7a6ec664cdf9a38a8a7ac2bf66cb73ed Reviewed-on: https://chromium-review.googlesource.com/c/1481240 Auto-Submit: Caleb Rouleau <crouleau@chromium.org> Reviewed-by:
Aaron Gable <agable@chromium.org> Commit-Queue: Aaron Gable <agable@chromium.org>
-
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. recipe_engine: https://crrev.com/e0d5a65ab39980274038a6a2ce86d49325b60c7f Revert "[scheduler] Avoid copying tags or build input from the parent build" (tandrii@chromium.org) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: I121a43881c58e403b0e9b098ade3699c2c25de88 Reviewed-on: https://chromium-review.googlesource.com/c/1481947Reviewed-by:
-
- 21 Feb, 2019 7 commits
-
-
Edward Lemur authored
Bug: 934494 Change-Id: I31035d78888747386a37d4a8b05c148aaed37529 Reviewed-on: https://chromium-review.googlesource.com/c/1481937Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
-
Takuto Ikuta authored
Change-Id: I67a063d59ae3a579e77a1a6817a66f9d5f1ea53d Reviewed-on: https://chromium-review.googlesource.com/c/1480752Reviewed-by:
-
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. recipe_engine: https://crrev.com/f22a7898566b18893e82d34157f66efd3985d869 [scheduler] Avoid copying tags or build input from the parent build (phosek@google.com) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: If458575e51aaa51c04690543096c2aaa8cead408 Reviewed-on: https://chromium-review.googlesource.com/c/1481247Reviewed-by:
-
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. recipe_engine: https://crrev.com/5a85597323db240b1849349c973263f57ad1af0b [buildbucket] Do not inherit tags (nodir@google.com) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: Ib0d9064c467c7cc84d637f54594ea99e77ce1192 Reviewed-on: https://chromium-review.googlesource.com/c/1481234Reviewed-by:
Sergiy Belozorov <sergiyb@chromium.org> Commit-Queue: Recipe Roller <recipe-roller@chromium.org>
-
Takuto Ikuta authored
This CL shows bit more detail of value collision. Change-Id: I612f1a80cf160e7eebb2b48f0ddd1b8f9aaa81b1 Reviewed-on: https://chromium-review.googlesource.com/c/1475265 Auto-Submit: Takuto Ikuta <tikuta@chromium.org> Reviewed-by:
-
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. recipe_engine: https://crrev.com/b50c1d8111756a47ef8ac6cf7915e467fbd35eaf [buildbucket] Use camelCase in JSONPB (nodir@google.com) TBR=iannucci@chromium.org Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Change-Id: I0df4c88e8cdca5e37342f50c0678e9dc0e480c17 Reviewed-on: https://chromium-review.googlesource.com/c/1479909Reviewed-by:
-
Bruce Dawson authored
my_activity.py tries to determine the current user with the USER environment variable but on Windows it should use USERNAME. This change gets it to check both, which saves users from having to use the -u option and figure out what format the name should be in. Change-Id: Id2458d29f8a2635a96dd3859d6af0ed5aaea22bf Reviewed-on: https://chromium-review.googlesource.com/c/1479896Reviewed-by:
-
