genrandconfig: move instantiation of SystemInfo down

The SystemInfo class is instantiated globally and passed down to all
functions, but it is really only used in fixup_config. So instead,
instantiate it there.

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
This commit is contained in:
Arnout Vandecappelle 2017-07-21 03:05:12 +02:00 committed by Thomas Petazzoni
parent 22978c7399
commit e8c6d52c89

View File

@ -197,7 +197,7 @@ def is_toolchain_usable(outputdir, config):
return True return True
def fixup_config(outputdir, sysinfo): def fixup_config(outputdir):
"""Finalize the configuration and reject any problematic combinations """Finalize the configuration and reject any problematic combinations
This function returns 'True' when the configuration has been This function returns 'True' when the configuration has been
@ -206,6 +206,7 @@ def fixup_config(outputdir, sysinfo):
generated). generated).
""" """
sysinfo = SystemInfo()
with open(os.path.join(outputdir, ".config")) as configf: with open(os.path.join(outputdir, ".config")) as configf:
configlines = configf.readlines() configlines = configf.readlines()
@ -402,7 +403,7 @@ def gen_config(args):
if ret != 0: if ret != 0:
log_write(args.log, "ERROR: cannot generate random configuration") log_write(args.log, "ERROR: cannot generate random configuration")
return -1 return -1
if fixup_config(outputdir, args.sysinfo): if fixup_config(outputdir):
break break
ret = subprocess.call(["make", "O=%s" % outputdir, "-C", srcdir, ret = subprocess.call(["make", "O=%s" % outputdir, "-C", srcdir,
@ -436,7 +437,6 @@ if __name__ == '__main__':
# Arguments expected by gen_config for which we just set a default here # Arguments expected by gen_config for which we just set a default here
args.log = sys.stdout args.log = sys.stdout
args.sysinfo = SystemInfo()
# Output directory is already created by autobuild-run so emulate it here # Output directory is already created by autobuild-run so emulate it here
idir = "instance-%d" % args.instance idir = "instance-%d" % args.instance