Updates for phosphor-logging configure flags
Modified how configure flags are handled when a flag contains spaces
that are surrounded by quotes when ran manually on command line
Resolves openbmc/openbmc#1385
Change-Id: I651d37000e5b104b3ebb3f895806f37d1ea3ee28
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
diff --git a/scripts/unit-test.py b/scripts/unit-test.py
index a1ca5ce..5374327 100755
--- a/scripts/unit-test.py
+++ b/scripts/unit-test.py
@@ -114,13 +114,13 @@
# Build & install this package
if not dep_installed[pkg]:
- conf_flags = ""
+ conf_flags = []
os.chdir(pkgdir)
# Add any necessary configure flags for package
if CONFIGURE_FLAGS.get(pkg) is not None:
- conf_flags = " ".join(CONFIGURE_FLAGS.get(pkg))
+ conf_flags.extend(CONFIGURE_FLAGS.get(pkg))
check_call_cmd(pkgdir, './bootstrap.sh')
- check_call_cmd(pkgdir, './configure', conf_flags)
+ check_call_cmd(pkgdir, './configure', *conf_flags)
check_call_cmd(pkgdir, 'make')
check_call_cmd(pkgdir, 'make', 'install')
dep_installed[pkg] = True
@@ -132,7 +132,10 @@
# CONFIGURE_FLAGS = [GIT REPO]:[CONFIGURE FLAGS]
CONFIGURE_FLAGS = {
'phosphor-objmgr': ['--enable-unpatched-systemd'],
- 'sdbusplus': ['--enable-transaction']
+ 'sdbusplus': ['--enable-transaction'],
+ 'phosphor-logging':
+ ['--enable-metadata-processing',
+ 'YAML_DIR=/usr/local/share/phosphor-dbus-interfaces/yaml']
}
# DEPENDENCIES = [MACRO]:[library/header]:[GIT REPO]