Commits · fce7bc9e08256a6c59e85cd37fefe6c754d4bb6b · Linshizhi / Chromium_Depot_tools

14 Jun, 2018 5 commits

Add infradata/config repo to gclient. · fce7bc9e

Andrii Shyshkalov authored Jun 14, 2018

This way it'll be discoverable by run_presubmit recipe.

R=vadimsh

Bug: 613739
Change-Id: I84ec3b84292940e74e4d707e4d257eb560004f56
Reviewed-on: https://chromium-review.googlesource.com/1102010Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

fce7bc9e

Revert "gclient_scm: Use cherry-picking instead of rebasing." · 690d8d43

Andrii Shyshkalov authored Jun 14, 2018

This reverts commit c9121141.

Reason for revert: broke patch application on infra/config https://crbug.com/853032

Original change's description:
> gclient_scm: Use cherry-picking instead of rebasing.
> 
> We have a problem when in this situation, we checkout |patch| and rebase it on
> top of |base|, thus including an |extra commit| that we shouldn't.
> 
> o master
> |
> . o patch
> |/
> o extra commit
> |
> o base (what gclient synced src at)
> 
> This does merge-base between |patch| and |master|, and cherry-picks only the
> changes belonging to the patch.
> 
> Bug: 850812
> Change-Id: Id09ae1682e53b69ed49b2fb649310de6a6a8a29e
> Reviewed-on: https://chromium-review.googlesource.com/1098228
> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
> Reviewed-by: Aaron Gable <agable@chromium.org>

TBR=agable@chromium.org,ehmaldonado@chromium.org

Change-Id: Ib3feeee2f44f5441713383f1dbf08db16fae4717
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 850812, 853032
Reviewed-on: https://chromium-review.googlesource.com/1101977Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

690d8d43

gclient_scm: Use cherry-picking instead of rebasing. · c9121141

Edward Lemur authored Jun 14, 2018

We have a problem when in this situation, we checkout |patch| and rebase it on
top of |base|, thus including an |extra commit| that we shouldn't.

o master
|
. o patch
|/
o extra commit
|
o base (what gclient synced src at)

This does merge-base between |patch| and |master|, and cherry-picks only the
changes belonging to the patch.

Bug: 850812
Change-Id: Id09ae1682e53b69ed49b2fb649310de6a6a8a29e
Reviewed-on: https://chromium-review.googlesource.com/1098228
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Aaron Gable <agable@chromium.org>

c9121141

gclient: Don't allow None URLs (except in .gclient files) · 0c91147d

Edward Lemur authored Jun 14, 2018

This reverts commit crrev.com/4e9b50ab
and thus relands the following commits:

  ebdd0db4: "gclient: Remove URLs from hierarchy."
  54a5c2ba: "gclient: Refactor PrintRevInfo"
  083eb25f: "gclient: Don't allow URL to be None."

When a None URL is specified in a .gclient file, and a DEPS file is
given, the DEPS file is treated as a .gclient file and its dependencies
are added.

Bug: 839925

Change-Id: I1068b66487874bfa0a788bf9da5273714b6ad39e
Reviewed-on: https://chromium-review.googlesource.com/1083340
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Aaron Gable <agable@chromium.org>
Reviewed-by: Michael Moss <mmoss@chromium.org>

0c91147d

Allow CheckPatchFormatted to cause a failure · 425d9ce2

Nodir Turakulov authored Jun 14, 2018

Extend CheckPatchFormatted to allow other presubmit result types.

Change-Id: I1c592d73657aa9c8a8fd04dbd66f086ce7bb1274
Reviewed-on: https://chromium-review.googlesource.com/1100127Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Nodir Turakulov <nodir@chromium.org>

425d9ce2

13 Jun, 2018 3 commits

gclient recipe: use env_suffixes insetad of custom ENV manipulation. · cd8d4a65

Andrii Shyshkalov authored Jun 13, 2018

R=iannucci@chromium.org

