[PATCH 3/6] patman: Don't look at sys.argv when parsing settings

Douglas Anderson
2022-06-30T21:09:25Z

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@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