On 22/01/2019 03:33, Chris Johns wrote:
---
  source-builder/sb/options.py | 15 +++++++++------
  1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/source-builder/sb/options.py b/source-builder/sb/options.py
index c5fcabe..eb0c2d5 100644
--- a/source-builder/sb/options.py
+++ b/source-builder/sb/options.py
@@ -53,7 +53,7 @@ class command_line:
def __init__(self, argv, optargs, _defaults, command_path):
          self._long_opts = {
-            # key                 macro                handler            
param  defs   init
+            # key                       macro                handler           
 param  defs   init
              '--prefix'               : ('_prefix',           self._lo_path,   
  True,  None,  False),
              '--topdir'               : ('_topdir',           self._lo_path,   
  True,  None,  False),
              '--configdir'            : ('_configdir',        self._lo_path,   
  True,  None,  False),
@@ -83,6 +83,9 @@ class command_line:
              '--host'                 : ('_host',             
self._lo_triplets, True,  None,  False),
              '--build'                : ('_build',            
self._lo_triplets, True,  None,  False),
              '--target'               : ('_target',           
self._lo_triplets, True,  None,  False),
+            '--rtems-tools'          : ('_rtems_tools',      self._lo_string,  
 True,  None,  False),
+            '--rtems-bsp'            : ('_rtems_bsp',        self._lo_string,  
 True,  None,  False),
+            '--rtems-version'        : ('_rtems_version',    self._lo_string,  
 True,  None,  False),
              '--help'                 : (None,                self._lo_help,   
  False, None,  False)
              }
@@ -575,6 +578,11 @@ class command_line:
              if self.get_arg('--with-tools') is not None:
                  raise error.general('--rtems-tools and --with-tools cannot be 
used together')
              self.args.append('--with-tools=%s' % (rtems_tools[1]))
+        rtems_version = self.parse_args('--rtems-version')
+        if rtems_version is None:
+            rtems_version = version.version()
+        else:
+            rtems_version = rtems_version[1]
          rtems_arch_bsp = self.parse_args('--rtems-bsp')
          if rtems_arch_bsp is not None:
              if self.get_arg('--target') is not None:
@@ -582,11 +590,6 @@ class command_line:
              ab = rtems_arch_bsp[1].split('/')
              if len(ab) != 2:
                  raise error.general('invalid --rtems-bsp option')
-            rtems_version = self.parse_args('--rtems-version')
-            if rtems_version is None:
-                rtems_version = version.version()
-            else:
-                rtems_version = rtems_version[1]
              self.args.append('--target=%s-rtems%s' % (ab[0], rtems_version))
              self.args.append('--with-rtems-bsp=%s' % (ab[1]))

Thanks, this makes the options available. I would like to use it also to change the default prefix, e.g.

../source-builder/sb-set-builder --rtems-version=6 6/rtems-sparc

-> "/opt/rtems/6"

I need to save the RTEMS version somewhere, the rtems_version is not a global variable.

--
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.hu...@embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.

_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to