Age | Commit message (Collapse) | Author |
|
BUG=None
TEST=Scripts terminated when ran inside the chroot
Change-Id: Id7060ca6554a05f158160e73759763663dec1ce9
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1742544
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
We were querying github to get the tot git hash. However,
the ebuilds use the chromium mirror for llvm sources.
As the mirror can be behind github, it caused a version skew
where the sources could not be checked out to the desired tot hash.
BUG=None
TEST=update_packages script.
Change-Id: I6510213f8de2ea29b26f46741a5da7b4c81f59e0
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1747167
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
|
|
BUG=None
TEST=None
Change-Id: Ie112596f6297d5b60346e85bda45aa41284ecbf5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1740547
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
BUG=None
TEST='./update_chromeos_llvm_next_hash_unittest.py' and
'./llvm_patch_management_unittest.py' pass
Change-Id: I59bb1f8a1be5610c3a2c64b365392c07e95f97fb
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1739606
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
stderr often can have a non-empty value.
e.g. On my machine, I got:
WARNING cros_sdk:enter_chroot: Gclient cache dir "/tmp/git-cache" is not a directory.
Therefore remove the assert for empty stderr.
BUG=None
TEST=No asserts
Change-Id: I9f175aca9bbcee2acbb77bacfd8806de985d3447
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1738324
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
|
|
BUG=None
TEST=Ran the script and a CL was generated for ToT correctly and two
trybots were created, one for kevin and one for nocturne
Change-Id: I856c23d4368f5e45d9837e6f4341ae1f97f8f136
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1733650
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
|
|
BUG=None
TEST='./get_llvm_hash_unittest.py' passes
Change-Id: I7cbb34a9e7924104aaa5934ca2156f547f421450
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1731735
Tested-by: Salud Lemus <saludlemus@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
BUG=None
TEST='./update_chromeos_llvm_next_unittest.py' passes
Change-Id: I6701bfe2472e2bbec2d791f6f044d14bfe483195
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1732430
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
BUG=None
TEST=Ran the script and the correct g3 LLVM version was returned
Change-Id: I37847c56b1f72f3ceadc9e5fb7171faae273203c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1729508
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
BUG=None
TEST='./get_llvm_hash_unittest.py' passes
Change-Id: I6c5a58b82811f2d3abe8d377651f0036fa1bc491
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1724419
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
Updated and added unittests for the script now that the script calls a
script that setups up the arguments for the patch manager.
BUG=None
TEST='./update_chromeos_llvm_next_hash_unittest.py' passes
Change-Id: If16797725818d9f3c08c9e030e870e6bdbc3c83b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1719774
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
BUG=None
TEST='./llvm_patch_management_unittest.py' passes
Change-Id: I03cb727669039c9f89431ceec34c32d5d940777b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1717250
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
BUG=None
TEST='./patch_manager_unittest.py' passes
Change-Id: I66235de2153bb32160d7ce301a86a246cef967a3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1719213
Tested-by: Salud Lemus <saludlemus@google.com>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
|
|
The script 'llvm_patch_management.py' constructs the arguments for the
patch manager to handle patches of a package. The script 'patch_manager.py'
is the patch manager. The patch manager has different modes to handle
all applicable patches that fail to apply. The mode 'bisect_patches' is
a WIP.
BUG=None
TEST=Created a json file that has the patches of the LLVM ebuild and used
the default failure mode of the patch manager. All patches applied successfully
for r365631.
Change-Id: I42f339ac0bb7c35d70a96cc0c2daeb470336b13c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1708223
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
Some unittests were changed as well to affect this change.
BUG=None
TEST='./update_chromeos_llvm_next_unittest.py' passes
Change-Id: I179ae679eaac7f34c72c688e6ed15af79d8d6749
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1714124
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
In the original code, the chroot_path will be overwriten by stdout,
and if there are multiple entries in package_list, the second run
will fail because chromeos_root is not correct.
TEST=Tested with ./$SCRIPT --update_packages sys-devel/llvm
sys-libs/compiler-rt
BUG=None
Change-Id: I2a9b924ed44f648bc5a27e8633fcf1e9c43819d9
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1711160
Tested-by: Zhizhou Yang <zhizhouy@google.com>
Legacy-Commit-Queue: Commit Bot <commit-bot@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Auto-Submit: Zhizhou Yang <zhizhouy@google.com>
Commit-Queue: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
|
|
BUG=None
TEST='./update_chromeos_llvm_next_hash_unittest.py' passes
Change-Id: I1716861a81ef74cfa94d5894050fb1ecf7b0a727
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1697026
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
BUG=None
TEST=Ran the script and a CL was uploaded for review for 'sys-devel/llvm'
Change-Id: I827843f4841f18b586ae2dce39197291014cb559
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1691520
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
BUG=None
TEST='./get_llvm_hash_unittest.py' passes
Change-Id: I421e58dda744c6df6ae37a2fbc6559b98bcab2a8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1687777
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|
|
BUG=None
TEST=Ran the script and returned expected results
Change-Id: I360884c2dc4088a7875e88f2115460c5aa75ab27
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/1676593
Reviewed-by: George Burgess <gbiv@chromium.org>
Tested-by: Salud Lemus <saludlemus@google.com>
|