gclient sync reports success when steps have failed |
|||
Issue description1. Try to gclient sync in a checkout with android 2. Your gsutil credentials are not set up. Expected: A failure message at the end. Actual: "Success!" is reported. If you scroll up you can find the credential errors above. Example: ________ running '/usr/bin/python src/build/android/update_deps/update_third_party_deps.py download -b chromium-robolectric -l third_party/xstream' in '/usr/local/google/home/danakj/s/c' Failed to fetch file gs://chromium-robolectric/520d90f30f36a0d6ba2dc929d980831631ad6a92 for /usr/local/google/home/danakj/s/c/src/third_party/xstream/lib/xstream-1.4.8.jar. [Err: Your credentials are invalid. For more help, see "gsutil help creds", or re-run the gsutil config command (see "gsutil help config"). ]
,
Nov 3 2017
,
Nov 4 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f806d496435126ab581701f098eb92a293932469 commit f806d496435126ab581701f098eb92a293932469 Author: John Budorick <jbudorick@chromium.org> Date: Fri Nov 03 23:42:51 2017 [android] Propagate exit codes from update_third_party_deps. Bug: 766773 Change-Id: I4f1ea649f4b2904fb0c30793c840afc63d5ec177 Reviewed-on: https://chromium-review.googlesource.com/753129 Reviewed-by: Benjamin Pastene <bpastene@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#513960} [modify] https://crrev.com/f806d496435126ab581701f098eb92a293932469/build/android/update_deps/update_third_party_deps.py
,
Nov 6 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by aga...@chromium.org
, Nov 3 2017Status: Available (was: Untriaged)