diff options
author | Manoj Gupta <manojgupta@google.com> | 2019-08-06 14:23:42 -0700 |
---|---|---|
committer | Manoj Gupta <manojgupta@chromium.org> | 2019-08-06 21:36:22 +0000 |
commit | 52483232077360cf278af3c45c1977275fc72d83 (patch) | |
tree | 81a1a279b194ef0c5c26ba1e749765b16889eee2 /llvm_tools | |
parent | c369e299c1421cc5687bead8fb3b74888d817d55 (diff) | |
download | toolchain-utils-52483232077360cf278af3c45c1977275fc72d83.tar.gz |
llvm-tools: Remove assert when stderr is non-empty.
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>
Diffstat (limited to 'llvm_tools')
-rwxr-xr-x | llvm_tools/update_packages_and_run_tryjobs.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm_tools/update_packages_and_run_tryjobs.py b/llvm_tools/update_packages_and_run_tryjobs.py index 672aee04..6e6063bc 100755 --- a/llvm_tools/update_packages_and_run_tryjobs.py +++ b/llvm_tools/update_packages_and_run_tryjobs.py @@ -208,8 +208,8 @@ def RunTryJobs(cl_number, extra_change_lists, options, builders, chroot_path, print(err, file=sys.stderr) raise ValueError('Failed to submit tryjob.') - assert not err, 'Unexpected contents in stderr: %s' % err - + # stderr can be noisy e.g. warnings when entering chroot, so ignore it. + # e.g. cros_sdk:enter_chroot: Gclient cache dir "/tmp/git-cache" is not... tryjob_results.append(out.rstrip()) return tryjob_results |