Age | Commit message (Collapse) | Author |
|
This CL makes new skip_denedencies arg in get_upstream patch default to
False so that the new behavior introduced in CL:3255665 is disabled by
default. This is intended to fix a failure in the nightly revert checker
that was preventing cherry_picks.
BUG=b:206461485
TEST=Local test of get_upstream_patch.py, and
nightly_revert_checker_test.py
Change-Id: Ia7586b69f9aefb5ca820a60eed593d45a4b9a5bb
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3282951
Reviewed-by: Christopher Di Bella <cjdb@google.com>
Commit-Queue: Christopher Di Bella <cjdb@google.com>
Tested-by: Christopher Di Bella <cjdb@google.com>
Auto-Submit: Ryan Beltran <ryanbeltran@chromium.org>
|
|
Current timeout 2 min may not be sufficiant for `crosfleet dut
lease`.
Many tests fail with the lock error while lease succeeds.
Increase the timeout to 8 minutes.
Remove backslashes and fix other minor lints to make upload happy.
BUG=b:205325997
TEST=run crosperf locally on veyron
Change-Id: Ie3b74a0036cce3daeb58379fc5362797b0e5db03
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3265564
Tested-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Phabricator has a way to list dependencies, which Arcanist will
pull in succession to make sure the patch is correct. This is
fine until some of the patches land in main: Arcanist then gets
confused and doesn't always correctly apply the patches.
This patch makes it possible to skip dependencies by adding a new flag
to our upstream patch getter.
The current design is only going to permit `--skip_dependencies` when
there's exactly one `--differential` patch, since `--skip_dependencies`
is a global option. If we're finding this to be a common nuisance,
then we can look at redesigning the feature to allow lists of
differentials with skipped dependencies.
```
./get_upstream_patch.py --differential D12345 --skip_dependencies
./get_upstream_patch.py --cl 12345 --differential D45678 --skip_dependencies
./get_upstream_patch.py --cl 12345 --skip-dependencies
./get_upstream_patch.py --differential D12345 \
--differential D45678 \
--skip_dependencies
```
BUG=b:204779256
TEST=Tested locally
Change-Id: I89032cba82c17c9f844bfac3cba330d173826ffd
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3255665
Commit-Queue: Christopher Di Bella <cjdb@google.com>
Tested-by: Christopher Di Bella <cjdb@google.com>
Auto-Submit: Christopher Di Bella <cjdb@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
Missed a single instance of this, as it's not a violation in
go/pi-terms, but is a violation in go/coil.
BUG=b:204057159
TEST=Presubmit tests
Change-Id: I4bc4997cb42c82c66e4d56281bb16e4452072e01
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3260693
Tested-by: Jordan R Abrahams <ajordanr@google.com>
Auto-Submit: Jordan R Abrahams <ajordanr@google.com>
Commit-Queue: George Burgess <gbiv@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
Update chromeos-kernel-4_4
Update chromeos-kernel-4_14
Update chromeos-kernel-4_19
Update chromeos-kernel-5_4
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: I59e1e016248a80227b6657236b72bc73168bfa6d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3260685
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>
|
|
DUTs have been moved to different rows and racks. This updates the
default_remotes accordingly.
These devices are chosen by checking the swarming dimension setting
"label-pool=toolchain".
Additionally, this removes Lulu machines as they are no longer in
the toolchain pool.
BUG=b:178744715
TEST=Presubmit hooks
Change-Id: Ifefb6cb6d2dae58d3c27b55f3a39bccf2fd27eb4
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3252241
Commit-Queue: Jordan R Abrahams <ajordanr@google.com>
Tested-by: Jordan R Abrahams <ajordanr@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
Wherever possible, this commit removes non-inclusive languages
as per COIL policy. Some of this language exists in dead code,
so this commit also removes confirmed dead code.
Also some minor lint fixes to get the code past the lint checks.
BUG=b:204057159
TEST=Presubmit tests
Change-Id: I299bfbe5dfcbe761efc5dcb29caea2337abe3f30
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3243940
Tested-by: Jordan R Abrahams <ajordanr@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
This CL applies fixes automatically generated by yapf during presubmit
hook as well as a few manual fixes:
* Removing erroneous `verbose` argument from call to RunTryJobs
* Replacing / with parenthsis for long lines
* Shortened some variable names to fix lines that were too long
BUG=None
TEST=rerean unit tests
Change-Id: Ic6ed4bb74f067dd5c67991481caf75d8829bf86e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3248930
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
Auto-Submit: Ryan Beltran <ryanbeltran@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
This CL adds dev-util/lldb-server to the list of packages updated by
various tools. Additionally, it updates the default packages in
update_chromeos_llvm_hash.py so that we don't have to manually supply
them anymore, and refactors other scripts to use that list instead of
maintaining their own.
BUG=b:196602751
TEST=`python3 -m unittest discover -s . -p '*_unittest.py'`
Change-Id: I0856c8f854dc2cbd0f2f21b71db96d3ef000caa8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3248929
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
Auto-Submit: Ryan Beltran <ryanbeltran@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
This CL adds -Wno-unused-but-set-variable to clang post flags to prevent
these warnings from blocking the LLVM uprev.
BUG=b:195433889
TEST=cq
Change-Id: I5195412183e2684aecd20b6ff2c3345f8b8b9dfb
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3235575
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Ryan Beltran <ryanbeltran@chromium.org>
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
|
|
This crosperf wrapper allows the user to lock devices by board
type. It will query the swarm using the crosfleet API to find
and lock an appropriate board.
The remaining arguments are passed to crosperf, so the command
will act like crosperf otherwise, once the remote has been set.
One limitation is that this wrapper does not conduct any SSH
port forwarding (which would be required for flashing
devices from a cloudtop, for example).
BUG=b:187535367
TEST=crosperf_autolock.py --board chell chell_exp.exp
Change-Id: If8dc4630c42be63d6427716654e7dd7fa4132df5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3221950
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Commit-Queue: Jordan R Abrahams <ajordanr@google.com>
Tested-by: Jordan R Abrahams <ajordanr@google.com>
|
|
Update chromeos-kernel-4_4
Update chromeos-kernel-4_14
Update chromeos-kernel-4_19
Update chromeos-kernel-5_4
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: Ia538fab15c409b67bfe21de04db3dc3a1e4ab544
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3228899
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Auto-Submit: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
Reviewed-by: Denis Nikitin <denik@chromium.org>
|
|
Currently, these crosperf files do not abide by the pep8 standard
that CrOS requires. This applies the necessary formatting to
get past the commit hook.
BUG=None
TEST=CQ (formatting)
Change-Id: I501fd37f8a1d20fd0b987587eb52eb582f18c1a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3215085
Tested-by: Jordan R Abrahams <ajordanr@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Jordan R Abrahams <ajordanr@google.com>
|
|
This global experiment setting prevents crosperf from
attempting to lock the associated devices. This is important
if the devices were locked previously before by crosfleet.
BUG=b:187535367
TEST=CQ
TEST=Lease board from crosfleet. Forward SSH port. Run crosperf on
forwarded port.
Change-Id: I0f5341e07e325a613c6316778c0f096f6d8088a2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3210525
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Jordan R Abrahams <ajordanr@google.com>
Commit-Queue: Jordan R Abrahams <ajordanr@google.com>
|
|
Update chromeos-kernel-4_4
Update chromeos-kernel-4_14
Update chromeos-kernel-4_19
Update chromeos-kernel-5_4
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: Ie0f2728c1a4800dbfe1f8f98ec98c73a6d8a33a6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3214700
Reviewed-by: Bob Haarman <inglorion@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
Tested-by: Denis Nikitin <denik@chromium.org>
|
|
This CL moves flags added for LLVM Next that disabled
variable_unused_but_set warnings to PostFlags in order to make sure it
gets added after Wall and Wextra.
BUG=b:195433889
TEST=reran package tests
Change-Id: Icdd2cbe3ced7b8427257c87f9e527b0d0e8da57d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3214640
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Ryan Beltran <ryanbeltran@chromium.org>
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
|
|
e0c2630c64 am: 929241978e am: f103b9f78b am: 137bac244d
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1845270
Change-Id: I673c3881544f5694c1380cd2d06b1d75902c76b3
|
|
e0c2630c64 am: 929241978e am: f103b9f78b
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1845270
Change-Id: I0976f8d1aee4219ad43237d0977631f9a8d283a5
|
|
e0c2630c64 am: 929241978e
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1845270
Change-Id: I497d318091cd0bb5742068f99bd7d3998962e1f0
|
|
e0c2630c64
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1845270
Change-Id: I8dce396f103bca021095fd47332d379fb57f76d1
|
|
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1845270
Change-Id: I251d68eec2ffe8b1b440486d7b200edf9f9cfa0f
|
|
Merged commit digest:
fb4c08bd compiler_wrapper: android clang-tidy warnings on used seconds
18220b91 rust_uprev: fetch distfiles from local mirror
6910c87f update_kernel_afdo: Skip 95 branch
511e10da afdo_metadata: Publish the new kernel profiles
58c28b56 afdo_metadata: Publish the new kernel profiles
e9810076 rust_uprev_test: mock equery w commands
391b1b03 afdo_metadata: Publish the new kernel profiles
4140da55 update_kernel_afdo: Remove 3.18
5e2dce90 Clean up cros-logging-import pylint attributes
Change-Id: Icfeef52053e103b622268543295563998b7e4bb1
|
|
Give a warning if clang-tidy used more than TIDY_TIMEOUT/2 seconds.
Users can fix it before it gets worse and times out.
BUG=b:199451930
TEST=Use a fake Android clang-tidy.real binary,
TEST=to dump passed environment variables and run for required time.
Change-Id: I8df7e4e3bb621cbf40094c7b97494e5978d7c6a2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3200220
Reviewed-by: Chih-Hung Hsieh <chh@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
|
|
This adds a few extra steps to rust_uprev where rust sources and
rust-bootstrap prebuilts are fetched from the local mirror. This
ensures that these files exist and are readable, and that the
local versions (that will be used to compute the digests for the
manifest) match the ones on the mirror.
BUG=None
TEST=Ran the script with existing and non-existing rust versions
Change-Id: I9868b027356cf14fc10d9e56665c74ad26555345
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3182632
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
|
|
Add a list of skipped chromium branches and adjust canary branch number
calculation.
BUG=None
TEST=`./update_kernel_afdo` shows 96 as canary.
Change-Id: If96711f5a0d52c128d4829e338b9b950001c6e75
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3199488
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
|
|
Update chromeos-kernel-4_4
Update chromeos-kernel-4_14
Update chromeos-kernel-4_19
Update chromeos-kernel-5_4
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: Ibb3548142490b8a8277bd09eb2b53365c43507e7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3200213
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
Tested-by: Denis Nikitin <denik@chromium.org>
|
|
Update chromeos-kernel-4_4
Update chromeos-kernel-4_14
Update chromeos-kernel-4_19
Update chromeos-kernel-5_4
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: I0990d136a0307c14242b95a0b7e7e158c7318d91
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3183335
Tested-by: Denis Nikitin <denik@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
|
|
rust_uprev uses `equery w` to find the ebuild file for a package.
This depends on filesystem state and so should not affect the test.
This change mocks these calls so that they return specified values
instead of actually running commands and accessing the filesystem.
BUG=None
TEST=Ran the tests outside the chroot, where equery does not exist
Change-Id: Id48a0e063f9bf112888d378d549420f008fdcb4f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3182629
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Michael Benfield <mbenfield@google.com>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
|
|
Update chromeos-kernel-3_18
Update chromeos-kernel-4_4
Update chromeos-kernel-4_14
Update chromeos-kernel-4_19
Update chromeos-kernel-5_4
BUG=None
TEST=Verified in kernel-release-afdo-verify-orchestrator
Change-Id: I2762c38bcc24070660de24a349f5829a7436a3b9
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3171121
Reviewed-by: Denis Nikitin <denik@chromium.org>
Commit-Queue: Ryan Beltran <ryanbeltran@chromium.org>
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
|
|
Remove 3.18 from the kernel AFDO update.
BUG=b:199787442
TEST=./update_kernel_afdo
Change-Id: I343d2048fb6bbd9c3c9f37bdfe64526a91c68220
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3162669
Tested-by: Denis Nikitin <denik@chromium.org>
Reviewed-by: Bob Haarman <inglorion@chromium.org>
Commit-Queue: Denis Nikitin <denik@chromium.org>
|
|
c2567cdcc2 am: a5563f87c4 am: 75d0ad1e93 am: 946524972d
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1831012
Change-Id: I8c7166ffac8be5e673d5d4393bcd27955739204d
|
|
c2567cdcc2 am: a5563f87c4 am: 75d0ad1e93
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1831012
Change-Id: Iaa96fe93bb3fab4b0f06de7c973b3551b9fe25f2
|
|
c2567cdcc2 am: a5563f87c4
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1831012
Change-Id: I4a050ccb3e7b70ecbd83dcb33d3a819f05a4dd56
|
|
c2567cdcc2
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1831012
Change-Id: Ifc868030e4614765ec7c0a14028800740acec650
|
|
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1831012
Change-Id: I334c9338d0a31a978d8a8f002d5518fed6e2c481
|
|
BUG=b:200542097
TEST=None
Change-Id: I82f4011d499b323a4eceda2949e103dc1559299f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3171413
Commit-Queue: Jaques Clapauch <jaquesc@google.com>
Commit-Queue: Chris McDonald <cjmcdonald@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Tested-by: Jaques Clapauch <jaquesc@google.com>
Auto-Submit: Jaques Clapauch <jaquesc@google.com>
Reviewed-by: Chris McDonald <cjmcdonald@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
Merged commit digest:
59bd789 Add mass_seccomp_editor.py for seccomp editing
018c7a7 run_tests_for: recognize `test_`/`unittest_` prefixes as tests
cfc2bb9 rust_uprev: changes for rust-1.55
8c582e5 compiler_wrapper: Handle TIDY_TIMEOUT for Android clang-tidy
297b3d6 compiler_wrapper: fix use-llvm-next test
76cc811 compiler_wrapper: update golden files
3ce773e Excise cros_logging module
db01057 compiler_wrapper: disable unused-but-set warnings
3bc3b1d OWNERS.toolchain: clean up
e87f528 users of mail-sheriff: s/sheriff/detective
338c889 email_sender: s/sheriff/detective
7d81944 compiler_wrapper: support -W[no-]unused-set-variable
7510727 Add upstream-workon.bash script
bf26752 llvm_tools: update llvm major in non-llvm ebuilds
af7b646 style.yapf: force indentation to 2
2adf813 llvm_tools: more informative message when checking out llvm-project-copy
046d57a compiler_wrapper: Update install script
50dc9b8 afdo_metadata: Publish the new kernel profiles
a57cfe5 llvm_tools: use upstream revert_checker
35c7e00 crosperf: reuse perf reports
a5ff058 compiler_wrapper: Fix static libgcc handling
a0a6360 llvm_tools: add newline at the end of PATCHES.json
5cd9c8d Edit style guide to match pep8
14fda11 git_llvm_rev: prefer upstream/* branches over non-upstream/ branches
05c8210 crosperf: remove idle samples from crosperf report
8fe5e65 crosperf: update location debug info
3796acf compiler_wrapper: Update for gcc -> llvm ebuild migration
e5d02c0 Revert "[llvm_tools] automatically uprevs sys-devel/gcc whenever LLVM_NEXT is changed"
ba388e1 Temporarily disable -Wdeprecated-copy and -Wdeprecated-declarations.
2a5ee3d afdo_metadata: Publish the new kernel profiles
a572519 [llvm_tools] automatically uprevs sys-devel/gcc whenever LLVM_NEXT is changed
bd4f0ce compiler_wrapper: bundle.py default output dir
52f7bb6 afdo_metadata: Publish the new kernel profiles
7ea3335 compiler_wrapper: add --rewrapper-path support
ed206c8 compiler_wrapper: refactor goma => "remote compile"
2ecec0c compiler_wrapper: refactor goma flag parsing
469f5d2 compiler_wrapper: make goma testing use its own function
be8490a compiler_wrapper: fix flaky tests
24accd9 afdo_metadata: Publish the new kernel profiles
b2757d0 [compiler_wrapper] explicitly enables `libgcc` as the unwindlib
502fb1d llvm-next: add nostart-stop-gc to the list of flags.
566da22 afdo_metadata: Publish the new kernel profiles
09e2dac llvm_tools: detect cros checkout for default chroot_path
a8142b1 llvm_tools: Fix lint warnings in llvm update tools
485c25a llvm_tools: llvm version detection for roll tools
011361f update_kernel_afdo: Check last week profiles
ffd95a1 toolchain-utils: Remove uses of "readelf --debug-dump=info"
51a2a9a Bisection: add better support for Chrome.
Test: make
Change-Id: I64eed452a885c9057f0df48ff87262d2d72598e2
|
|
This tool lets you work with large swaths of seccomp files quickly,
determining which have missing syscalls, and allowing fast editing
for all of them.
BUG=None
TEST=cd mass_seccomp_editor; python3 -m unittest
TEST=Ran many times for glibc noexec patch
Change-Id: I7dbad6eff431d41e23d31465a444b209520e953a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3101468
Tested-by: Jordan R Abrahams <ajordanr@google.com>
Auto-Submit: Jordan R Abrahams <ajordanr@google.com>
Commit-Queue: George Burgess <gbiv@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
Apparently some upstream conventions use `test_${x}.py` instead of this
repo's standard of `${x}_(test|unittest).py`. Either seems fine, and
supporting both seems simple.
This includes some formatting updates from yapf.
BUG=None
TEST=`./run_tests_for.py llvm_tools/*.py`
TEST=Made a new test_foo.py script; saw that it was picked up.
Change-Id: I533a9395227d8342411cacfbd1bb6d5c63b8ec2a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3163797
Reviewed-by: Jordan R Abrahams <ajordanr@google.com>
Commit-Queue: George Burgess <gbiv@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
Updating to rust-1.55.0 required a few changes to rust_uprev:
- patches for rust-bootstrap no longer need to be renamed and
removed.
- the version sequence is now RUST_RAW_FULL_BOOTSTRAP_SEQUENCE
BUG=None
TEST=created uprev for rust-1.55
Change-Id: I56a64d5fd7902e01ee2ca3c13dcfc3eee52d6fc6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3162670
Tested-by: Bob Haarman <inglorion@chromium.org>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
* To check in upstream chromeos tools.
* In Android mode, for clang-tidy, call
runAndroidClangTidy to handle TIDY_TIMEOUT
* Add runWithTimeout to env.go and runCmdWithTimeouit to command.go.
BUG=b:199451930
TEST=Use a fake Android clang-tidy.real binary,
TEST=to dump passed environment variables and run for required time.
Change-Id: I629ae372251c034595011ef70559f8b12ed8568c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3162668
Reviewed-by: Chih-Hung Hsieh <chh@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
|
|
The history of this test looks weird. We've had it intermittently
enabled/disabled depending on whether we're passing flags specifically
to llvm-next versions of clang. At the moment, it's failing due to a
lack of `-Wl,-z,nostart-stop-gc`, which was removed in a previous CL.
BUG=None
TEST=go test
Change-Id: I4b1ab9e5d39ba0e5575f89f51e4e83afba24a025
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3162667
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
Tests are failing at HEAD; looks like prior changes forgot to update
these.
BUG=None
TEST=go test
Change-Id: Ia88b182ba0a922807f0857d8dc3df96de4528aa7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3162666
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
The cros_logging module is deprecated and all usages are being removed.
The lint referred to by the comment removed in this CL is no longer a
thing, so remove this mention of the deprecated module from the tree.
BUG=b:191490453
TEST=None
Change-Id: Ifcd4cb18e87fdbd3a92747b0fdbac434d51fd055
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3161649
Tested-by: Chris McDonald <cjmcdonald@chromium.org>
Auto-Submit: Chris McDonald <cjmcdonald@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
This CL disables no-unused-but-set-variable warnings globally in the
compiler wrapper. A seperate CL will specifically reenable this warning
for platform2. This prevents breakage do to unised variables in
third_party.
BUG=None
TEST=None
Change-Id: I2ea44b6249715d3b036843b13d668d50b9b7669d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3150656
Tested-by: Ryan Beltran <ryanbeltran@chromium.org>
Auto-Submit: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
|
|
The CrOS toolchain team has seen changes over the last ~year and a half.
This CL updates this file to match the current toolchain team.
BUG=None
TEST=None
Change-Id: I65e08922bb91fd5634e30c1e717fe28e62c68ce6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3133551
Reviewed-by: Ryan Beltran <ryanbeltran@chromium.org>
Reviewed-by: Christopher Di Bella <cjdb@google.com>
Reviewed-by: Jordan R Abrahams <ajordanr@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Michael Benfield <mbenfield@google.com>
Commit-Queue: George Burgess <gbiv@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
Starting tonight, an alias for the mail-sheriff program will be
mail-detective. Swap to that.
This includes forced auto-style-fixing from yapf.
BUG=b:198446837
TEST=None
Change-Id: I8f7375347c17095a7d53408617f8e46078f3db54
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3137913
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
Now that our backend for this supports detective as an alternate
spelling of sheriff, we can migrate this.
This includes forced auto-style-fixing from yapf.
BUG=b:198446837
TEST=Unittests
Change-Id: Ib371911a006d40149dfbf91cc5daf834066a451c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3137202
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
|
|
BUG=b:194845241
TEST=emerge-atlas harfbuzz
Change-Id: I262883ee20ed0fc69fcc3bf119d3068f4cf7729b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3061363
Tested-by: Jian Cai <jiancai@google.com>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
upstream-workon is a script which aids toolchain developers by
keeping the local, fully patched source in some non-ephemeral
local directory inside the chroot.
At the moment, it's still in a prototype phase (the end goal
would be to make it quite a bit safer, and also move it out of
bash).
This is the first commit for the script, meant to act as a
base for future work.
BUG=b:186803200
TEST=mkdir ~/llvm; upstream_workon.bash init sys-devel/llvm ~/llvm
TEST=upstream_workon.bash build sys-devel/llvm
TEST=upstream_workon.bash install sys-devel/llvm
TEST=upstream_workon.bash link sys-devel/llvm ~/llvm_src_tree
Change-Id: Ifa1b1520d621cbe8b435d4ebb1ae71dc28548fcf
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/3068075
Commit-Queue: Jordan R Abrahams <ajordanr@google.com>
Tested-by: Jordan R Abrahams <ajordanr@google.com>
Reviewed-by: George Burgess <gbiv@chromium.org>
|