From: Maxim Cournoyer Date: Tue, 20 Dec 2022 05:38:39 +0000 (-0500) Subject: patman: set the default config_fname argument value to None X-Git-Url: http://git.dujemihanovic.xyz/login.html?a=commitdiff_plain;h=2c58a5e2753e4f0afc4c937b07b9380398bbe8fd;p=u-boot.git patman: set the default config_fname argument value to None This better matches Python conventions, allowing to easily test whether the optional argument is provided. Signed-off-by: Maxim Cournoyer Reviewed-by: Simon Glass --- diff --git a/tools/patman/__main__.py b/tools/patman/__main__.py index 8e32655ad2..37489304f7 100755 --- a/tools/patman/__main__.py +++ b/tools/patman/__main__.py @@ -117,7 +117,7 @@ status.add_argument('-f', '--force', action='store_true', argv = sys.argv[1:] args, rest = parser.parse_known_args(argv) if hasattr(args, 'project'): - settings.Setup(parser, args.project, '') + settings.Setup(parser, args.project) args, rest = parser.parse_known_args(argv) # If we have a command, it is safe to parse all arguments diff --git a/tools/patman/settings.py b/tools/patman/settings.py index 5efad5ed78..8b846799df 100644 --- a/tools/patman/settings.py +++ b/tools/patman/settings.py @@ -333,19 +333,20 @@ def GetItems(config, section): return [] -def Setup(parser, project_name, config_fname=''): +def Setup(parser, project_name, config_fname=None): """Set up the settings module by reading config files. Args: - parser: The parser to update + parser: The parser to update. project_name: Name of project that we're working on; we'll look for sections named "project_section" as well. - config_fname: Config filename to read ('' for default) + config_fname: Config filename to read. """ # First read the git alias file if available _ReadAliasFile('doc/git-mailrc') config = _ProjectConfigParser(project_name) - if config_fname == '': + + if not config_fname: config_fname = '%s/.patman' % os.getenv('HOME') if not os.path.exists(config_fname):