Recipe-Nontrivial-Roll: skia
Recipe-Nontrivial-Roll: build_limited_scripts_slave
Recipe-Nontrivial-Roll: release_scripts
Recipe-Nontrivial-Roll: infra
Recipe-Nontrivial-Roll: build
Change-Id: Ie0b042389b7e2c505cd8dbecc1a30a37244ae238
Reviewed-on: https://chromium-review.googlesource.com/1100110Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

cd8d4a65

Roll recipe dependencies (trivial). · 41ff7a03

recipe-roller authored Jun 13, 2018

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/7b5d8f513ee8ba48625d806c5e23d7a4a4b4f44c Update the recipes user guide. (gbeaty@chromium.org)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: Ic5a243651225d6f4ca2bc1fb39addca2f4f29428
Reviewed-on: https://chromium-review.googlesource.com/1099635Reviewed-by: Recipe Roller <recipe-roller@chromium.org>
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>

41ff7a03

Roll recipe dependencies (trivial). · 31d53948

recipe-roller authored Jun 13, 2018

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/c6807980ba3b1770d3c5ad595f21d512e8d6a6de [buildbucket] add test_api to simulate LUCI ci and try builds. (tandrii@chromium.org)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: Ia5d1f8eb85b465bac11461fedb81f2f78e7d33c3
Reviewed-on: https://chromium-review.googlesource.com/1099428Reviewed-by: Recipe Roller <recipe-roller@chromium.org>
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>

31d53948

12 Jun, 2018 1 commit

Roll recipe dependencies (trivial). · e09b6845

recipe-roller authored Jun 12, 2018

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/8565c92fbfcc717ce5d8bab4489a1d202b85fee0 [raw_io] add support to remove read-only files (maruel@chromium.org)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I4ad9167725eef3421409382ad8d255198b9d4552
Reviewed-on: https://chromium-review.googlesource.com/1096600Reviewed-by: Recipe Roller <recipe-roller@chromium.org>
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>

e09b6845

11 Jun, 2018 6 commits

gclient: Use only recursedeps to decide whether to process a dependency (Trigger recipe roller) · b63c4663

Edward Lemur authored Jun 11, 2018

Tbr: agable@chromium.org
Bug: 839925
Change-Id: Ie1dab7cfaa831af65fd9f8d2962161a729fb7a36
Reviewed-on: https://chromium-review.googlesource.com/1096121Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

b63c4663

gclient: Use only recursedeps to decide whether to process a dependency. · fbb06aaa

Edward Lemur authored Jun 11, 2018

Bug: 839925
Change-Id: If86cfbdbea795b6da53dec1ea6e44c5af8a3b846
Reviewed-on: https://chromium-review.googlesource.com/1086270Reviewed-by: Aaron Gable <agable@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

fbb06aaa

Improve post_build_ninja_summary.py's incremental build handling · 6be8afd5

Bruce Dawson authored Jun 11, 2018

post_build_ninja_summary.py detects a new incremental build by looking
for the end time stamps to go backwards. This means that repeated builds
that have a single long step will not be reliably detected and will, in
some cases, be missed entirely. This hits me sometimes when doing link
testing - delete an output DLL, rebuild, and the old results may be
displayed again.

This change updates the script to check for a different duration for an
existing record as well as an earlier end time should handle almost all
cases.

This also renames the targets local variable to avoid confusion due
to the targets member variable of the Targets class.

Bug: chromium:787983
Change-Id: I60a371df75d6cb7a55bd46b38156cb109feb8f15
Reviewed-on: https://chromium-review.googlesource.com/1061413
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>

6be8afd5

Roll recipe dependencies (trivial). · 9f107316

recipe-roller authored Jun 11, 2018

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/11db7d01eaa21d7706966a532ee36bddc9000e9a recipe_engine: Remove Gitiles backend from fetch. (ehmaldonado@chromium.org)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: Id160150697f6d1be129605c21ce1b22e8408510a
Reviewed-on: https://chromium-review.googlesource.com/1095835Reviewed-by: Recipe Roller <recipe-roller@chromium.org>
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>

