upload_symbols: use new prolog field w/boolean helper
Let's use the new prolog field rather than printing our own message.
BUG=None
TEST=`upload_symbols foo.sym` still prompts nicely
Change-Id: I0d8ceeeef99fcfc276a170f029c185de5c6d7045
Reviewed-on: https://chromium-review.googlesource.com/185620
Reviewed-by: Yu-Ju Hong <yjhong@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
diff --git a/scripts/upload_symbols.py b/scripts/upload_symbols.py
index 18b34a6..1a0cc19 100644
--- a/scripts/upload_symbols.py
+++ b/scripts/upload_symbols.py
@@ -446,17 +446,16 @@
SymUpload = TestingSymUpload
if not opts.yes:
- query = textwrap.wrap(textwrap.dedent("""
+ prolog = '\n'.join(textwrap.wrap(textwrap.dedent("""
Uploading symbols for an entire Chromium OS build is really only
necessary for release builds and in a few cases for developers
to debug problems. It will take considerable time to run. For
developer debugging purposes, consider instead passing specific
files to upload.
- """), 80)
- cros_build_lib.Warning('\n%s', '\n'.join(query))
+ """), 80)).strip()
if not cros_build_lib.BooleanPrompt(
prompt='Are you sure you want to upload all build symbols',
- default=False):
+ default=False, prolog=prolog):
cros_build_lib.Die('better safe than sorry')
ret = 0