The script can't find the `find_depot_tools` module if this repo is
checked out under chromium/src.
Bug: b:364177019
Change-Id: I75997a41e1a9732d0e9f6567b00c86784c7b4ce9
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/361680
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Henrik Andreassson <henrika@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#42954}
RollError: WebRTC DEPS entries are missing from Chromium: ['src/third_party/android_build_tools/aapt2', 'src/third_party/hamcrest', 'src/third_party/icu4j', 'src/third_party/kotlin_stdlib'].
Remove them or add them to either WEBRTC_ONLY_DEPS or DONT_AUTOROLL_THESE.
Bug: None
Change-Id: I69b4912e13cb7b84ffd822856e2273594a52717c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/360420
Commit-Queue: Jeremy Leconte <jleconte@webrtc.org>
Auto-Submit: Christoffer Dewerin <jansson@google.com>
Reviewed-by: Jeremy Leconte <jleconte@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#42844}
If the webrtc/src repo is checked out in a directory that isn't named
"src", FindSrcDirPath will loop forever. Instead of trying to find the
repo root, just use the location of the scripts and work out the
root with os.pardir.
Bug: b/333744051
Change-Id: Ifccdb85d3f9c7cb27ca57cc0b7bb96adf783660d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/357980
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Gavin Mak <gavinmak@google.com>
Cr-Commit-Position: refs/heads/main@{#42701}
While GCS dependencies aren't currently used, their support is required
to prevent the autoroller from breaking when encountering GCS dep types.
Bug: None
Change-Id: I58601e9eaeb8372058da4d4ee02cd2ca589e02c7
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/349740
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#42222}
Chromium is being updated to 'android_toolchain', which means the
'android_ndk' DEPS is no longer present. Remove it from the roller until
the transition is complete, then it can be removed from this script
entirely.
Bug: chromium:1448383
Test: Verified fix in libyuv, which uses identical roller script.
Change-Id: I0b07aefaa8edf0a8f87b2accfbcffd4a2307f9f4
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/308160
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Auto-Submit: Prashanth Swaminathan <prashanthsw@google.com>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#40243}
* The 'deps_os' list processed 'unix' twice. The former is likely meant
to process 'linux' instead.
* The roller skips dependencies that do not an '@' to split on. However,
it is valid to specify a DEPS that has no specific revision. In this
case, gclient assumes to pull HEAD.
Bug: webrtc:15215
Change-Id: Idf2f51ebd55f00a96b7b7f4a1935cd69881cabd8
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/307240
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Prashanth Swaminathan <prashanthsw@google.com>
Cr-Commit-Position: refs/heads/main@{#40189}
In the previous commit, I changed to modify deps_content,
but it was no-op since the content was already written to the DEPS file.
Bug: None
Change-Id: I278fbbb628422a42e616708f00529e935d75cd1f
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/292660
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Christoffer Jansson <jansson@google.com>
Commit-Queue: Daniel.L (Byoungchan) Lee <daniel.l@hpcnt.com>
Cr-Commit-Position: refs/heads/main@{#39268}
By making this change, we ensure that these variables are not outdated.
Also, remove unnecessary list calls to python generators.
Bug: None
Change-Id: I53babe03da1cb78cf5dc127b7e1f753b63be20de
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/292620
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Daniel.L (Byoungchan) Lee <daniel.l@hpcnt.com>
Cr-Commit-Position: refs/heads/main@{#39267}
I went through more thoroghly this time I think I gotem all
Bug: webrtc:13607
Change-Id: I0e8773903bff16e807cc32a1a6f0177d43543e44
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/252062
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Christoffer Jansson <jansson@google.com>
Cr-Commit-Position: refs/heads/main@{#36050}
After recently changing .pylintrc (see [1]) we discovered that
the presubmit check always checks all the python files when just
one python file gets updated.
This CL moves all these files one step closer to what the linter
wants.
Autogenerated with:
# Added all the files under pylint control to ~/Desktop/to-reformat
cat ~/Desktop/to-reformat | xargs sed -i '1i\\'
git cl format --python --full
This is part 1 out of 2. The second part will fix function names and
will not be automated.
[1] - https://webrtc-review.googlesource.com/c/src/+/186664
No-Presubmit: True
Bug: webrtc:12114
Change-Id: Idfec4d759f209a2090440d0af2413a1ddc01b841
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/190980
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#32530}
This is a reland of 8df59bc74ebb4d0b8e5d80a8b4f5c674db2a9c4d
Relanding as is for testing.
Original change's description:
> Test luci.notifier.
>
> This CL will be reverted as soon as we verify that the fix applied in
> https://webrtc-review.googlesource.com/c/src/+/179062 works.
>
> Bug: None
> Change-Id: I1ef48f8e69d5592154b154de1700858aea4c32c4
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/179063
> Reviewed-by: Jeremy Leconte <jleconte@webrtc.org>
> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#31703}
TBR=jleconte@webrtc.org
No-Try: True
Bug: None
Change-Id: Ib83eeb57d41f20cdfd54271f59d22f932e3b96b3
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/179086
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31707}
This reverts commit 8df59bc74ebb4d0b8e5d80a8b4f5c674db2a9c4d.
Reason for revert: Let's turn the bot green and the try to turn it
read again since the notification comes only on status change.
Original change's description:
> Test luci.notifier.
>
> This CL will be reverted as soon as we verify that the fix applied in
> https://webrtc-review.googlesource.com/c/src/+/179062 works.
>
> Bug: None
> Change-Id: I1ef48f8e69d5592154b154de1700858aea4c32c4
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/179063
> Reviewed-by: Jeremy Leconte <jleconte@webrtc.org>
> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#31703}
TBR=mbonadei@webrtc.org,jleconte@webrtc.org
Change-Id: I91e6debfbcb18bc840ca02a91ff736724ea9bbd1
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: None
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/179085
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31706}
This is a reland of bfb6124a73af8c9fb806d9798c21aa28d36b125f
Reland with no changes, the revert was just a test.
Original change's description:
> Add possibility to expand strings in the autoroller.
>
> Starting from crrev.com/c/2265498, the Chromium DEPS file has a new
> function Str(), which seems to just expand to the string it wraps.
>
> This causes the following error:
>
> NameError: name 'Str' is not defined
>
> In //tools_webrtc/autoroller/roll_deps.py.
>
> This CL adds a way to expand the string.
>
> No-Try: True
> Bug: None
> Change-Id: I4cdb43410edeed72b393f200314c0ee7eea9cb2a
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/178865
> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> Reviewed-by: Jeremy Leconte <jleconte@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#31661}
TBR=jleconte@webrtc.org
No-Try: True
Bug: None
Change-Id: I616f7ef2ef68f9a9cb6c5e2ca332d860a750b239
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/178900
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31677}
This reverts commit bfb6124a73af8c9fb806d9798c21aa28d36b125f.
Reason for revert: Testing Autoroller notifications. I will reland this later.
Original change's description:
> Add possibility to expand strings in the autoroller.
>
> Starting from crrev.com/c/2265498, the Chromium DEPS file has a new
> function Str(), which seems to just expand to the string it wraps.
>
> This causes the following error:
>
> NameError: name 'Str' is not defined
>
> In //tools_webrtc/autoroller/roll_deps.py.
>
> This CL adds a way to expand the string.
>
> No-Try: True
> Bug: None
> Change-Id: I4cdb43410edeed72b393f200314c0ee7eea9cb2a
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/178865
> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> Reviewed-by: Jeremy Leconte <jleconte@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#31661}
TBR=mbonadei@webrtc.org,jleconte@webrtc.org
Change-Id: I69f75da132d6b838b96a559712ad4c0da929021c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: None
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/178877
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31673}
Starting from crrev.com/c/2265498, the Chromium DEPS file has a new
function Str(), which seems to just expand to the string it wraps.
This causes the following error:
NameError: name 'Str' is not defined
In //tools_webrtc/autoroller/roll_deps.py.
This CL adds a way to expand the string.
No-Try: True
Bug: None
Change-Id: I4cdb43410edeed72b393f200314c0ee7eea9cb2a
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/178865
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Jeremy Leconte <jleconte@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31661}