9f107316

Add --log-file arg to post_build_ninja_summary.py · a10370c4

Daniel Bratell authored Jun 11, 2018

Sometimes the log files are not named .ninja_log and then it's
good if you can give the script the actual name and path.

Change-Id: I59e9812c81a83ffac3ecdc8b29887e99bf71b60f
Reviewed-on: https://chromium-review.googlesource.com/1012112Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>

a10370c4

Roll recipe dependencies (trivial). · d21878d7

recipe-roller authored Jun 11, 2018

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/40699af88c76d65ef1b3d364de221611603e2dc6 [cipd] Fix the cipd ensure output handling (phosek@google.com)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: If6a19c0a13fafdc06b572a4dfb7d403a13a1baee
Reviewed-on: https://chromium-review.googlesource.com/1094705
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>
Reviewed-by: Recipe Roller <recipe-roller@chromium.org>

d21878d7

09 Jun, 2018 1 commit

Roll recipe dependencies (trivial). · c3e514ed

recipe-roller authored Jun 09, 2018

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/b5ea6a8309ecc6cde288f1f97667e495e1d3466d [buildbucket] Make gitiles_commit singular (nodir@google.com)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: Ie72c0068bbf88238b047453a0c63e3edd5584227
Reviewed-on: https://chromium-review.googlesource.com/1094039Reviewed-by: Recipe Roller <recipe-roller@chromium.org>
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>

c3e514ed

08 Jun, 2018 3 commits

Remove dbghelp.dll hack from packaging script · 3fb5aa7b

Bruce Dawson authored Jun 08, 2018

The script that packages the Visual Studio toolchain had special code to
handle grabbing dbghelp.dll from the VS install and copying it to the
Debuggers directories. This was necessary around VS 2017 RTM to handle
/debug:fastlink but it now causes failures due to mismatched DLLs
leading to a missing ordinal. The hack shouldn't be needed anymore
because we no longer depend on /debug:fastlink (lld!) and because the
Debuggers package is assumed to have been updated to handle VS 2017 by
now.

Bug: 846313
Change-Id: I2b5fd87aaa785ce24a0905d70e7e586ff4838b1c
Reviewed-on: https://chromium-review.googlesource.com/1086990Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>

3fb5aa7b

Reland "Reland "Expand variables in gclient flattened output."" · e05f18d4

Edward Lemur authored Jun 08, 2018

This is a reland of ff622244
should_process was set to None in the previous attempt, so CIPD dependencies were not processed.
This CL fixed that.

Original change's description:
> Reland "Expand variables in gclient flattened output."
>
> This is a reland of a32f98e6
>
> Original change's description:
> > Expand variables in gclient flattened output.
> >
> > Bug: 848990
> > Change-Id: I0ad7e4f965973edbc5a335bd30f9cbd7b04abff2
> > Reviewed-on: https://chromium-review.googlesource.com/1085996
> > Reviewed-by: Michael Moss <mmoss@chromium.org>
> > Reviewed-by: Aaron Gable <agable@chromium.org>
> > Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
>
> Tbr: agable@chromium.org
> Bug: 848990
> Change-Id: I7843544b79b2ab7e2046c187d13ea3eb65fc1b7d
> Reviewed-on: https://chromium-review.googlesource.com/1085975
> Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

Bug: 848990
Change-Id: Ic804be1b84bf8402a741a4189b60372075dfb6f3
Reviewed-on: https://chromium-review.googlesource.com/1087368Reviewed-by: Aaron Gable <agable@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

e05f18d4

Add an `--ignore-current` flag to `git cl owners` to start from scratch. · 8e6f58c7

Sidney San Martín authored Jun 08, 2018

Change-Id: Iee6d7cf87281d4ca0abf71a7109f678d5cbbc9f1
Reviewed-on: https://chromium-review.googlesource.com/1091801Reviewed-by: Aaron Gable <agable@chromium.org>
Commit-Queue: Sidney San Martín <sdy@chromium.org>

8e6f58c7

07 Jun, 2018 1 commit

