- 22 Jul, 2016 1 commit
-
-
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 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/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
-
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/c3683107b331a0f9f0392c0a376ecf8143b1e2ac Revert of shutil recipe_module: port chromium_utils rmtree implementation. (patchset #2 id:20001 of https://codereview.chromium.org/2146523003/ ) (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/2147703002
-
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/774c69370e740a8a5c85dce90f88a2bbd11911f5 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/2148493002
-
tandrii authored
R=andybons@chromium.org BUG= Review-Url: https://codereview.chromium.org/2135313005
-
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/d6ce1c413a63efd635af9cac02d21f52228c97bd Make Step API support type long in commands (borenet@chromium.org) R=borenet@chromium.org BUG= TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2147623003
-
tandrii authored
R=andybons@chromium.org BUG= Review-Url: https://codereview.chromium.org/2143693002
-
tandrii authored
Follow up of https://codereview.chromium.org/2144723002. BUG=627540 TBR=andybons@chromium.org Review-Url: https://codereview.chromium.org/2139603004
-
tandrii authored
TBR=andybons@chromium.org BUG= Review-Url: https://codereview.chromium.org/2144723002
-
phajdan.jr authored
BUG=584079, 593999 Review-Url: https://codereview.chromium.org/2144473003
-
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/ce485a8a394dcc1561b5ecdd337a85002083de5b Fix bootstrap on windows (martiniss@chromium.org) https://crrev.com/ba76686b6729174d9b7c7900ada4946ea5329632 Revert of Fix bootstrap on windows (patchset #2 id:20001 of https://codereview.chromium.org/2136303002/ ) (martiniss@chromium.org) R=dnj@chromium.org,martiniss@chromium.org BUG= TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2138293002
-
- 11 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/0a6ab680699355151bf3387074f4656c48698d72 Fix bootstrap cwd (martiniss@chromium.org) R=martiniss@chromium.org BUG= TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2137253002
-