opencos-eda 0.2.54__tar.gz → 0.2.56__tar.gz

{opencos_eda-0.2.54/opencos_eda.egg-info → opencos_eda-0.2.56}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: opencos-eda
-Version: 0.2.54
+Version: 0.2.56
 Summary: A simple Python package for wrapping RTL simuliatons and synthesis
 Author-email: Simon Sabato <simon@cognichip.ai>, Drew Ranck <drew@cognichip.ai>
 Project-URL: Homepage, https://github.com/cognichip/opencos
@@ -15,5 +15,5 @@ Requires-Dist: schema>=0.7.7
 Requires-Dist: toml>=0.10.2
 Requires-Dist: yamllint>=1.35.1
 Requires-Dist: PySerial>=3.5
-Requires-Dist: cocotb>=2.0.0b1
+Requires-Dist: cocotb>=2.0
 Dynamic: license-file

{opencos_eda-0.2.54 → opencos_eda-0.2.56}/opencos/commands/__init__.py

@@ -10,6 +10,7 @@ from .elab import CommandElab
 from .export import CommandExport
 from .flist import CommandFList
 from .multi import CommandMulti, CommandToolsMulti
+from .lint import CommandLint
 from .open import CommandOpen
 from .proj import CommandProj
 from .sim import CommandSim