gclient: Add a test to make sure we're syncing CIPD dependencies. · efe4f897

Edward Lemur authored Jun 07, 2018

Bug: 849374
Change-Id: I5654ed65843c6dae6b7dbee10a40aa35e16803dc
Reviewed-on: https://chromium-review.googlesource.com/1087390
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Aaron Gable <agable@chromium.org>

efe4f897

06 Jun, 2018 1 commit

Roll mac_toolchain to 796d2b92c · e89dcf7d

Sergey Berezin authored Jun 06, 2018

Relevant changes:
  796d2b92c [mac_toolchain] Always run xcodebuild commands in the correct Xcode context
  42953a5b3 [mac_toolchain] Bug fixes for Xcode 10 and DevToolsSecurity

TBR=maruel@chromium.org

Bug: 849728, 846485
Change-Id: Ifbe6af428d609f4530ba6dcb19c16ebab9432af5
Reviewed-on: https://chromium-review.googlesource.com/1088231Reviewed-by: Sergey Berezin <sergeyberezin@chromium.org>
Commit-Queue: Sergey Berezin <sergeyberezin@chromium.org>

e89dcf7d

05 Jun, 2018 4 commits

Revert "Reland "Expand variables in gclient flattened output."" · b9907703

John Budorick authored Jun 05, 2018

This reverts commit ff622244.

Reason for revert: spec revert for crbug.com/849374

Original change's description:
> Reland "Expand variables in gclient flattened output."
> 
> This is a reland of a32f98e6
> 
> Original change's description:
> > Expand variables in gclient flattened output.
> >
> > Bug: 848990
> > Change-Id: I0ad7e4f965973edbc5a335bd30f9cbd7b04abff2
> > Reviewed-on: https://chromium-review.googlesource.com/1085996
> > Reviewed-by: Michael Moss <mmoss@chromium.org>
> > Reviewed-by: Aaron Gable <agable@chromium.org>
> > Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
> 
> Tbr: agable@chromium.org
> Bug: 848990
> Change-Id: I7843544b79b2ab7e2046c187d13ea3eb65fc1b7d
> Reviewed-on: https://chromium-review.googlesource.com/1085975
> Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

TBR=agable@chromium.org,mmoss@chromium.org,ehmaldonado@chromium.org

Change-Id: I89ffb80893be72f60043e8a6e02f6c0fb6fe5cc1
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 848990
Reviewed-on: https://chromium-review.googlesource.com/1086747Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: John Budorick <jbudorick@chromium.org>

b9907703

Reland "Expand variables in gclient flattened output." · ff622244

Edward Lemur authored Jun 05, 2018

This is a reland of a32f98e6

Original change's description:
> Expand variables in gclient flattened output.
>
> Bug: 848990
> Change-Id: I0ad7e4f965973edbc5a335bd30f9cbd7b04abff2
> Reviewed-on: https://chromium-review.googlesource.com/1085996
> Reviewed-by: Michael Moss <mmoss@chromium.org>
> Reviewed-by: Aaron Gable <agable@chromium.org>
> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

Tbr: agable@chromium.org
Bug: 848990
Change-Id: I7843544b79b2ab7e2046c187d13ea3eb65fc1b7d
Reviewed-on: https://chromium-review.googlesource.com/1085975Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

ff622244

Revert "Expand variables in gclient flattened output." · 01cb5105

Edward Lesmes authored Jun 05, 2018

This reverts commit a32f98e6.

Reason for revert:
Doesn't work when there are variables which definition includes other variables, e.g.:

'cros_download_vm': '"{cros_board}" == "amd64_generic"',

Original change's description:
> Expand variables in gclient flattened output.
> 
> Bug: 848990
> Change-Id: I0ad7e4f965973edbc5a335bd30f9cbd7b04abff2
> Reviewed-on: https://chromium-review.googlesource.com/1085996
> Reviewed-by: Michael Moss <mmoss@chromium.org>
> Reviewed-by: Aaron Gable <agable@chromium.org>
> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

