[PATCH 3/6] patman: Don't look at sys.argv when parsing settings
Douglas Anderson
dianders at chromium.org
Thu Jun 30 23:08:06 CEST 2022
If you call the parser and tell it to parse but don't pass arguments
in then it will default to looking at sys.argv. This isn't really what
was intended and seems to have some side effects. Let's not do it.
NOTE: to see some of the side effects, note that this patch breaks
"patman -c1" if you don't have the patch ("patman: Fix implicit
command inserting") before it.
Signed-off-by: Douglas Anderson <dianders at chromium.org>
---
tools/patman/settings.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/patman/settings.py b/tools/patman/settings.py
index 5eefe3d1f55e..92d82d5e8682 100644
--- a/tools/patman/settings.py
+++ b/tools/patman/settings.py
@@ -248,7 +248,7 @@ def _UpdateDefaults(main_parser, config):
# Collect the defaults from each parser
for parser in parsers:
- pdefs = parser.parse_known_args()[0]
+ pdefs = parser.parse_known_args([])[0]
defaults = dict(vars(pdefs))
# Go through the settings and collect defaults
--
2.37.0.rc0.161.g10f37bed90-goog
More information about the U-Boot
mailing list