[Buildroot] [PATCHv3 buildroot-test 3/9] autobuild-run: add option --make-opts for custom Buildroot options
Thomas De Schampheleire
patrickdepinguin at gmail.com
Mon Oct 20 19:39:22 UTC 2014
From: Thomas De Schampheleire <thomas.de.schampheleire at gmail.com>
In some environments, the Buildroot configuration may need minor
tweaking. For example, in corporate environments behind a firewall,
version control commands like git, svn, ... may actually need wrappers.
Such wrappers could be configured using the corresponding BR2_<CMD>
configuration options.
Since these tweaks are very environment-specific, they should not be
part of the configuration file itself. Otherwise, other people trying to
reproduce the build in another environment will see builds failing due
to other reasons, for example the wrapper not being found.
This commit provides a new configuration option 'make-opts', passable as
argument or in the config file, that accepts a string of make options to
pass to the Buildroot make command. For example:
--make-opts="BR2_GIT=git-wrapper BR2_svn=svn-wrapper"
Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire at gmail.com>
---
scripts/autobuild-run | 18 ++++++++++++------
1 file changed, 12 insertions(+), 6 deletions(-)
diff --git a/scripts/autobuild-run b/scripts/autobuild-run
index 835ca58..7ce0715 100755
--- a/scripts/autobuild-run
+++ b/scripts/autobuild-run
@@ -72,6 +72,9 @@ Options:
reasons it is recommended to define this in the
config file instead, with user-read permissions
only)
+ --make-opts OPTSTRING string of extra options to pass to Buildroot
+ make, such as specific command wrappers
+ [default: ]
-c, --config CONFIG path to configuration file
Format of the configuration file:
@@ -414,7 +417,7 @@ def gen_config(instance, log, sysinfo):
return 0
-def do_build(instance, njobs, log):
+def do_build(instance, njobs, log, make_opts):
"""Run the build itself"""
idir = "instance-%d" % instance
@@ -427,8 +430,10 @@ def do_build(instance, njobs, log):
srcdir = os.path.join(idir, "buildroot")
f = open(os.path.join(outputdir, "logfile"), "w+")
log_write(log, "INFO: build started")
- ret = subprocess.call(["timeout", str(MAX_DURATION), "make", "O=%s" % outputdir, "-C", srcdir,
- "BR2_DL_DIR=%s" % dldir, "BR2_JLEVEL=%s" % njobs], stdout=f, stderr=f)
+ cmd = ["timeout", str(MAX_DURATION), "make", "O=%s" % outputdir,
+ "-C", srcdir, "BR2_DL_DIR=%s" % dldir, "BR2_JLEVEL=%s" % njobs] \
+ + make_opts.split()
+ ret = subprocess.call(cmd, stdout=f, stderr=f)
# 124 is a special error code that indicates we have reached the
# timeout
if ret == 124:
@@ -519,7 +524,8 @@ def send_results(instance, http_login, http_password, submitter, log, result):
os.rename(os.path.join(outputdir, "results.tar.bz2"), resultfilename)
log_write(log, "INFO: results saved as %s" % resultfilename)
-def run_instance(instance, njobs, http_login, http_password, submitter, sysinfo):
+def run_instance(instance, njobs, http_login, http_password, submitter,
+ make_opts, sysinfo):
"""Main per-instance loop
Prepare the build, generate a configuration, run the build, and submit the
@@ -545,7 +551,7 @@ def run_instance(instance, njobs, http_login, http_password, submitter, sysinfo)
if ret != 0:
continue
- ret = do_build(instance, njobs, instance_log)
+ ret = do_build(instance, njobs, instance_log, make_opts)
send_results(instance, http_login, http_password, submitter, instance_log, ret)
# args / config file merging inspired by:
@@ -600,7 +606,7 @@ def main():
for i in range(0, int(args['--ninstances'])):
p = Process(target=run_instance, args=(i, int(args['--njobs']),
args['--http-login'], args['--http-password'],
- args['--submitter'], sysinfo))
+ args['--submitter'], args['--make-opts'], sysinfo))
p.start()
processes.append(p)
signal.signal(signal.SIGTERM, sigterm_handler)
--
1.8.5.1
More information about the buildroot
mailing list