- 25 Jul, 2016 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 (or complain) recipe_engine: https://crrev.com/dc09456c693ea913133396141d8d2ad384254ae9 Validate previous step result before attempting to access (martiniss@chromium.org) R=martiniss@chromium.org BUG= TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2175393002
-
tandrii authored
Instead of long stacktrace, explain what went wrong. R=sergiyb@chromium.org,andybons@chromium.org BUG= Review-Url: https://codereview.chromium.org/2175343002
-
- 23 Jul, 2016 2 commits
-
-
martiniss authored
If an exception occurs during a try block which prevents a step from actually executing, then the finally block's attempt to access step.active_result will fail, and the original exception will be hidden behind a new exception from the recipe engine. This CL changes this behavior so that the original exception will always fire. Review-Url: https://codereview.chromium.org/2175103002
-
Nodir Turakulov authored
R=estaab@chromium.org, vadimsh@chromium.org TBR=tandrii@chromium.org BUG=629676 Review URL: https://codereview.chromium.org/2177713002 .
-
- 22 Jul, 2016 7 commits
-
-
nodir authored
add infra_swarmbucket config for infra_paths module to override git_cache. The value of path_config property value will be specified by kitchen. This will still delete cache every time. The goal of this change is to fix build.git swarmbucket builds. TBR=dnj@chromium.org BUG= Review-Url: https://codereview.chromium.org/2176913002
-
nodir authored
Revert of bot_update: fix git dir check (patchset #1 id:1 of https://codereview.chromium.org/2174923002/ ) Reason for revert: this is wrong. if directory is not empty, git clone fails Original issue's description: > bot_update: fix git dir check > > existance of a dir does not mean it is a git repo. Check .git dir > presence instead. > > This is the reason why swarmbucket build.git builds fail. > > TBR=hinoka@chromium.org > BUG= > > Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/cdac80c9fb4768bdc5bc42734e852e1f216224c9 TBR=hinoka@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review-Url: https://codereview.chromium.org/2173683003
-
agable authored
Since different projects can have different configurations for what the maximum value of the "Code-Review" label is in Gerrit, this teaches presubmit_support to inspect the maximum configured value and see who has granted it (the same behavior as the Submit button in Gerrit itself). R=andybons@chromium.org, martiniss@chromium.org, tandrii@chromium.org BUG=630738 Review-Url: https://codereview.chromium.org/2178673002
-
nodir authored
existance of a dir does not mean it is a git repo. Check .git dir presence instead. This is the reason why swarmbucket build.git builds fail. TBR=hinoka@chromium.org BUG= Review-Url: https://codereview.chromium.org/2174923002
-
Arikon authored
BUG= Review-Url: https://codereview.chromium.org/2165643002
-
agable authored
R=iannucci@chromium.org BUG=630312 Review-Url: https://codereview.chromium.org/2170623003
-
tandrii authored
BUG=630601 R=phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2173823002
-
- 21 Jul, 2016 4 commits
-
-
agable authored
R=iannucci@chromium.org BUG=630312 Review-Url: https://codereview.chromium.org/2165633007
-
techtonik authored
BUG= Review-Url: https://codereview.chromium.org/2019223002
-
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/ece774fd870a6b883cdd0f541a686ec3ca581613 Vendor requests. (martiniss@chromium.org) R=martiniss@chromium.org BUG= TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2170733002
-
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/8ca03782b1fabf6c838d5d58f2f527b52ecb95e5 Require recipe tryjob for CQ. (martiniss@chromium.org) R=martiniss@chromium.org BUG=628801 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2170683002
-
- 20 Jul, 2016 7 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/4278a31afc8618c43ee029b85eba8f5aa3491dc7 Revert of Set proper result (purple/red) for "Uncaught Exception" and "Failure reason" steps (patchset #1 id:1 of https://codereview.chromium.org/2142833002/ ) (martiniss@chromium.org) R=iannucci@chromium.org,estaab@chromium.org,martiniss@chromium.org,phajdan.jr@chromium.org BUG=none TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2170553003
-
techtonik authored
BUG= Review-Url: https://codereview.chromium.org/2017373002
-
jbroman authored
That recipe no longer exists, as of https://chromium.googlesource.com/chromium/tools/depot_tools/+/e0cc9e494929d2fd821a3542096f533001e7d67d Referring to it in the tutorial is misleading. It seems that the docs scripts haven't been run in awhile, so there are some unrelated changes as well. Review-Url: https://codereview.chromium.org/2162223003
-
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/e1a78d4ffd60b5bb4ae5a15a32f587a1a1f20e95 Better handle utf-8 log lines (martiniss@chromium.org) R=martiniss@chromium.org BUG=625221 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2159273005
-
mikecase authored
BUG= Review-Url: https://codereview.chromium.org/2166443002
-
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/01dfcf4635e3182d31e5de98f18c079bdc90de1d fetch: fix fetching using gitiles (do not recreate existing dirs) (phajdan.jr@chromium.org) R=phajdan.jr@chromium.org BUG=chromium:459840 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2171433002
-
phajdan.jr authored
This covers 99% of successful runs on all builders based on event_mon data. BUG=629025 Review-Url: https://codereview.chromium.org/2162143004
-
- 19 Jul, 2016 3 commits
-
-
techtonik authored
Traceback (most recent call last): ... File "...\\git_map_branches.py", line 303, in main mapper.start() File "...\\git_map_branches.py", line 169, in start self.__append_branch(root) File "...\\git_map_branches.py", line 272, in __append_branch self.__append_branch(child, depth=depth + 1) File "...\\git_map_branches.py", line 267, in __append_branch line.append(run('log', '-n1', '--format=%s', branch)) File "...\git_common.py", line 599, in run return run_with_stderr(*cmd, **kwargs)[0] File "...\git_common.py", line 668, in run_with_stderr raise subprocess2.CalledProcessError(retcode, cmd, os.getcwd(), ret, err) subprocess2.CalledProcessError: Command '...\\git.bat -c color.ui=never log -n1 --format=%s readme' returned non-zero exit status 128 in ... fatal: ambiguous argument 'readme': both revision and filename Use '--' to separate paths from revisions, like this: 'git <command> [<revision>...] -- [<file>...]' BUG= Review-Url: https://codereview.chromium.org/2153803002
-
sdefresne authored
Chrome on iOS downstream repository tracks Chromium via DEPS and wants to use cpplint canned presubmit check but the cpplint.py errors due at the include guard as it stops at the inner most git repository. Add a new parameter --project_root to cpplint.py. If set, it overrides the automatic detection of the project root that searches the root of the version control repository. The --root parameter cannot be used as it is used after the automatic resolution is performed (and allow chopping the head of the relative path while the need is to expend the path to include ios_internal/). BUG=598090 Review-Url: https://codereview.chromium.org/2036773002
-
asanka authored
git-map-branches -v -v -v currently indicates the status of the CL using a color. This change adds the status string alongside the CL to match the output of 'git-cl status'. $ git map-branches -v -v -v Before: A 354ccf6 [ ahead 1 ] https://codereview... B e701150 [ ahead 1 ] https://codereview... C * 094cedd [ ahead 1 ] https://codereview... After: A 354ccf6 [ ahead 1 ] https://codereview... (commit) B e701150 [ ahead 1 ] https://codereview... (lgtm) C * 094cedd [ ahead 1 ] https://codereview... (waiting) R=dpranke@chromium.org BUG=none Review-Url: https://codereview.chromium.org/2138703003
-
- 18 Jul, 2016 1 commit
-
- 15 Jul, 2016 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 (or complain) recipe_engine: https://crrev.com/1f4e7683113215117af63084dfef12ef06ac2896 Assert using realpath when confirming "env". (dnj@chromium.org) R=dnj@chromium.org BUG=chromium:628770 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2159553002
-
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/aaa126975449a79bd150191e0f52f1194fe66167 Rework vendoring, imports, packages, protobuf. (dnj@chromium.org) R=dnj@chromium.org BUG=chromium:628770 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2152403002
-
- 14 Jul, 2016 4 commits
-
-
dtu authored
Previously if you deleted a file that you had per-file owners on, it would fail the owners check. This fixes that. Originally, owners.Database used glob to enumerate the directory and added all the matching files in the directory to some dicts holding the owners information. If a CL deleted a file, it'd no longer be on the filesystem, so it wouldn't be in these dicts. There'd be no per-file owners information for it. With this patch, the Database no longer enumerates individual files. It instead keeps track of the glob patterns and checks the CL's files against the patterns at lookup time. BUG=622381 TEST=tests/owners_unittest.py && tests/owners_finder_test.py # Unit test included. Review-Url: https://codereview.chromium.org/2148153002
-
mbjorge authored
If an OWNERS file used the file: directive with a relative file path, but was using a root other than '/' (e.g. '/path/to/my/real/root'), then the include resolver would incorrectly leave a leading '/' on the include path. When os_path.join was then called, the leading '/' meant the path was treated as an absolute path and the join did not behave as expected. Review-Url: https://codereview.chromium.org/2148683003
-
hinoka authored
BUG= Review-Url: https://codereview.chromium.org/2117943002
-
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/b088fe54ff42ce40f6ccb0fbb51aa12a8f831491 Implement "refs" command for easy lookup of recipe module references (phajdan.jr@chromium.org) R=phajdan.jr@chromium.org BUG=none TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2151873003
-
- 13 Jul, 2016 7 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/c445d9dcff82a1cbe10102384b360bd9a5f3688c shtuil.remove: get rid of the flag_smart_tree. (tandrii@chromium.org) R=phajdan.jr@chromium.org,martiniss@chromium.org,tandrii@chromium.org BUG=584783 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2143923003
-
lhchavez authored
This is a cherry-pick of https://github.com/google/styleguide/pull/159 (plus some code that it depends on). BUG=627514 TEST=cpplint_unittest.py Review-Url: https://codereview.chromium.org/2146553003
-
lhchavez authored
This is a cherry-pick of https://github.com/google/styleguide/pull/158/ BUG=627202 TEST=cpplint_unittest.py Review-Url: https://codereview.chromium.org/2147593002
-
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/3de962bcccb8eac10810d7247618c5c9089b8fe7 shutil.rmtree: transition everything to new implementation. (tandrii@chromium.org) R=phajdan.jr@chromium.org,martiniss@chromium.org,tandrii@chromium.org BUG=584783 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2148793002
-
tandrii authored
If bots are specified manually on command line OR in PRESUBMIT.py, then old behavior remains, namely triggering just those bots. Otherwise, git cl try will now trigger CQ Dry Run. This is a good default and also resolves several bugs: * https://crbug.com/581150 where git cl try used to load cq.cfg from infra/config, but that's not always correct for all projects. * https://crbug.com/625697 where even if cq.cfg is in default location. it might be outdated. * https://crbug.com/585237 where git cl try would ignore special in CL description CQ_INCLUDE_TRYBOTS, which is processed by CQ, but not git cl try. BUG=625697,585237,581150 R=sergiyb@chromium.org,emso@chromium.org Review-Url: https://codereview.chromium.org/2147563003
-
tandrii authored
aka sergiyb@ was right. R=sergiyb@chromium.org BUG= Review-Url: https://codereview.chromium.org/2141283002
-
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/6bf9ca763de94c641328900358642ec44cf3932e Set proper result (purple/red) for "Uncaught Exception" and "Failure reason" steps (phajdan.jr@chromium.org) R=phajdan.jr@chromium.org BUG=none TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2143373002
-
- 12 Jul, 2016 1 commit
-
-
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/2b31483ac5f58781c138af391a35ccf3a3f63245 shutil recipe_module: port chromium_utils rmtree implementation. (tandrii@chromium.org) R=martiniss@chromium.org,tandrii@chromium.org BUG=584783 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2135383006
-