TBR=agable@chromium.org,mmoss@chromium.org,ehmaldonado@chromium.org

Change-Id: I7e81000e92aa352e45b420845bb1dcc8572f1962
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 848990
Reviewed-on: https://chromium-review.googlesource.com/1085974Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

01cb5105

Expand variables in gclient flattened output. · a32f98e6

Edward Lemur authored Jun 04, 2018

Bug: 848990
Change-Id: I0ad7e4f965973edbc5a335bd30f9cbd7b04abff2
Reviewed-on: https://chromium-review.googlesource.com/1085996Reviewed-by: Michael Moss <mmoss@chromium.org>
Reviewed-by: Aaron Gable <agable@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

a32f98e6

04 Jun, 2018 3 commits

[git-cl-try] Handle absence of builds · 23d75d2d

Nodir Turakulov authored Jun 04, 2018

If there are no builds in build search response, the response is {}.
Handle this case.

R=qyearsley

Bug: 849361
Change-Id: I9f0472e140698a1d1c5b98b301bc562ac457785b
Reviewed-on: https://chromium-review.googlesource.com/1085899Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Commit-Queue: Nodir Turakulov <nodir@chromium.org>

23d75d2d

Fix buildbucket bug filing instructions · 9ac59795

Nodir Turakulov authored Jun 04, 2018

Use Infra>Platform>Buildbucket component, not Infra-BuildBucket label.

R=qyearsley

Bug: 849361
Change-Id: I27706d93097a20c6927ffa5405550bb63aa519dd
Reviewed-on: https://chromium-review.googlesource.com/1085898Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Commit-Queue: Nodir Turakulov <nodir@chromium.org>

9ac59795

Include "fuchsia" as a known "deps_os" value · 518bb684

Sergiy Byelozyorov authored Jun 03, 2018

This fixes an issue where "--deps=all" syncs everything but fuchsia.

R=machenbach@chromium.org, tandrii@chromium.org, wez@chromium.org

Bug: 848676
Change-Id: I782184b615968140d0e9a0146169fa12a9ebe0c5
Reviewed-on: https://chromium-review.googlesource.com/1084211Reviewed-by: Wez <wez@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org>

518bb684

01 Jun, 2018 2 commits

[led] Roll to 7ba61969ccbe1f9679661555f72ca947da508a70 · 88087bad

smut authored Jun 01, 2018

Relevant changes:
7ba61969c Update Swarming clients to hit /_ah/api instead of /api
4d67b77ab [led] Allow `led get-swarm` to read tasks with isolated recipes.

Bug: 805543
Change-Id: Iffad3818591a61572d347af95b7072e0811619af
Reviewed-on: https://chromium-review.googlesource.com/1083619Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Commit-Queue: smut <smut@google.com>

88087bad

Add ability to whitelist bot accounts as valid authors · 621fe6f9

Sergiy Byelozyorov authored May 26, 2018

Example usage: https://crrev.com/c/1074367

R=machenbach@chromium.org

Bug: 831171
Change-Id: Ic3deff5040a268e10161ccce635f06b7f8c08d36
Reviewed-on: https://chromium-review.googlesource.com/1074368Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org>

621fe6f9

31 May, 2018 2 commits

[presubmit] Add python formatting check · 0ae14e9a

Nodir Turakulov authored May 31, 2018

Add check_python parameter to CheckPatchFormatted to check that python
files are well formatted.

Change-Id: Ibd20d5ec9d1d5142a26f7f4cd5f05c51223da6ca
Reviewed-on: https://chromium-review.googlesource.com/1081342
Commit-Queue: Nodir Turakulov <nodir@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

0ae14e9a

Fix git cl format --python --full · af43f40e

Nodir Turakulov authored May 31, 2018

Make `git cl format --python -full --dry-run` return non-zero exit code if
files are not well formatted.

Make `git cl format --python -full --diff` print diff, rather than new file
contents.

Change-Id: I666d360051563898d3508d776817bf9a61433cb0
Reviewed-on: https://chromium-review.googlesource.com/1081340Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Nodir Turakulov <nodir@chromium.org>

