opencos-eda 0.3.1__tar.gz → 0.3.3__tar.gz

{opencos_eda-0.3.1/opencos_eda.egg-info → opencos_eda-0.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: opencos-eda
-Version: 0.3.1
+Version: 0.3.3
 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
@@ -17,4 +17,5 @@ Requires-Dist: toml>=0.10.2
 Requires-Dist: yamllint>=1.35.1
 Requires-Dist: PySerial>=3.5
 Requires-Dist: cocotb>=2.0
+Requires-Dist: supports_color>=0.2.0
 Dynamic: license-file

{opencos_eda-0.3.1 → opencos_eda-0.3.3}/opencos/deps/deps_file.py

@@ -249,8 +249,11 @@ class DepsFile:
                 self.rel_deps_file = os.path.join(os.path.relpath(deps_path), deps_leaf)
 
         self.error = getattr(command_design_ref, 'error', None)
+        self.error_ifarg = getattr(command_design_ref, 'error_ifarg', None)
         if not self.error:
             self.error = util.error
+        if not self.error_ifarg:
+            self.error_ifarg = util.error
 
 
     def found(self) -> bool:
@@ -320,19 +323,28 @@ class DepsFile:
                 # If we don't have caller_info, likely came from command line (or DEPS JSON data):
                 if '.' in target_node:
                     # Likely a filename (target_node does not include path)
-                    self.error(f'Trying to resolve command-line target={t_full} (file?):',
-                               f'File={t_node} not found in directory={t_path}',
-                               error_code=EDA_DEPS_FILE_NOT_FOUND)
+                    self.error_ifarg(
+                        f'Trying to resolve command-line target={t_full} (file?):',
+                        f'File={t_node} not found in directory={t_path}',
+                        arg='error-unknown-args',
+                        error_code=EDA_DEPS_FILE_NOT_FOUND
+                    )
                 elif not self.rel_deps_file:
                     # target, but there's no DEPS file
-                    self.error(f'Trying to resolve command-line target={t_full}:',
-                               f'but path {t_path} has no DEPS markup file (DEPS.yml)',
-                               error_code=EDA_DEPS_FILE_NOT_FOUND)
+                    self.error_ifarg(
+                        f'Trying to resolve command-line target={t_full}:',
+                        f'but path {t_path} has no DEPS markup file (DEPS.yml)',
+                        arg='error-unknown-args',
+                        error_code=EDA_DEPS_FILE_NOT_FOUND
+                    )
                 else:
                     self.warning_show_available_targets()
-                    self.error(f'Trying to resolve command-line target={t_full}:',
-                               f'was not found in deps_file={self.rel_deps_file}',
-                               error_code=EDA_DEPS_TARGET_NOT_FOUND)
+                    self.error_ifarg(
+                        f'Trying to resolve command-line target={t_full}:',
+                        f'was not found in deps_file={self.rel_deps_file}',
+                        arg='error-unknown-args',
+                        error_code=EDA_DEPS_TARGET_NOT_FOUND
+                    )
 
             else:
                 # If we have caller_info, then this was a recursive call from another
@@ -340,22 +352,31 @@ class DepsFile:
 
                 if '.' in target_node:
                     # Likely a filename (target_node does not include path)
-                    self.error(f'Trying to resolve target={t_full} (file?):',
-                               f'called from {caller_info},',
-                               f'File={t_node} not found in directory={t_path}',
-                               error_code=EDA_DEPS_FILE_NOT_FOUND)
+                    self.error_ifarg(
+                        f'Trying to resolve target={t_full} (file?):',
+                        f'called from {caller_info},',
+                        f'File={t_node} not found in directory={t_path}',
+                        arg='error-unknown-args',
+                        error_code=EDA_DEPS_FILE_NOT_FOUND
+                    )
                 elif not self.rel_deps_file:
                     # target, but there's no DEPS file
-                    self.error(f'Trying to resolve target={t_full}:',
-                               f'called from {caller_info},',
-                               f'but {t_path} has no DEPS markup file (DEPS.yml)',
-                               error_code=EDA_DEPS_FILE_NOT_FOUND)
+                    self.error_ifarg(
+                        f'Trying to resolve target={t_full}:',
+                        f'called from {caller_info},',
+                        f'but {t_path} has no DEPS markup file (DEPS.yml)',
+                        arg='error-unknown-args',
+                        error_code=EDA_DEPS_FILE_NOT_FOUND
+                    )
                 else:
                     self.warning_show_available_targets()
-                    self.error(f'Trying to resolve target={t_full}:',
-                               f'called from {caller_info},',
-                               f'Target not found in deps_file={self.rel_deps_file}',
-                               error_code=EDA_DEPS_TARGET_NOT_FOUND)
+                    self.error_ifarg(
+                        f'Trying to resolve target={t_full}:',
+                        f'called from {caller_info},',
+                        f'Target not found in deps_file={self.rel_deps_file}',
+                        arg='error-unknown-args',
+                        error_code=EDA_DEPS_TARGET_NOT_FOUND
+                    )
         else:
             debug(f'Found {target_node=} in deps_file={self.rel_deps_file}')
             found_target = True

{opencos_eda-0.3.1 → opencos_eda-0.3.3}/opencos/eda_base.py

@@ -250,7 +250,9 @@ class Command: # pylint: disable=too-many-public-methods
                             ' --disable-tags has higher precedence than --enable-tags.'),
             'test-mode': ('command and tool dependent, usually stops the command early without'
                           ' executing.'),
