leandrof/u-boot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

patman: Avoid importing gitutil in settings

Browse Source 
Pass this module in so that settings does not need to import it.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reported-by: Stefan Bosch <stefan_b@posteo.net>
This commit is contained in:
Simon Glass 2020-06-07 06:45:49 -06:00
parent 156e655372
commit dd3dac2f2c
2 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tools/patman/main.py
View File

@ -80,7 +80,7 @@ specified by tags you place in the commits. Use -n to do a dry run first."""
# Parse options twice: first to get the project and second to handle
# defaults properly (which depends on project).
(options, args) = parser.parse_args()
settings.Setup(parser, options.project, '')
settings.Setup(gitutil, parser, options.project, '')
(options, args) = parser.parse_args()
if __name__ != "__main__":

7
tools/patman/settings.py
View File

@ -11,7 +11,6 @@ import os
import re
from patman import command
from patman import gitutil
from patman import tools
"""Default settings per-project.
@ -185,7 +184,7 @@ def ReadGitAliases(fname):
fd.close()
def CreatePatmanConfigFile(config_fname):
def CreatePatmanConfigFile(gitutil, config_fname):
"""Creates a config file under $(HOME)/.patman if it can't find one.
Args:
@ -301,7 +300,7 @@ def GetItems(config, section):
except:
raise
def Setup(parser, project_name, config_fname=''):
def Setup(gitutil, parser, project_name, config_fname=''):
"""Set up the settings module by reading config files.
Args:
@ -318,7 +317,7 @@ def Setup(parser, project_name, config_fname=''):
if not os.path.exists(config_fname):
print("No config file found ~/.patman\nCreating one...\n")
CreatePatmanConfigFile(config_fname)
CreatePatmanConfigFile(gitutil, config_fname)
config.read(config_fname)