diff mbox series

[2/2] bitbake: main: add choices to restrict -S parameter

Message ID 20240917151157.780068-2-jstephan@baylibre.com
State New
Headers show
Series [1/2] bitbake: add documentation for "bitbake -S lockedsigs" | expand

Commit Message

Julien Stephan Sept. 17, 2024, 3:11 p.m. UTC 
Currently, available -S parameters are "none", "printdiff" and "lockedsigs",
but there is no check on parameter, so user can literally give anything
to -S and it will act as if "none" was specified.

Fix this by adding the allowed values in choices for -S parameter

Signed-off-by: Julien Stephan <jstephan@baylibre.com>
---
 bitbake/lib/bb/main.py | 1 +
 1 file changed, 1 insertion(+)
diff mbox series

Patch

diff --git a/bitbake/lib/bb/main.py b/bitbake/lib/bb/main.py
index 1f4313fa0dd..24758646eaa 100755
--- a/bitbake/lib/bb/main.py
+++ b/bitbake/lib/bb/main.py
@@ -213,6 +213,7 @@  def create_bitbake_parser():
 
     exec_group.add_argument("-S", "--dump-signatures", action="append",
                         default=[], metavar="SIGNATURE_HANDLER",
+                        choices=["none", "printdiff", "lockedsigs"],
                         help="Dump out the signature construction information, with no task "
                              "execution. The SIGNATURE_HANDLER parameter is passed to the "
                              "handler. Allowed values are none, printdiff and lockedsigs. "