@@ -28,6 +29,7 @@ __all__ = [
     'CommandExport',
     'CommandFList',
     'CommandMulti',
+    'CommandLint',
     'CommandOpen',
     'CommandProj',
     'CommandSim',

{opencos_eda-0.2.54 → opencos_eda-0.2.56}/opencos/commands/flist.py

@@ -10,6 +10,7 @@ import os
 from opencos import util
 from opencos.eda_base import CommandDesign
 from opencos.utils.str_helpers import strip_all_quotes
+from opencos.commands.sim import parameters_dict_get_command_list
 
 class CommandFList(CommandDesign):
     '''Base class command handler for: eda flist ...'''
@@ -22,6 +23,7 @@ class CommandFList(CommandDesign):
             'eda-dir'            : 'eda.flist', # user can specify eda-dir if files are generated.
             'out'                : "flist.out",
             'emit-define'        : True,
+            'emit-parameter'     : True,
             'emit-incdir'        : True,
             'emit-v'             : True,
             'emit-sv'            : True,
@@ -29,6 +31,7 @@ class CommandFList(CommandDesign):
             'emit-cpp'           : True,
             'emit-non-sources'   : True, # as comments, from DEPS 'reqs'
             'prefix-define'      : "+define+",
+            'prefix-parameter'   : "-G",
             'prefix-incdir'      : "+incdir+",
             'prefix-v'           : "",
             'prefix-sv'          : "",
@@ -179,6 +182,13 @@ class CommandFList(CommandDesign):
                     newline = prefix + qd1 + f"{d}{ed1}{value}" + qd2
                 print(newline, file=fo)
 
+        if self.args['emit-parameter']:
+            prefix = strip_all_quotes(self.args['prefix-parameter'])
+            for item in parameters_dict_get_command_list(
+                    params=self.parameters, arg_prefix=prefix
+            ):
+                print(item, file=fo)
+
         if self.args['emit-incdir']:
             prefix = strip_all_quotes(self.args['prefix-incdir'])
             for i in self.incdirs:

opencos_eda-0.2.56/opencos/commands/lint.py

@@ -0,0 +1,51 @@
+'''opencos.commands.lint - Base class command handler for: eda lint ...
+
+Intended to be overriden by Tool based classes (such as CommandLintVivado, etc)
+
+Note that many 'lint' command handlers that also can perform simulations, such
+as VerilatorLint, will instead inherit from VerilatorSim and simply perform a
+shortened sim as the lint, instead of inheriting CommandLint.
+
+Tools that don't support a 'sim' command will generally use CommandLint, such
+as CommandLintSlang.'''
+
+from opencos.commands.sim import CommandSim
+
+class CommandLint(CommandSim):
+    '''Base class command handler for: eda lint ...'''
+
+    command_name = 'lint'
+
+    def __init__(self, config: dict):
+        CommandSim.__init__(self, config=config)
+        # add args specific to this simulator
+        self.args['stop-after-compile'] = True
+        self.args['lint'] = True
+        self.args['verilate-args'] = []
+
+
+
+    def compile(self) -> None:
+        raise NotImplementedError
+
+    def elaborate(self) -> None:
+        raise NotImplementedError
+
+    def get_compile_command_lists(self, **kwargs) -> list:
+        ''' Returns a list of lists (list of command lists).'''
+        raise NotImplementedError
+
+    def get_elaborate_command_lists(self, **kwargs) -> list:
+        ''' Returns a list of lists (list of command lists).'''
+        raise NotImplementedError
+
+    # CommandSim methods that elab does not use:
+
+    def simulate(self):
+        pass
+
+    def get_simulate_command_lists(self, **kwargs) -> list:
+        return []
+
+    def get_post_simulate_command_lists(self, **kwargs) -> list:
+        return []

{opencos_eda-0.2.54 → opencos_eda-0.2.56}/opencos/commands/sim.py

@@ -17,6 +17,33 @@ from opencos import util, export_helper
 from opencos.eda_base import CommandDesign, Tool
 from opencos.utils import status_constants
 
+from opencos.utils.str_helpers import sanitize_defines_for_sh, strip_outer_quotes
+
+def parameters_dict_get_command_list(params: dict, arg_prefix: str = '-G') -> list:
+    '''Given dict of parameters, returns a command list'''
+
+    ret_list = []
+    if ' ' in arg_prefix:
+        arg_list_prefix = arg_prefix.split()
+        arg_str_prefix = ''
+    else:
+        arg_list_prefix = []
+        arg_str_prefix = arg_prefix
+
+    for k,v in params.items():
+        if not isinstance(v, (int, str)):
+            util.warning(f'parameter {k} has value: {v}, parameters must be int/string types')
+
+        ret_list.extend(arg_list_prefix)
+        if isinstance(v, int):
+            ret_list.append(f'{arg_str_prefix}{k}={v}')
+        else: # string
+            v = strip_outer_quotes(v.strip('\n'))
+            v = '"' + v + '"'
+            ret_list.append(f'{arg_str_prefix}{k}={sanitize_defines_for_sh(v)}')
+    return ret_list
+
+
 class CommandSim(CommandDesign):
     '''Base class command handler for: eda sim ...'''
 
@@ -80,6 +107,10 @@ class CommandSim(CommandDesign):
 
         self.args['verilate-args'] = []
 
+    def process_parameters_get_list(self, arg_prefix: str = '-G') -> list:
+        '''Returns list (suitable command list for shell or for tool) from self.parameters'''
+        return parameters_dict_get_command_list(params=self.parameters, arg_prefix=arg_prefix)
+
     def process_plusarg(self, plusarg: str, pwd: str = os.getcwd()) -> None:
         '''Override for CommandDesign.process_plusarg(..)'''
         maybe_plusarg = CommandDesign.process_plusarg(self, plusarg, pwd)

{opencos_eda-0.2.54 → opencos_eda-0.2.56}/opencos/deps/defaults.py

@@ -31,6 +31,7 @@ KNOWN_EDA_COMMANDS = set([
 SUPPORTED_TARGET_TABLE_KEYS = set([
     'args',
     'defines',
+    'parameters',
     'incdirs',
     'top',
     'deps',
@@ -52,6 +53,7 @@ SUPPORTED_TAG_KEYS = set([
     'deps',
     'reqs',
     'defines',
+    'parameters',
     'incdirs',
     'replace-config-tools',
     'additive-config-tools',

{opencos_eda-0.2.54 → opencos_eda-0.2.56}/opencos/deps/deps_file.py

@@ -306,7 +306,9 @@ class DepsFile:
                 return False
 
             if not is_valid_target_name(target_node):
-                util.warning(f"In file {self.rel_deps_file}, {target_node} {VALID_TARGET_INFO_STR}")
+                util.warning(
+                    f"In file {self.rel_deps_file}, {target_node} {VALID_TARGET_INFO_STR}"
+                )
 
             if not caller_info:
                 # If we don't have caller_info, likely came from command line (or DEPS JSON data):

{opencos_eda-0.2.54 → opencos_eda-0.2.56}/opencos/deps/deps_processor.py

@@ -91,6 +91,22 @@ class DepsProcessor: # pylint: disable=too-many-instance-attributes
                         v = v.replace('%SEED%', str(self.args.get('seed', 1)))
                 self.command_design_ref.process_plusarg(f'+define+{k}={v}')
 
+
+    def apply_parameters(self, parameters_dict: dict):
+        '''Given parameters_dict, applies them to our self.command_design_ref obj'''
+        if not isinstance(parameters_dict, dict):
+            self.error(f"{parameters_dict=} is not type dict, can't apply defines,",
+                       f"in {self.caller_info}")
+        for k,v in parameters_dict.items():
+            if v is None or v == '' or not isinstance(v, (int, str, bool)):
+                warning(f'parameter {k} has value: {v}, parameters must be bool/int/string types',
+                        f'from {self.caller_info}')
+            else:
+                self.command_design_ref.set_parameter(
+                    name=k, value=v, caller_info=self.caller_info
+                )
+
+
     def apply_incdirs(self, incdirs_list:list):
         '''Given incdirs_list, applies them to our self.command_design_ref obj'''
         if not isinstance(incdirs_list, (str, list)):
@@ -252,6 +268,8 @@ class DepsProcessor: # pylint: disable=too-many-instance-attributes
                 remaining_deps_list += self.process_tags()
             elif key == 'defines':
                 self.process_defines()
+            elif key == 'parameters':
+                self.process_parameters()
             elif key == 'incdirs':
                 self.process_incdirs()
             elif key == 'top':
@@ -426,6 +444,9 @@ class DepsProcessor: # pylint: disable=too-many-instance-attributes
                         # apply defines:
                         self.apply_defines(value.get('defines', {}))
 
+                    elif key == 'parameters':
+                        self.apply_parameters(value.get('parameters', {}))
+
                     elif key == 'incdirs':
                         # apply incdirs:
                         self.apply_incdirs(value.get('incdirs', []))
@@ -517,6 +538,22 @@ class DepsProcessor: # pylint: disable=too-many-instance-attributes
 
         self.apply_defines(entry_defines)
 
+    def process_parameters(self):
+        '''Returns None, applies parameters (dict, if any) from self.deps_entry to
+        self.command_design_ref.'''
+
+        # Parameters:
+        # apply command specific parameters, with higher priority than the a
+        # deps_entry['sim']['parameters'] entry,
+        # do this with dict1.update(dict2):
+        entry_parameters = {}
+        entry_parameters.update(self.deps_entry.get('parameters', {}))
+        entry_parameters.update(self.entry_eda_command.get('parameters', {}))
+        assert isinstance(entry_parameters, dict), \
+            f'{entry_parameters=} for in {self.caller_info} must be a dict'
+
+        self.apply_parameters(entry_parameters)
+
     def process_incdirs(self) -> None:
         '''Returns None, applies incdirs (dict, if any) from self.deps_entry to
         self.command_design_ref.'''
@@ -700,7 +737,8 @@ class DepsProcessor: # pylint: disable=too-many-instance-attributes
                 deps_targets_to_resolve.append(command_tuple)
 
 
-            elif isinstance(dep, str) and any(dep.startswith(x) for x in ['+define+', '+incdir']):
+            elif isinstance(dep, str) and \
+                 any(dep.startswith(x) for x in ['+define+', '+incdir+']):
                 # Note: we still support +define+ and +incdir in the deps list.
                 # check for compile-time Verilog style plusarg, which are supported under targets
                 # These are not run-time Verilog style plusargs comsumable from within the .sv:

{opencos_eda-0.2.54 → opencos_eda-0.2.56}/opencos/deps_schema.py

@@ -11,6 +11,9 @@ my_target_name:
   defines:                           <---- defines, optional table
     USER_DEFINE_VALUE: 12
 
+  parameters:                        <---- parameters, optional table
+    USER_PARAM_VALUE: 12
+
   incdirs:                           <---- incdirs, optional array (or string)
     - ./
 
@@ -32,6 +35,9 @@ my_target_name:
   defines:                           <---- defines, optional table of (key: str Or null)
     USER_DEFINE_VALUE: 12
 
+  parameters:                        <---- parameters, optional table
+    USER_PARAM_VALUE: 12
+
   incdirs:                           <---- incdirs, optional array of strings
     - ./
 
@@ -55,6 +61,7 @@ METADATA:                            <---- (optional) unstructured table
 
 my_target_name:
   defines:                           <---- defines, optional table
+  parameters:                        <---- parameters, optional table
   incdirs:                           <---- incdirs, optional array (or string)
   top: tb                            <---- top, optional string
   deps:                              <---- TARGET_DEPS_CONTENTS schema
@@ -105,6 +112,8 @@ my_target_name:
                                            to the current target for this named eda command.
     defines:                         <---- optional table, defines to be applied to the current
                                            target for this named eda command.
+    parameters:                      <---- optional table, parameters to be applied to the current
+                                           target to for this named eda command.
     incdirs:                         <---- optional array (or string) incdirs to be applied to
                                            the current target for this named eda command.
   tags:                              <---- tags, optional table using TARGET_TAGS_TABLE schema.
@@ -204,6 +213,9 @@ TARGET_EDA_COMMAND_ENTRY_TABLE = {
     Optional('defines'): {
         Optional(str): Or(str, int, type(None)),
     },
+    Optional('parameters'): {
+        Optional(str): Or(str, int),
+    },
     Optional('incdirs'): ARRAY_OR_SPACE_SEPARATED_STRING,
 }
 
@@ -226,6 +238,9 @@ TARGET_TAGS_TABLE = {
         Optional('defines'): {
             Optional(str): Or(str, int, type(None)),
         },
+        Optional('parameters'): {
+            Optional(str): Or(str, int),
+        },
         Optional('incdirs'): ARRAY_OR_SPACE_SEPARATED_STRING,
         Optional('replace-config-tools'): dict,
         Optional('additive-config-tools'): dict,
@@ -244,6 +259,10 @@ TARGET_CONTENTS = Or(
         Optional('defines'): {
             Optional(str): Or(str, int, type(None)),
         },
+        # parameters: table of key-value
+        Optional('parameters'): {
+            Optional(str): Or(str, int),
+        },
         # incdirs: array
         Optional('incdirs'): ARRAY_OR_SPACE_SEPARATED_STRING,
         # top: string
@@ -302,6 +321,9 @@ FILE_SIMPLIFIED = Schema(
                     Optional('defines'): {
                         Optional(str): Or(type(None), str),
                     },
+                    Optional('parameters'): {
+                        Optional(str): str,
+                    },
                     Optional('incdirs'): [str],
                     Optional('top'): str,
                     Optional('deps'): [str],

{opencos_eda-0.2.54 → opencos_eda-0.2.56}/opencos/eda.py

@@ -23,7 +23,7 @@ from opencos import util, eda_config, eda_base
 from opencos.eda_base import Tool, which_tool, get_eda_exec
 from opencos.utils import vsim_helper, vscode_helper
 from opencos.utils.subprocess_helpers import subprocess_run_background
-from opencos.utils import status_constants
+from opencos.utils import status_constants, str_helpers
 
 # Configure util:
 util.progname = "EDA"
@@ -78,7 +78,7 @@ def get_all_commands_help_str(config: dict) -> str:
     return '\n'.join(all_commands_help)
 
 
-def usage(tokens: list, config: dict, command="") -> int:
+def usage(tokens: list, config: dict, command: str = "", tool: str = "") -> int:
     '''Returns an int shell return code, given remaining args (tokens list) and eda command.
 
     config is the config dict. Used to check valid commands in config['command_handler']
@@ -112,6 +112,9 @@ And <files|targets, ...> is one or more source file or DEPS markup file target,
 
     if command in config['command_handler'].keys():
         sco = config['command_handler'][command](config=config) # sub command object
+        sco_tool = getattr(sco, '_TOOL', '')
+        if tool and tool != sco_tool:
+            util.warning(f'{tool=} does not support {command=}')
         sco.help(tokens=tokens)
         return util.exit(0)
 
@@ -282,9 +285,15 @@ def tool_setup(tool: str, config: dict, quiet: bool = False, auto_setup: bool =
     if tool not in config['auto_tools_order'][0]:
         tools = list(config.get('auto_tools_order', [{}])[0].keys())
         cfg_yaml_fname = config.get('config-yml', None)
+        util.warning(f'Unknown tool: {tool}')
+        if tools:
+            util.info('Known tools:')
+            pretty_tools = str_helpers.pretty_list_columns_manual(data=tools)
+            for row in pretty_tools:
+                if row:
+                    util.info(row)
         util.error(f"Don't know how to run tool_setup({tool=}), is not in",
-                   f"config['auto_tools_order'] for {tools=}",
-                   f"from {cfg_yaml_fname}")
+                   f"config['auto_tools_order'] from {cfg_yaml_fname}")
         return
 
     if tool not in config['auto_tools_found']:
@@ -394,7 +403,9 @@ def process_tokens( # pylint: disable=too-many-branches,too-many-statements,too-
             for arg in unparsed:
                 if not arg.startswith('-'):
                     command = arg
-        return usage(tokens=unparsed, config=config, command=command)
+        if parsed.tool:
+            tool_setup(parsed.tool, config=config)
+        return usage(tokens=unparsed, config=config, command=command, tool=parsed.tool)
 
     if parsed.tool:
         tool_setup(parsed.tool, config=config)

{opencos_eda-0.2.54 → opencos_eda-0.2.56}/opencos/eda_base.py

@@ -831,7 +831,33 @@ class Command: # pylint: disable=too-many-public-methods
                 lines.append(indent_me(f"  --{k:20} : string     : {vstr:12}{khelp}"))
             else:
                 lines.append(indent_me(f"  --{k:20} : <unknown>  : {vstr:12}{khelp}"))
+
+        lines.append('')
+        lines.append(indent_me((
+            "  -G<parameterName>=<value>      "
+            " Add parameter to top level, support bit/int/string types only."
+            " Example: -GDEPTH=8 (DEPTH treated as SV int/integer)."
+            " -GENABLE=1 (ENABLED treated as SV bit/int/integer)."
+            " -GName=eda (Name treated as SV string \"eda\")."
+        )))
+        lines.append(indent_me((
+            "  +define+<defineName>           "
+            " Add define w/out value to tool ahead of SV sources"
+            " Example: +define+SIM_SPEEDUP"
+        )))
+        lines.append(indent_me((
+            "  +define+<defineName>=<value>   "
+            " Add define w/ value to tool ahead of SV sources"
+            " Example: +define+TECH_LIB=2 +define+FULL_NAME=\"E D A\""
+        )))
+        lines.append(indent_me((
+            "  +incdir+<path>                 "
+            " Add path (absolute or relative) for include directories"
+            " for SystemVerilog `include \"<some-file>.svh\""
+            " Example: +incdir+../lib"
+        )))
         lines.append('')
+
         for line in lines:
             print(line)
 
@@ -922,6 +948,7 @@ class CommandDesign(Command): # pylint: disable=too-many-instance-attributes
             ),
         })
         self.defines = {}
+        self.parameters = {}
         self.incdirs = []
         self.files = {}
         self.files_v = []
@@ -1086,10 +1113,72 @@ class CommandDesign(Command): # pylint: disable=too-many-instance-attributes
         # TODO(drew): Use the helper method in util for this instead to peek in file contents?
         return os.path.splitext(os.path.basename(name))[0]
 
+    def set_parameter(
+            self, name: str, value, caller_info: str = '(CLI)',
+            wrap_str_double_quotes: bool = True
+    ) -> None:
+        '''Safe wrapper for setting a parameter (can only set once, from CLI or DEPS)'''
+
+        if name in self.parameters:
+            util.debug("Parameter not set because it is already modified",
+                       f"(via -G<Name>=<Value>): {name}={value}; orig value",
+                       f'{self.parameters[name]} from {caller_info}')
+        else:
+            if isinstance(value, bool):
+                value = int(value)
+            elif isinstance(value, str):
+                value = strip_outer_quotes(value.strip('\n'))
+                if wrap_str_double_quotes:
+                    value = '"' + value + '"'
+            self.parameters[name] = value
+            util.debug(f"Parameter (via -G<Name>=<Value>): {name}={value}",
+                       f'from {caller_info}')
+
+
+    def process_parameter_arg(
+            self, text: str, pwd: str = os.getcwd()
+    ) -> None:
+        '''Retuns None, parses -G<Name>=<Value> adds to internal self.parameters.'''
+
+        # Deal with raw CLI/bash/powershell argparser, strip all outer quotes.
+        text = strip_outer_quotes(text)
+        if not pwd:
+            pwd = ''
+
+        if not text.startswith('-G'):
+            self.error(f"Didn't understand -G parameter arg: '{text}'",
+                       error_code=status_constants.EDA_COMMAND_OR_ARGS_ERROR)
+            return
+
+        text = text[2:] # strip leading -G
+        m = re.match(r'^(\w+)$', text)
+        if m:
+            k = m.group(1)
+            util.warning(f"Parameter {k} has no value and will not be applied")
+            return
+        m = re.match(r'^(\w+)\=(\S+)$', text)
+        if not m:
+            m = re.match(r'^(\w+)\=(\"[^\"]*\")$', text)
+        if m:
+            k = m.group(1)
+            v = m.group(2)
+            # since this is coming from a CLI string, we have to guess at types,
+            # for int or bool (not str) and convert bool to int:
+            if not isinstance(v, (int, str)):
+                self.error(f"Didn't understand -G parameter arg: name={k} value={v}",
+                           f"value must be int/str type, from: '{text}'",
+                           error_code=status_constants.EDA_COMMAND_OR_ARGS_ERROR)
+                return
+            try:
+                v = int(v)
+            except ValueError:
+                pass
+            self.set_parameter(k, v)
+
 
     def process_plusarg( # pylint: disable=too-many-branches
             self, plusarg: str, pwd: str = os.getcwd()
-    ) -> None:
+    ) -> str:
         '''Retuns str, parses a +define+, +incdir+, +key=value str; adds to internal.
 
         Adds to self.defines, self.incdirs,
@@ -1112,7 +1201,7 @@ class CommandDesign(Command): # pylint: disable=too-many-instance-attributes
                 k = m.group(1)
                 self.defines[k] = None
                 util.debug(f"Defined {k}")
-                return None
+                return ''
             m = re.match(r'^(\w+)\=(\S+)$', plusarg)
             if not m:
                 m = re.match(r'^(\w+)\=(\"[^\"]*\")$', plusarg)
@@ -1126,10 +1215,10 @@ class CommandDesign(Command): # pylint: disable=too-many-instance-attributes
                         v = v.replace('%SEED%', str(self.args.get('seed', 1)))
                 self.defines[k] = v
                 util.debug(f"Defined {k}={v}")
-                return None
+                return ''
             self.error(f"Didn't understand +define+: '{plusarg}'",
                        error_code=status_constants.EDA_COMMAND_OR_ARGS_ERROR)
-            return None
+            return ''
 
         if plusarg.startswith('+incdir+'):
             plusarg = plusarg[len('+incdir+'):]
@@ -1139,16 +1228,16 @@ class CommandDesign(Command): # pylint: disable=too-many-instance-attributes
                 if incdir not in self.incdirs:
                     self.incdirs.append(os.path.abspath(incdir))
                     util.debug(f"Added include dir '{os.path.abspath(incdir)}'")
-                return None
+                return ''
             self.error(f"Didn't understand +incdir+: '{plusarg}'",
                        error_code=status_constants.EDA_COMMAND_OR_ARGS_ERROR)
-            return None
+            return ''
 
         # remaining plusargs as stored in self.args['unprocessed-plusargs'] (list)
         if plusarg.startswith('+'):
             if not self.config.get('bare_plusarg_supported', False):
                 self.error(f"bare plusarg(s) are not supported: {plusarg}'")
-                return None
+                return ''
             if plusarg not in self.args['unprocessed-plusargs']:
                 self.args['unprocessed-plusargs'].append(plusarg)
                 # For anything added to unprocessed-plusarg, we have to return it, to let
@@ -1157,7 +1246,7 @@ class CommandDesign(Command): # pylint: disable=too-many-instance-attributes
 
         self.error(f"Didn't understand +plusarg: '{plusarg}'",
                    error_code=status_constants.EDA_COMMAND_OR_ARGS_ERROR)
-        return None
+        return ''
 
 
     def append_shell_commands(self, cmds : list) -> None:
@@ -1439,8 +1528,8 @@ class CommandDesign(Command): # pylint: disable=too-many-instance-attributes
         unparsed = Command.process_tokens(self, tokens, process_all=False, pwd=pwd)
         util.debug(f'CommandDesign - after Command.process_tokens(..) {unparsed=}')
 
-        # deal with +define+ or +incdir+, consume it and remove from unparsed
-        # walk the list, remove all items after we're done.
+        # deal with +define+, +incdir+, +(plusargName)+, or -GParameterName=Value:
+        # consume it and remove from unparsed, walk the list, remove all items after we're done.
         remove_list = []
         for token in unparsed:
             # Since this is a raw argparser, we may have args that come from shlex.quote(token),
@@ -1454,6 +1543,15 @@ class CommandDesign(Command): # pylint: disable=too-many-instance-attributes
                 plusarg = strip_outer_quotes(token)
                 self.process_plusarg(plusarg, pwd=pwd)
                 remove_list.append(token)
+                continue
+
+            # Parameters in -G<word>=<something>
+            m = re.match(r"^\'?\-G\w+\=.+", token)
+            if m:
+                # Copy and strip all outer ' or " on the text:
+                param = strip_outer_quotes(token)
+                self.process_parameter_arg(param, pwd=pwd)
+                remove_list.append(token)
 
         for x in remove_list:
             unparsed.remove(x)

{opencos_eda-0.2.54 → opencos_eda-0.2.56}/opencos/eda_config.py

@@ -305,7 +305,8 @@ def write_eda_config_and_args(
     fullpath = os.path.join(dirpath, filename)
     data = {}
     for x in ['command_name', 'config', 'target', 'args', 'modified_args', 'defines',
-              'incdirs', 'files_v', 'files_sv', 'files_vhd']:
+              'incdirs', 'files_v', 'files_sv', 'files_vhd', 'files_cpp', 'files_sdc',
+              'files_non_source']:
         # Use deep copy b/c otherwise these are references to opencos.eda.
         data[x] = copy.deepcopy(getattr(command_obj_ref, x, ''))