-            'error-unknown-args': 'Enable errors on unknown/unparsable args',
+            'error-unknown-args': (
+                'Enable errors on unknown/unparsable args, or unknown/nonexistent files, or targets'
+            ),
         })
         self.modified_args = {}
         self.config = copy.deepcopy(config) # avoid external modifications.
@@ -497,9 +499,11 @@ class Command: # pylint: disable=too-many-public-methods
 
         # Do some minimal type handling, preserving the type(self.args[key])
         if key not in self.args:
-            self.error_unknown_arg(
+            self.error_ifarg(
                 f'set_arg, {key=} not in self.args {value=}',
-                f'({self.command_name=}, {self.__class__.__name__=})'
+                f'({self.command_name=}, {self.__class__.__name__=})',
+                arg='error-unknown-args',
+                error_code=status_constants.EDA_COMMAND_OR_ARGS_ERROR
             )
 
         cur_value = self.args[key]
@@ -695,9 +699,11 @@ class Command: # pylint: disable=too-many-public-methods
         _, unparsed = self.run_argparser_on_list(tokens)
         if process_all and unparsed:
             self.warning_show_known_args()
-            self.error_unknown_arg(
+            self.error_ifarg(
                 f"Didn't understand argument: '{unparsed=}' in",
-                f"{self.command_name=} context, {pwd=}"
+                f"{self.command_name=} context, {pwd=}",
+                arg='error-unknown-args',
+                error_code=status_constants.EDA_COMMAND_OR_ARGS_ERROR
             )
 
         return unparsed
@@ -914,13 +920,17 @@ class Command: # pylint: disable=too-many-public-methods
         lines.append(self.pretty_str_known_args(command=commands[-1])) # use last command if > 1
         util.warning("\n".join(lines))
 
-    def error_unknown_arg(self, *msg) -> None:
+    def error_ifarg(
+            self, *msg, arg: str, error_code: int = status_constants.EDA_COMMAND_OR_ARGS_ERROR
+    ) -> None:
         '''For errors involving an unknown --arg, they can be optionally disabled
 
-        using --no-error-unknown-args
+        using CLI: --no-error-unknown-args, and this method arg='error-uknown-arg'
+
+        Note if arg is not present in self.args, the error is enabled.
         '''
-        if self.args['error-unknown-args']:
-            self.error(*msg, error_code=status_constants.EDA_COMMAND_OR_ARGS_ERROR)
+        if self.args.get(arg, True):
+            self.error(*msg, error_code=error_code)
         else:
             util.warning(*msg)
 
@@ -1497,8 +1507,12 @@ class CommandDesign(Command): # pylint: disable=too-many-instance-attributes
                     found_target = True
                     break # move on to the next target
             if not found_target and error_on_not_found: # if STILL not found_this_target...
-                self.error(f"Unable to resolve {target=}",
-                           error_code=status_constants.EDA_DEPS_TARGET_NOT_FOUND)
+                # allow this if --no-error-unknown-args:
+                self.error_ifarg(
+                    f"Unable to resolve {target=}",
+                    arg='error-unknown-args',
+                    error_code=status_constants.EDA_DEPS_TARGET_NOT_FOUND
+                )
 
         # if we've found any target since being called, it means we found the one we were called for
         return found_target
@@ -1660,9 +1674,11 @@ class CommandDesign(Command): # pylint: disable=too-many-instance-attributes
         if process_all and possible_unparsed_args:
             _tool = self.safe_which_tool()
             self.warning_show_known_args()
-            self.error_unknown_arg(
+            self.error_ifarg(
                 f"Didn't understand unparsed args: {possible_unparsed_args}, for command",
-                f"'{self.command_name}', tool '{_tool}'"
+                f"'{self.command_name}', tool '{_tool}'",
+                arg='error-unknown-args',
+                error_code=status_constants.EDA_COMMAND_OR_ARGS_ERROR
             )
 
         remove_list = []
@@ -1721,9 +1737,11 @@ class CommandDesign(Command): # pylint: disable=too-many-instance-attributes
         # we were unable to figure out what this command line token is for...
         if process_all and unparsed:
             self.warning_show_known_args()
-            self.error_unknown_arg(
+            self.error_ifarg(
                 f"Didn't understand remaining args or targets {unparsed=} for command",
-                f"'{self.command_name}'"
+                f"'{self.command_name}'",
+                arg='error-unknown-args',
+                error_code=status_constants.EDA_COMMAND_OR_ARGS_ERROR
             )
 
         # handle a missing self.args['top'] with last filepath or last target:
@@ -2269,9 +2287,11 @@ class CommandParallel(Command):
         bad_remaining_args = [x for x in single_cmd_unparsed if x.startswith('-')]
         if bad_remaining_args:
             self.warning_show_known_args(command=f'{self.command_name} {command}')
-            self.error_unknown_arg(
+            self.error_ifarg(
                 f'for {self.command_name} {command=} the following args are unknown',
-                f'{bad_remaining_args}'
+                f'{bad_remaining_args}',
+                arg='error-unknown-args',
+                error_code=status_constants.EDA_COMMAND_OR_ARGS_ERROR
             )
 
         # Remove unparsed args starting with '+', since those are commonly sent downstream to

{opencos_eda-0.3.1 → opencos_eda-0.3.3}/opencos/eda_config_defaults.yml

@@ -145,6 +145,7 @@ tools:
       --timescale=1ns/1ns
       --allow-dup-initial-drivers
       -Wno-missing-top
+    compile-waivers: []
 
   verilator:
     defines: { }
@@ -153,11 +154,11 @@ tools:
       - "%Fatal"
     log-must-strings:
       - "R e p o r t: Verilator" # These come at end of compile or simulation, since we don't suppress it.
+    # Note: -j $(nproc) is handled in verilator.py, but you can override it in compile-args
     compile-args: |
       --timing
       --assert
       --autoflush
-      -j 2
       -sv
     compile-coverage-args: |
       --coverage

{opencos_eda-0.3.1 → opencos_eda-0.3.3}/opencos/tests/helpers.py

@@ -9,12 +9,14 @@ from pathlib import Path
 
 from contextlib import redirect_stdout, redirect_stderr
 
-from opencos import eda
-from opencos import deps_schema
+from opencos import eda, eda_tool_helper, deps_schema
 from opencos.utils.markup_helpers import yaml_safe_load
 from opencos.utils import status_constants
 from opencos.utils.subprocess_helpers import subprocess_run_background
 
+# Figure out what tools the system has available, without calling eda.main(..)
+config, tools_loaded = eda_tool_helper.get_config_and_tools_loaded()
+
 
 def eda_wrap_is_sim_fail(rc: int, quiet: bool = False) -> bool:
     '''Because eda_wrap calls eda_main(..) and will continue running
@@ -30,11 +32,11 @@ def eda_wrap_is_sim_fail(rc: int, quiet: bool = False) -> bool:
         status_constants.EDA_DEFAULT_ERROR
     )
 
-def can_run_eda_command(*commands, config: dict) -> bool:
+def can_run_eda_command(*commands, cfg: dict = config) -> bool:
     '''Returns True if we have any installed tool that can run: eda <command>'''
     runnable = []
     for command in list(commands):
-        handler = config['command_handler'].get(command, None)
+        handler = cfg['command_handler'].get(command, None)
         if not handler:
             return False
         if handler and getattr(handler, 'CHECK_REQUIRES', []):
@@ -44,7 +46,7 @@ def can_run_eda_command(*commands, config: dict) -> bool:
         # We cannot run tools that have disable-auto set:
         tool = getattr(handler, '_TOOL', '')
         if handler and tool:
-            entry = config['auto_tools_order'][0].get(tool, {})
+            entry = cfg['auto_tools_order'][0].get(tool, {})
             if entry and entry.get('disable-auto', False):
                 # This tool cannot automatically run our command.
                 return False
@@ -52,6 +54,14 @@ def can_run_eda_command(*commands, config: dict) -> bool:
         runnable.append(True)
     return runnable and all(runnable)
 
+def can_run_eda_sim(cfg: dict = config) -> bool:
+    '''Returns True if we have any installed tool that can run: eda sim'''
+    return can_run_eda_command('sim', cfg=cfg)
+
+def can_run_eda_elab(cfg: dict = config) -> bool:
+    '''Returns True if we have any installed tool that can run: eda elab'''
+    return can_run_eda_command('elab', cfg=cfg)
+
 def chdir_remove_work_dir(startpath, relpath):
     '''Changes dir to startpath/relpath, removes the work directories (eda.work, eda.export*)'''
     os.chdir(os.path.join(str(Path(startpath)), str(Path(relpath))))
@@ -232,7 +242,9 @@ class Helpers:
 
         return rc
 
-    def is_in_log(self, *want_str, logfile=None, windows_path_support=False):
+    def is_in_log(
+            self, *want_str, logfile=None, windows_path_support: bool = False
+    ) -> bool:
         '''Check if want_str (joined) is in the logfile, or self.DEFAULT_LOG'''
         logfile = self._resolve_logfile(logfile)
         want_str0 = ' '.join(list(want_str))
@@ -244,7 +256,9 @@ class Helpers:
                     return True
         return False
 
-    def get_log_lines_with(self, *want_str, logfile=None, windows_path_support=False):
+    def get_log_lines_with(
+            self, *want_str, logfile=None, windows_path_support: bool = False
+    ) -> list:
         '''gets all log lines with any of want_str args are in the logfile, or self.DEFAULT_LOG'''
         logfile = self._resolve_logfile(logfile)
         ret_list = []
@@ -257,6 +271,21 @@ class Helpers:
                     ret_list.append(line)
         return ret_list
 
+    def get_log_lines_with_all(
+            self, *want_str, logfile=None, windows_path_support: bool = False
+    ) -> list:
+        '''gets all log lines with ALL of want_str args are in the logfile, or self.DEFAULT_LOG'''
+        logfile = self._resolve_logfile(logfile)
+        ret_list = []
+        with open(logfile, encoding='utf-8') as f:
+            for line in f.readlines():
+                if all(x in line for x in list(want_str)):
+                    ret_list.append(line)
+                elif windows_path_support and \
+                     all(x.replace('/', '\\') in line for x in list(want_str)):
+                    ret_list.append(line)
+        return ret_list
+
     def get_log_words_with(self, *want_str, logfile=None, windows_path_support=False):
         '''gets all log words with any of *want_str within a single word
         in the logfile or self.DEFAULT_LOG

{opencos_eda-0.3.1 → opencos_eda-0.3.3}/opencos/tests/test_eda.py

@@ -23,11 +23,12 @@ import subprocess
 
 import pytest
 
-from opencos import eda, eda_tool_helper
+from opencos import eda
 from opencos.utils.markup_helpers import yaml_safe_load
 from opencos.tests import helpers
 from opencos.tests.helpers import eda_wrap, eda_sim_wrap, eda_elab_wrap, \
-    Helpers
+    Helpers, tools_loaded, can_run_eda_sim
+
 
 
 THISPATH = os.path.dirname(__file__)
@@ -36,17 +37,6 @@ def chdir_remove_work_dir(relpath):
     '''Changes dir to relpath, removes the work directories (eda.work, eda.export*)'''
     return helpers.chdir_remove_work_dir(THISPATH, relpath)
 
-# Figure out what tools the system has available, without calling eda.main(..)
-config, tools_loaded = eda_tool_helper.get_config_and_tools_loaded()
-
-def can_run_eda_sim() -> bool:
-    '''Returns True if we have any installed tool that can run: eda sim'''
-    return helpers.can_run_eda_command('sim', config=config)
-
-def can_run_eda_elab() -> bool:
-    '''Returns True if we have any installed tool that can run: eda elab'''
-    return helpers.can_run_eda_command('elab', config=config)
-
 @pytest.mark.skipif(
     'verilator' not in tools_loaded and 'vivado' not in tools_loaded,
     reason="requires verilator OR vivado"

{opencos_eda-0.3.1 → opencos_eda-0.3.3}/opencos/tests/test_tools.py

@@ -6,25 +6,23 @@ import os
 import sys
 import pytest
 
-from opencos import eda, eda_tool_helper, eda_base
+from opencos import eda, eda_base
 
 from opencos.tools.verilator import ToolVerilator
 from opencos.tools.vivado import ToolVivado
 from opencos.tools.cocotb import ToolCocotb
 from opencos.tests import helpers
-from opencos.tests.helpers import eda_wrap, eda_wrap_is_sim_fail
+from opencos.tests.helpers import eda_wrap, eda_wrap_is_sim_fail, config, tools_loaded
 from opencos.utils.markup_helpers import yaml_safe_load
 from opencos.utils import status_constants
 
 
-thispath = os.path.dirname(__file__)
+THISPATH = os.path.dirname(__file__)
 
 def chdir_remove_work_dir(relpath):
     '''Changes dir to relpath, removes the work directories (eda.work, eda.export*)'''
-    return helpers.chdir_remove_work_dir(thispath, relpath)
+    return helpers.chdir_remove_work_dir(THISPATH, relpath)
 
-# Figure out what tools the system has available, without calling eda.main(..)
-config, tools_loaded = eda_tool_helper.get_config_and_tools_loaded()
 
 def test_tools_loaded():
     '''Does not directly call 'eda.main' instead create a few Tool
@@ -37,7 +35,7 @@ def test_tools_loaded():
     # It's possible we're running in some container or install that has no tools, for example,
     # Windows.
     if sys.platform.startswith('win') and \
-       not helpers.can_run_eda_command('elab', 'sim', config=config):
+       not helpers.can_run_eda_command('elab', 'sim', cfg=config):
         # Windows, not handlers for elab or sim:
         pass
     else:
@@ -129,7 +127,7 @@ def test_sim_elab_tools_pass_or_fail(command, tool, target, sim_expect_pass, add
         added_args = added_sim_args_str.split()
 
     relative_dir = "deps_files/test_err_fatal"
-    os.chdir(os.path.join(thispath, relative_dir))
+    os.chdir(os.path.join(THISPATH, relative_dir))
     rc = eda.main(command, '--tool', tool, *(added_args), target)
     print(f'{rc=}')
     if command != 'sim' or sim_expect_pass:

{opencos_eda-0.3.1 → opencos_eda-0.3.3}/opencos/tools/slang.py

@@ -128,6 +128,7 @@ class CommandElabSlang(CommandElab, ToolSlang):
         command_list += self.tool_config.get('compile-args', '--single-unit').split()
         command_list += self.args['slang-args'] # add user args.
         command_list += self._get_slang_json_args(command_exe=command_list[0])
+        command_list += self._get_slang_tool_config_waivers()
 
         # incdirs
         for value in self.incdirs:
@@ -214,6 +215,11 @@ class CommandElabSlang(CommandElab, ToolSlang):
 
         return command_list
 
+    def _get_slang_tool_config_waivers(self) -> list:
+        # Add compile waivers from config and command-line args
+        return [f'-Wno-{waiver}' for waiver in
+                self.tool_config.get('compile-waivers', []) + self.args['compile-waivers']]
+
 
 class CommandLintSlang(CommandElabSlang):
     '''CommandLintSlang is a command handler for: eda lint --tool=slang.'''

{opencos_eda-0.3.1 → opencos_eda-0.3.3}/opencos/tools/verilator.py

@@ -5,10 +5,12 @@ Contains classes for ToolVerilator and VerilatorSim, VerilatorElab.
 
 # pylint: disable=R0801 # (calling functions with same arguments)
 
+import multiprocessing
 import os
 import shutil
 import subprocess
 
+
 from opencos import util
 from opencos.eda_base import Tool
 from opencos.commands import CommandSim
@@ -210,15 +212,9 @@ class VerilatorSim(CommandSim, ToolVerilator):
 
         verilate_command_list = self._get_start_verilator_command_list(lint_only=lint_only)
 
-        # Add compile args from our self.config (tools.verilator.compile-args str)
-        config_compile_args = self.tool_config.get(
-            'compile-args',
-            '--timing --assert --autoflush -sv').split()
-        verilate_command_list += config_compile_args
-
         verilate_command_list += self._get_verilator_tool_config_waivers()
 
-        verilate_command_list += self._verilator_args_deal_with_cflags()
+        verilate_command_list += self._verilator_args_defaults_cflags_nproc()
 
         verilate_command_list += self._get_verilator_waves_args(lint_only=lint_only)
 
@@ -425,45 +421,51 @@ class VerilatorSim(CommandSim, ToolVerilator):
         return ret
 
 
-    def _verilator_args_deal_with_cflags(self) -> list:
+    def _verilator_args_defaults_cflags_nproc(self) -> list:
         '''Returns list of args to be added to verilator (compile) step
 
-        Uses self.args['verilate-args'] and self.args['compile-args']
+        Uses self.args['verilate-args'], self.args['compile-args'], and self.tool_config
+
+        Sets -j <value> and -CFLAGS -O<value> if not present in --config-yml, --compile-args,
+        or --verilate-args. If present, chooses the first instance (does not present duplicates
+        to verilator call).
         '''
 
-        # We can only support one -CFLAGS followed by one -O[0-9] arg in self.args['verilate-args']:
-        verilate_cflags_args_dict = {}
-        verilate_args = [] # will be combined verilate_args + compile-args
-        prev_arg_is_cflags = False
+        # We can only support one -CFLAGS followed by one -O[0-9] arg in
+        # --verilate-args or --compile-args.
+
+        # Add compile args from our self.config (tools.verilator.compile-args str)
+        verilate_args = self.args['verilate-args'] + \
+            self.args['compile-args'] + \
+            self.tool_config.get(
+                'compile-args',
+                '--timing --assert --autoflush -sv').split()
+
         util.debug(f"{self.args['verilate-args']=}")
         util.debug(f"{self.args['compile-args']=}")
-        for arg in self.args['verilate-args'] + self.args['compile-args']:
-            # pick the first ones we see of these:
-            if arg == '-CFLAGS':
-                prev_arg_is_cflags = True
-                if arg not in verilate_cflags_args_dict:
-                    # We can only have 1
-                    verilate_cflags_args_dict[arg] = True
-                    verilate_args.append(arg)
-                else:
-                    util.debug(f'Previous saw -CFLAGS args {verilate_cflags_args_dict=},',
-                               f'skipping new {arg=}')
-
-            elif arg.startswith('-O') and len(arg) == 3:
-                if '-O' not in verilate_cflags_args_dict and prev_arg_is_cflags:
-                    # We can only have 1
-                    verilate_cflags_args_dict['-O'] = arg[-1]
-                    verilate_args.append(arg)
-                else:
-                    util.debug(f'Previous saw -CFLAGS args {verilate_cflags_args_dict=},',
-                               f'skipping new {arg=}')
-                prev_arg_is_cflags = False
-
-            else:
-                prev_arg_is_cflags = False
-                verilate_args.append(arg)
 
-        if '-CFLAGS' in verilate_args:
+        dash_j_arg_indices = []
+        cflags_dasho_args_indices = []
+        for i, arg in enumerate(list(verilate_args)):
+            # There can only be one of these: -j <value>, similarly can only be one of
+            # -CFLAGS -O<value>
+            if (i + 1) < len(verilate_args):
+                if arg == '-j':
+                    dash_j_arg_indices.extend([i, i + 1])
+                if arg == '-CFLAGS':
+                    next_arg = verilate_args[i + 1]
+                    if next_arg.startswith('-O') and len(next_arg) == 3:
+                        cflags_dasho_args_indices.extend([i, i + 1])
+
+        # For -j <value> we'll pick the first one, remove the rest.
+        # Same goes for -CFLAGS -O<value>
+        for index in dash_j_arg_indices[2:] + cflags_dasho_args_indices[2:]:
+            verilate_args[index] = ''
+
+        verilate_args = [x for x in verilate_args if x != ''] # strip empty str.
+
+        # Support for --optimize which will use -CFLAGS -O3, if -CFLAGS is not present at all.
+        if cflags_dasho_args_indices:
             # add whatever args were passed via 'compile-args' or 'verilate_args'. Note these will
             # take precedence over the --optimize arg.
             pass
@@ -472,8 +474,15 @@ class VerilatorSim(CommandSim, ToolVerilator):
             # (slower compile, better runtime)
             verilate_args += '-CFLAGS', '-O3'
         else:
+            # Default to -O1:
             verilate_args += '-CFLAGS', '-O1'
 
+        # If there was no -j setting, then use max(2, $(nproc) - 1)
+        if not dash_j_arg_indices:
+            nproc = max(2, multiprocessing.cpu_count() - 1)
+            verilate_args += '-j', f'{nproc}'
+
+
         return verilate_args
 
 

{opencos_eda-0.3.1 → opencos_eda-0.3.3}/opencos/util.py

@@ -18,6 +18,7 @@ from enum import Enum
 from pathlib import Path
 from importlib import import_module
 from dotenv import load_dotenv
+from supports_color import supportsColor
 
 from opencos.utils import status_constants
 
@@ -29,7 +30,7 @@ dot_f_files_expanded = set() # pylint: disable=invalid-name
 env_files_loaded = set()     # pylint: disable=invalid-name
 
 args = {                    # pylint: disable=invalid-name
-    'color' : False,
+    'color' : bool(supportsColor.stdout),
     'quiet' : False,
     'verbose' : False,
     'debug' : False,
@@ -351,13 +352,14 @@ def get_argparser() -> argparse.ArgumentParser:
     bool_action_kwargs = get_argparse_bool_action_kwargs()
 
     parser.add_argument('--version', default=False, action='store_true')
-    parser.add_argument('--color', **bool_action_kwargs, default=True,
+    parser.add_argument('--color', **bool_action_kwargs, default=bool(supportsColor.stdout),
                         help='Use shell colors for info/warning/error messaging')
-    parser.add_argument('--quiet', **bool_action_kwargs, help='Do not display info messaging')
-    parser.add_argument('--verbose', **bool_action_kwargs,
+    parser.add_argument('--quiet', **bool_action_kwargs, default=args['quiet'],
+                        help='Do not display info messaging')
+    parser.add_argument('--verbose', **bool_action_kwargs, default=args['verbose'],
                         help='Display additional messaging level 2 or higher')
-    parser.add_argument('--fancy', **bool_action_kwargs)
-    parser.add_argument('--debug', **bool_action_kwargs,
+    parser.add_argument('--fancy', **bool_action_kwargs, default=args['fancy'])
+    parser.add_argument('--debug', **bool_action_kwargs, default=args['debug'],
                         help='Display additional debug messaging level 1 or higher')
     parser.add_argument('--debug-level', type=int, default=0,
                         help='Set debug level messaging (default: 0)')

{opencos_eda-0.3.1 → opencos_eda-0.3.3/opencos_eda.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: opencos-eda
-Version: 0.3.1
+Version: 0.3.3
 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
@@ -17,4 +17,5 @@ Requires-Dist: toml>=0.10.2
 Requires-Dist: yamllint>=1.35.1
 Requires-Dist: PySerial>=3.5
 Requires-Dist: cocotb>=2.0
+Requires-Dist: supports_color>=0.2.0
 Dynamic: license-file

{opencos_eda-0.3.1 → opencos_eda-0.3.3}/opencos_eda.egg-info/requires.txt

@@ -8,3 +8,4 @@ toml>=0.10.2
 yamllint>=1.35.1
 PySerial>=3.5
 cocotb>=2.0
+supports_color>=0.2.0

{opencos_eda-0.3.1 → opencos_eda-0.3.3}/pyproject.toml

@@ -2,7 +2,7 @@
 
 [project]
 name = "opencos-eda"
-version = "0.3.1"
+version = "0.3.3"
 dependencies = [
   # opencos/eda.py dependencies
   "mergedeep >= 1.3.4",
@@ -16,6 +16,7 @@ dependencies = [
   # opencos/oc_cli.py dependencies
   "PySerial >= 3.5",
   "cocotb >= 2.0",
+  "supports_color >= 0.2.0",
 ]
 requires-python = ">= 3.8"
 authors = [