[rtems commit] build: Fix option default values

Sebastian Huber sebh at rtems.org
Sun Sep 27 07:40:37 UTC 2020


Module:    rtems
Branch:    master
Commit:    71ccc156a55af6abda1be611610ceb0420c5e923
Changeset: http://git.rtems.org/rtems/commit/?id=71ccc156a55af6abda1be611610ceb0420c5e923

Author:    Sebastian Huber <sebastian.huber at embedded-brains.de>
Date:      Sun Sep 27 09:36:07 2020 +0200

build: Fix option default values

For the option default values we have to use the BSP name defined by the build
specification and not the user.

Update #3818.

---

 wscript | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/wscript b/wscript
index e5ac53f..6280afa 100755
--- a/wscript
+++ b/wscript
@@ -912,7 +912,7 @@ class OptionItem(Item):
             value = cic.cp.getboolean(conf.variant, name)
             cic.add_option(name)
         except configparser.NoOptionError:
-            value = self.default_value(conf.variant)
+            value = self.default_value(conf.env.ARCH_BSP)
         except ValueError as ve:
             conf.fatal(
                 "Invalid value for configuration option {}: {}".format(name, ve)
@@ -928,7 +928,7 @@ class OptionItem(Item):
             value = cic.cp.get(conf.variant, name)
             cic.add_option(name)
         except configparser.NoOptionError:
-            value = self.default_value(conf.variant)
+            value = self.default_value(conf.env.ARCH_BSP)
             if value is None:
                 return value
         try:
@@ -947,7 +947,7 @@ class OptionItem(Item):
             cic.add_option(name)
             value = no_unicode(value)
         except configparser.NoOptionError:
-            value = self.default_value(conf.variant)
+            value = self.default_value(conf.env.ARCH_BSP)
         return value
 
     def _script(self, conf, cic, value, arg):
@@ -1358,6 +1358,7 @@ def configure_variant(conf, cp, bsp_map, path_list, top_group, variant):
     arch_bsp = arch + "/" + bsp_base
 
     conf.env["ARCH"] = arch
+    conf.env["ARCH_BSP"] = arch_bsp
     conf.env["BSP_BASE"] = bsp_base
     conf.env["BSP_NAME"] = bsp_name
     conf.env["DEST_OS"] = "rtems"



More information about the vc mailing list