af43f40e

29 May, 2018 1 commit

Update windows git to 2.17.1 · cf4aced3

Aaron Gable authored May 29, 2018

This is for security updates related to
CVE 2018-11235
CVE 2018-11234
https://public-inbox.org/git/xmqqy3g2flb6.fsf@gitster-ct.c.googlers.com/

Bug: 847597
Change-Id: I3b2d6b7bf9fe2d93bdc375903357f07f4fff8573
Reviewed-on: https://chromium-review.googlesource.com/1077307Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Aaron Gable <agable@chromium.org>

cf4aced3

25 May, 2018 2 commits

Roll recipe dependencies (trivial). · f16fdf31

recipe-roller authored May 25, 2018

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/10f67fe223a7973a95bc5fff219d1a7a3d95a326 [recipe_modules] Create Tricium module (juliehockett@google.com)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I8037b9c0bed59dc082de43fc3824ce6bb6842a1c
Reviewed-on: https://chromium-review.googlesource.com/1073197Reviewed-by: Recipe Roller <recipe-roller@chromium.org>
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>

f16fdf31

Roll recipe dependencies (trivial). · 4d2e8bff

recipe-roller authored May 25, 2018

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/c51413ab9d4c5ed5c3db8117b80afe64e54016b4 [recipe_api] Add content type for source manifests (mknyszek@google.com)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I019b19c164ccf0507d6864d08fa6ed5887f34930
Reviewed-on: https://chromium-review.googlesource.com/1072326Reviewed-by: Recipe Roller <recipe-roller@chromium.org>
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>

4d2e8bff

24 May, 2018 5 commits

Remove cq_client from depot_tools · af2ffd93

Aaron Gable authored May 24, 2018

This library is no longer used by anything in depot_tools.

Bug: 770408
Change-Id: I8f0a99f73cf9c52731994c830085678ca652d2c4
Reviewed-on: https://chromium-review.googlesource.com/1072491Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Aaron Gable <agable@chromium.org>

af2ffd93

Roll recipe dependencies (trivial). · b1a5e452

recipe-roller authored May 24, 2018

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/611cfbdc328b50144e2790ed1e4b9db24adb9eea [raw_io] Make it succeed even with read only files. (maruel@chromium.org)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I706661a48211e87e395d5cf148a4a02ba3ead613
Reviewed-on: https://chromium-review.googlesource.com/1072549Reviewed-by: Recipe Roller <recipe-roller@chromium.org>
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>

b1a5e452

Remove Rietveld-only apply_issue · b09e03ec

Aaron Gable authored May 23, 2018

This script only works (worked) for Rietveld issues, which
effectively no longer exist. It serves no purpose anymore.

Bug: 770408
Change-Id: Ic93ab3ce90d9c35b3bf5189e2beb4aa0c3983d44
Reviewed-on: https://chromium-review.googlesource.com/1070859Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Aaron Gable <agable@chromium.org>

b09e03ec

Remove more Rietveld support from presubmit · 4b23a2c6

Aaron Gable authored May 24, 2018

This simplifies future removals of unnecessary code from git-cl.

Bug: 770408
Change-Id: I20329fb6d93b349bf2e9f4aebc795b3f7ef0b646
Reviewed-on: https://chromium-review.googlesource.com/1070588
Commit-Queue: Aaron Gable <agable@chromium.org>
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>

4b23a2c6

Roll mac_toolchain to 478dc9dbb · 0115c356

Justin Cohen authored May 24, 2018

Relevant changes:
  [mac_toolchain] Always call runFirstLaunch for Xcode 9+.

Bug: 841910
TBR=iannucci@chromium.org
Change-Id: I3f4b8fb793e66315492f8e53832e0ac9d00ceb0c
Reviewed-on: https://chromium-review.googlesource.com/1072007
Commit-Queue: Justin Cohen <justincohen@chromium.org>
Reviewed-by: Sergey Berezin <sergeyberezin@chromium.org>

0115c356