Age | Commit message (Collapse) | Author |
|
mainline-appsearch-release
Change-Id: I4d2b66fa4677777f6c8488a39dad98480a919fa7
|
|
8b320f7173 am: d444309511 am: 584b8e46d1
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/2292563
Change-Id: Ida212764c332af8ba8ab8aaa62a7d2a007e3314e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
8b320f7173 am: d444309511
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/2292563
Change-Id: I0c22edd3e962fb6c4b0cbd412a1eb884e00c77cb
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
8b320f7173
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/2292563
Change-Id: I79284c4b1560d3539f787127acf7c5b8f7dda5d5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
am: 9dc07341f9 am: d33de92dc9 am: c2cf8392f9
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/2290473
Change-Id: I44a3bba2d40809aa19dd70fa3991b5d9d3cc9b35
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
am: 9dc07341f9 am: d33de92dc9
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/2290473
Change-Id: I4e8a49928e1c511bc570af5068e6a6a8af16a575
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
am: 9dc07341f9
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/2290473
Change-Id: I948f5d71c87f36ffa0a53ee118e702e00dc20d20
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
This project was upgraded with external_updater.
Usage: tools/external_updater/updater.sh update toolchain-utils
For more info, check https://cs.android.com/android/platform/superproject/+/master:tools/external_updater/README.md
Test: TreeHugger
Change-Id: I60dd39f019e1287ea42fe48435ff3da44796c633
|
|
SDK builder artifacts have changed, so copying them directly to
localmirror doesn't work. Add a script that gives us a similar level of
convenience.
Due to the simplicity of this script, no tests are provided.
BUG=b:237786394
TEST=Ran it to upload rust-bootstrap-1.59.0 artifacts
Change-Id: I64ff8f6e9043487b3dd5abf04e5f7cca36852f0e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3764008
Commit-Queue: George Burgess <gbiv@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Michael Benfield <mbenfield@google.com>
|
|
Test: TreeHugger
Change-Id: Ib0cc543efb57df6b0c863fdc8e10ed4b5cbfd348
|
|
This reverts commit 537f2ecfe7bc9eb996cd4abedae26bf699ab54dd.
It also fixes the logic bug which caused the reert in the first place.
IWYU flag prcessing was removing the file names if IWYU was not needed,
which meant clang tidy calls were not getting the file names added to
the back of the command. It also fixes the header to the most recent
style guidance and updates some golang variable names to make sure we
pass go lint.
BUG=b:237320348
TEST=go test
Change-Id: I14885fb90f97d2fb483ae1a01e14c44050852dc3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3997486
Commit-Queue: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
|
|
See go/chromeos-zero-init
BUG=b:234642720
TEST=CQ
Change-Id: Icbcb9e9b3928b46875c78c99727b1ec9839f27c3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3999814
Reviewed-by: Adrian Dole <adriandole@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Adrian Dole <adriandole@google.com>
Auto-Submit: Adrian Dole <adriandole@google.com>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
|
|
This reverts commit 6b285afc5e9caadeb6741efe872981ffdbb3bfe1.
Reason for revert: Likely erroneous change.
Original change's description:
> compiler-wrapper: adds an IWYU component
>
> This first patch makes it possible to run IWYU as a part of the build
> process. It's not currently possible for us to make changes to packages:
> this functionality will appear in a later CL.
>
> BUG=b:237320348
> TEST=Tested locally
>
> Change-Id: I00610284143cf478b242b2c0ca1c05e2c8d43de4
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3820351
> Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
> Auto-Submit: Christopher Di Bella <cjdb@google.com>
> Commit-Queue: Ryan Beltran <ryanbeltran@chromium.org>
> Tested-by: Christopher Di Bella <cjdb@google.com>
Bug: b:237320348
Change-Id: Ia563382c3302b2a4c69942db95d315088763f97c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3995401
Auto-Submit: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Ryan Beltran <ryanbeltran@chromium.org>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
|
|
This first patch makes it possible to run IWYU as a part of the build
process. It's not currently possible for us to make changes to packages:
this functionality will appear in a later CL.
BUG=b:237320348
TEST=Tested locally
Change-Id: I00610284143cf478b242b2c0ca1c05e2c8d43de4
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3820351
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
Auto-Submit: Christopher Di Bella <cjdb@google.com>
Commit-Queue: Ryan Beltran <ryanbeltran@chromium.org>
Tested-by: Christopher Di Bella <cjdb@google.com>
|
|
Added options --upload (default) and --noupload.
BUG=None
TEST=./update_kernel_afdo
Change-Id: Ib720646a7fbec5b41205beecd84ffbeb31227a15
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3994210
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
Tested-by: Denis Nikitin <denik@chromium.org>
|
|
BUG=b:244337204
TEST=./update_kernel_afdo
Change-Id: I47722a6c3e4446bff816baed7711279dd923a9c3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3994209
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
Tested-by: Denis Nikitin <denik@chromium.org>
|
|
Update amd profile on chromeos-kernel-4.4
Update amd profile on chromeos-kernel-4.14
Update amd profile on chromeos-kernel-4.19
Update amd profile on chromeos-kernel-5.4
Update amd profile on chromeos-kernel-5.10
Update arm profile on chromeos-kernel-5.15
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: Ia47d80599bb635323851da768db4595d55dfceaf
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3994105
Tested-by: Denis Nikitin <denik@chromium.org>
Auto-Submit: Denis Nikitin <denik@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
|
|
This allows multiple benchmarks and profdata to be uploaded for the same
Rust version, distinguished by a filename suffix.
BUG=None
TEST=CQ
Change-Id: I226d26a8094e4b1e6a119ee8b6b002717d89026e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3988763
Auto-Submit: Michael Benfield <mbenfield@google.com>
Reviewed-by: George Burgess <gbiv@chromium.org>
Commit-Queue: Michael Benfield <mbenfield@google.com>
Tested-by: Michael Benfield <mbenfield@google.com>
Commit-Queue: George Burgess <gbiv@chromium.org>
|
|
Update chromeos-kernel-4.14
Update chromeos-kernel-4.19
Update chromeos-kernel-5.4
Update chromeos-kernel-5.10
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: Ia9af7b4fb69d6a04be6bc2d08cb6ded27c2f4391
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3972099
Tested-by: Bob Haarman <inglorion@chromium.org>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Reviewed-by: Denis Nikitin <denik@chromium.org>
|
|
BUG=b:217578492
TEST=./update_kernel_afdo
Change-Id: I67e35c0bd28936e968b533280d3e6d0d3149e412
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3965288
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
Auto-Submit: Denis Nikitin <denik@chromium.org>
Tested-by: Denis Nikitin <denik@chromium.org>
|
|
This fixes the autofix command so that we enforce a line length
of 80 chars.
BUG=None
TEST=Checked presubmit on violating file
Change-Id: If8817cdf9ab70d5d9e5db10c0818207f976f519c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3958135
Reviewed-by: George Burgess <gbiv@chromium.org>
Auto-Submit: Jordan Abrahams-Whitehead <ajordanr@google.com>
Tested-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Commit-Queue: Jordan Abrahams-Whitehead <ajordanr@google.com>
|
|
There were two bugs with patch_utils here:
1. When deciding to modify patches, it didn't ignore old patches.
2. When writing out changes, it wrote the dicts out in the wrong
order.
Both of these issues have been resolved.
BUG=b:253660089
TEST=./patch_utils_unittest.py
TEST=./patch_manager_unittest.py
TEST=./update_chromeos_llvm_hash_unittest.py
TEST=./update_chromeos_llvm_hash.py
TEST=./update_chromeos_llvm_hash.py \
--is_llvm_next \
--llvm_version tot \
--failure_mode disable_patches
Change-Id: If25c30ffa2bea55aeafb8d03aa19c5fc07fb4b7b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3956311
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Commit-Queue: Jordan Abrahams-Whitehead <ajordanr@google.com>
|
|
Support 'disable_patches' and 'remove_patches'
failure mode options.
BUG=b:250648178
TEST=./patch_utils_unittest.py
./update_chromeos_llvm_hash_unittest.py
./patch_manager_unittest.py
./update_chromeos_llvm_hash [...] --failure_mode remove_patches
./update_chromeos_llvm_hash [...] --failure_mode disable_patches
Change-Id: I6269b2220cf05413c7776087030297773ab9a154
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3935651
Reviewed-by: Adrian Dole <adriandole@google.com>
Auto-Submit: Adrian Dole <adriandole@google.com>
Tested-by: Adrian Dole <adriandole@google.com>
Reviewed-by: Denis Nikitin <denik@chromium.org>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Commit-Queue: Adrian Dole <adriandole@google.com>
|
|
Update chromeos-kernel-4.4
Update chromeos-kernel-4.14
Update chromeos-kernel-4.19
Update chromeos-kernel-5.4
Update chromeos-kernel-5.10
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: Ic97b86f9bd0f40ac76d954a0effb733359555e2c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3946883
Tested-by: Denis Nikitin <denik@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Auto-Submit: Denis Nikitin <denik@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
|
|
The arm profile is currently not used in the 5.15 kernel
so it's no-op.
Update to version to keep it up-to-date and test the PUpr pipeline.
BUG=b:244337204
TEST=emerge-trogdor chromeos-kernel-5_15 with the new verions
Change-Id: I9d0dca41758bc04f54d77a71c30bab8cbb42cb98
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3945805
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Auto-Submit: Denis Nikitin <denik@chromium.org>
Tested-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
|
|
The root of the ChromiumOS source tree is not always in the same
location relative to where the crosperf scripts are. This makes
the logic for finding sshwatcher.go not work everywhere. To fix
this, this change uses the label.chromeos_root path that is
present in crosperf runs to find sshwatcher.go.
BUG=b:235119397
TEST=run crosperf in standalone toolchain-utils as well as
inside ChromiumOS tree to see that sshwatcher.go is found
in both cases.
Change-Id: I981a55d3cfd2638da0bf320b86ebc5601101ce3f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3935652
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
|
|
Usually we don't need the manifest update with llvm-next
because pgo is not default there.
Remove default manifest packages from llvm-next.
Add unit test for main.
BUG=None
TEST=unit test
Change-Id: I1ef78be8184985e047db8ae68eda2f01c989a7a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3932864
Tested-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Some DUTs (notably those on gTransit) require port forwarding to be
accessible from inside the chroot. This CL makes crosperf's SuitRunner
use sshwatcher to set up the required forwarding.
BUG=b:235119397
TEST=suite_runner_unittest, then try it out for real
Change-Id: Idad2abf40713e2d0aa3b81fdb4762a22b638400e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3928354
Reviewed-by: Denis Nikitin <denik@chromium.org>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Tested-by: Bob Haarman <inglorion@chromium.org>
|
|
If `exec` times out, we need to wrap the error it died with, rather than
embedding it in another error. This allows
`TestRunWithTimeoutReturnsErrorOnTimeout`'s `errors.Is` to correctly
detect that `err` is a `context.DeadlineExceeded`.
BUG=None
TEST=ran this test 1000x
Change-Id: I69e0dcea25dfd0cb01ac5ec0a2e8ce2f116deff7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3932869
Auto-Submit: George Burgess <gbiv@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
|
|
Kefka reached EOL and was already removed from the toolchain pool.
BUG=b:232256537
TEST=CQ
Change-Id: Ic12b03177422c10319aa489b7f3baf177269d9ac
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3930952
Tested-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Jordan Abrahams-Whitehead <ajordanr@google.com>
Auto-Submit: Denis Nikitin <denik@chromium.org>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
|
|
crosperf/suit_runner.py uses .format(), which we no longer want.
This converts the code to use f-strings instead.
BUG=None
TEST=See that pylint no longer complains about .format()
Change-Id: I7085f2da5659843d2501eb2f19137b64dff7b106
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3928353
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Reviewed-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
|
|
Although rust_uprev emerges rust, this does not ensure that
all affected packages (rust, rust-bootstrap, rust-host) can be built
from the new sources. For example, it is possible to build and install
rust-host, then make changes to it, and re-running rust_uprev will then
not rebuild it, because it is already installed at the requested version.
This CL changes rust_uprev so that it first removes rust, rust-host, and
rust-bootstrap if they are installed, so that unbuildable packages will
be detected by rust_uprev.
BUG=b:232833366
TEST=Run with installed rust-host package that is unbuildable, see error
Change-Id: I7aeb7d1ac01e737e322bf005a31f1d678e52e042
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3653197
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
|
|
Previously, rust_uprev required that there be exactly one ebuild
file for a given rust{,-bootstrap,-uprev} version. This breaks
when using the common idiom of having a rust-1.2.3.ebuild and
a symlink to it with an -r version. This change adds support
for that by excluding such symlinks from the count, and removing
both the base files and the symlinks when removing ebuild versions.
BUG=b:227370760
TEST=Example uprev created by this script: crrev.com/c/3919575
Change-Id: I901ac8208c6bc48670860644ae91ee408e5f155d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3651375
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
dev-lang/rust is being forked into a new dev-lang/rust-host package.
As a result, Rust uprevs now need to update that package, too. This
adds the necessary support for that to rust_uprev.py.
BUG=b:227370760
TEST=Used the script to create a rust-1.61.0 to 1.62.1 uprev
Change-Id: I99733db7a799f1c234b628b035557ac429d9e470
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3590914
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Bob Haarman <inglorion@chromium.org>
|
|
Normally we don't do this, but enough changes have accumulated that
we're doing a tree-wide one-off update of the name & style.
BUG=chromium:1098010
TEST=`repo upload` works
Change-Id: Icb42e5012a87920c2cd13b666fb3e55e7e4fb3b8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3891080
Auto-Submit: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
BUG=b:244337204
TEST=cq
Change-Id: I3a0cac0145f7bdc4b63a791eca2f1ebbb413e3a2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3887561
Tested-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
Auto-Submit: Denis Nikitin <denik@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Fixing feedback from crrev.com/c/3877334, which isn't strictly related
to that change, but is still useful to clean up
BUG=None
TEST=`repo upload`
Change-Id: I1e183e174cc99dc9e309cc53fbde9f545eb8536f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3877525
Tested-by: George Burgess <gbiv@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
|
|
These are only useful when we're running code in a Python 2.7
interpreter. Since we no longer support python2, drop these.
BUG=b:244644217
TEST=run_tests_for.py shows no new failures
Change-Id: Ief9a12b87a560ab38ca71668636874bcb434a0b3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3877339
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
Since we're swapping to using `black`, this old YAPF file is no longer
needed (and instructs `yapf` to actively do the wrong thing with
indentation/etc)
BUG=b:244644217
TEST=None
Change-Id: I1bbf5be37c313b7fbbd841132570b3e5dd197ed5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3877338
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
|
|
This autoformats all Python code with our new Python formatter, `black`.
BUG=b:244644217
TEST=None
Change-Id: I15ee49233d98fb6295c0c53c129bbf8e78e0d9ff
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3877337
Tested-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Commit-Queue: George Burgess <gbiv@chromium.org>
|
|
Looks like some of our tests for older projects are failing by way of
running forever, weirdly. Those need to be addressed, but we should also
handle timing out tests more gracefully. Implement that.
BUG=b:244644217
TEST=Ran across all tests. Timeout WAI.
Change-Id: I7a8e2a809a64d2a07db52cfc59c4a9dc6e9e9e76
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3877336
Commit-Queue: George Burgess <gbiv@chromium.org>
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
|
|
This runs our new python formatter on `run_tests_for`, which is
necessary for a later commit prior to the mass-reformat.
BUG=b:244644217
TEST=Ran run_tests_for.py
Change-Id: I3ae240dfe5bdb4343d49ba8b954ab9fa731c052d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3877335
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Tested-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
|
|
This is redundant with the `check_black:` flag that we could add to our
`PRESUBMIT.cfg`, but `check_black` doesn't offer to autofix formatting.
BUG=b:244644217
TEST=Ran the presubmit on this python file. It failed. :)
Change-Id: Ie1d392e430ca64c3136eb2d6ab0ed62c550104dc
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3877334
Tested-by: George Burgess <gbiv@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
|
|
Update chromeos-kernel-4.4
Update chromeos-kernel-4.14
Update chromeos-kernel-4.19
Update chromeos-kernel-5.4
Update chromeos-kernel-5.10
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: I740f7221827c9697d4d0565bcbef9d747a7cbf7b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3877530
Commit-Queue: Denis Nikitin <denik@chromium.org>
Reviewed-by: Denis Nikitin <denik@chromium.org>
Auto-Submit: Jordan Abrahams-Whitehead <ajordanr@google.com>
Tested-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
|
|
Aquarium stories were recently disabled in telemetry rendering.desktop.
Remove the tests to unblock nightly tests.
BUG=b:244607231
TEST=cq
Change-Id: I12fd75d874228ef08949ee2e8ba27ee81dc48c1f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3869851
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
|
|
Move the variable with comments away from import. It causes a conflict
between yapf and lint/isort.
BUG=b:244644217
TEST=repo hooks pass
Change-Id: I50942a6355d30e3d39e9e60fc26e8eba687eba05
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3869850
Tested-by: Denis Nikitin <denik@chromium.org>
Auto-Submit: Denis Nikitin <denik@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
|
|
This CL add the Wno-unused-but-set-variable flag back into the compiler
wrapper. It was previously removed, but there are still occurances of
the warning and so we were never able to bundle that change into
chromoumos-overlay/llvm. An attempt to revert the original CL failed due
to a merge conflict.
BUG=b:227655984
TEST=None
Change-Id: Ic148a6ae577d5e4394249693ce9b09b95cb9df16
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3868978
Commit-Queue: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
BUG=b:243897875
TEST=./modify_a_tryjob_unittest.py &&
./update_packages_and_run_tests_unittest.py
Change-Id: I46bb7577134134e44e5f0571261031475ebdef7d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3867948
Auto-Submit: Adrian Dole <adriandole@google.com>
Tested-by: Adrian Dole <adriandole@google.com>
Commit-Queue: Adrian Dole <adriandole@google.com>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Reviewed-by: Adrian Dole <adriandole@google.com>
|
|
BUG=b:234635394
TEST=./update_chromeos_llvm_hash_unittest.py
Change-Id: Iff52984a44b41b230a74ebef7b5f8dd25b581237
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3852368
Tested-by: Adrian Dole <adriandole@google.com>
Commit-Queue: Adrian Dole <adriandole@google.com>
Reviewed-by: Jordan Abrahams-Whitehead <ajordanr@google.com>
Auto-Submit: Adrian Dole <adriandole@google.com>
Reviewed-by: Adrian Dole <adriandole@google.com>
|
|
BUG=b:235023022
TEST=./update_chromeos_llvm_hash_unittest.py
Change-Id: I4d015daeefff57087852c5aae0faad7d439d9267
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3834534
Tested-by: Adrian Dole <adriandole@google.com>
Commit-Queue: Adrian Dole <adriandole@google.com>
Reviewed-by: Adrian Dole <adriandole@google.com>
Auto-Submit: Adrian Dole <adriandole@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|