All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Glass <sjg@chromium.org>
To: U-Boot Mailing List <u-boot@lists.denx.de>
Cc: Tom Rini <trini@konsulko.com>,
	Rasmus Villemoes <rasmus.villemoes@prevas.dk>,
	Simon Glass <sjg@chromium.org>
Subject: [PATCH v4 08/13] buildman: Add a flag to control the traceback
Date: Sun, 16 Jan 2022 13:19:32 -0700	[thread overview]
Message-ID: <20220116131907.v4.8.I636748c91d1fdc7d2e39a3d636c03751004427d3@changeid> (raw)
In-Reply-To: <20220116201937.1387697-1-sjg@chromium.org>

At present the full horror of the Python traceback is shown by default. It
is normally only useful for debugging. Turn it off by default and add a
--debug flag to enable it.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

(no changes since v1)

 tools/buildman/cmdline.py | 2 ++
 tools/buildman/main.py    | 3 +++
 2 files changed, 5 insertions(+)

diff --git a/tools/buildman/cmdline.py b/tools/buildman/cmdline.py
index 274b5ac3f45..092a82f4e9c 100644
--- a/tools/buildman/cmdline.py
+++ b/tools/buildman/cmdline.py
@@ -32,6 +32,8 @@ def ParseArgs():
           help='Show detailed size delta for each board in the -S summary')
     parser.add_option('-D', '--config-only', action='store_true', default=False,
           help="Don't build, just configure each commit")
+    parser.add_option('--debug', action='store_true',
+        help='Enabling debugging (provides a full traceback on error)')
     parser.add_option('-e', '--show_errors', action='store_true',
           default=False, help='Show errors and warnings')
     parser.add_option('-E', '--warnings-as-errors', action='store_true',
diff --git a/tools/buildman/main.py b/tools/buildman/main.py
index 2b714739a20..04698ce9e50 100755
--- a/tools/buildman/main.py
+++ b/tools/buildman/main.py
@@ -54,6 +54,9 @@ def RunTests(skip_net_tests):
 
 options, args = cmdline.ParseArgs()
 
+if not options.debug:
+    sys.tracebacklimit = 0
+
 # Run our meagre tests
 if options.test:
     RunTests(options.skip_net_tests)
-- 
2.34.1.703.g22d0c6ccf7-goog


  parent reply	other threads:[~2022-01-16 20:21 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-16 20:19 [PATCH v4 00/13] kconfig: Add some new macros Simon Glass
2022-01-16 20:19 ` [PATCH v4 01/13] imx: Don't define __ASSEMBLY__ in C files Simon Glass
2022-01-17 13:53   ` Tom Rini
2022-01-19 14:37     ` Simon Glass
2022-01-16 20:19 ` [PATCH v4 02/13] mips: Avoid using config_enabled() directly Simon Glass
2022-01-18 19:47   ` Daniel Schwierzeck
2022-01-16 20:19 ` [PATCH v4 03/13] mmc: fsl: Use brackets around if() Simon Glass
2022-01-16 21:06   ` Sean Anderson
2022-01-16 23:50   ` Jaehoon Chung
2022-01-17 14:26     ` Minkyu Kang
2022-01-16 20:19 ` [PATCH v4 04/13] kconfig: Update IS_ENABLED() internals Simon Glass
2022-01-16 20:19 ` [PATCH v4 05/13] kconfig: Add support for conditional values Simon Glass
2022-01-16 20:19 ` [PATCH v4 06/13] bloblist: Update to use conditional value Simon Glass
2022-01-16 20:19 ` [PATCH v4 07/13] patman: Update test_util to run doc tests Simon Glass
2022-01-16 20:19 ` Simon Glass [this message]
2022-01-16 20:19 ` [PATCH v4 09/13] buildman: Make use of test_util Simon Glass
2022-01-16 20:19 ` [PATCH v4 10/13] buildman: Add helper functions for updating .config files Simon Glass
2022-01-16 20:19 ` [PATCH v4 11/13] buildman: Provide a hint on how to debug thread crashes Simon Glass
2022-01-16 20:19 ` [PATCH v4 12/13] buildman: Allow adjusting board config on the fly Simon Glass
2022-01-16 20:19 ` [PATCH v4 13/13] test: Add some tests for kconfig.h Simon Glass

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220116131907.v4.8.I636748c91d1fdc7d2e39a3d636c03751004427d3@changeid \
    --to=sjg@chromium.org \
    --cc=rasmus.villemoes@prevas.dk \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.