SimplHDL 0.5.4__tar.gz → 0.5.5__tar.gz

{simplhdl-0.5.4 → simplhdl-0.5.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: SimplHDL
-Version: 0.5.4
+Version: 0.5.5
 Summary: A framework for simulating and implementing HDL designs
 Author-email: Rasmus Grøndahl Olsen <simplhdl@gmail.com>
 Maintainer-email: Rasmus Grøndahl Olsen <simplhdl@gmail.com>

{simplhdl-0.5.4 → simplhdl-0.5.5}/src/SimplHDL.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: SimplHDL
-Version: 0.5.4
+Version: 0.5.5
 Summary: A framework for simulating and implementing HDL designs
 Author-email: Rasmus Grøndahl Olsen <simplhdl@gmail.com>
 Maintainer-email: Rasmus Grøndahl Olsen <simplhdl@gmail.com>

simplhdl-0.5.5/src/simplhdl/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.5.5"

{simplhdl-0.5.4 → simplhdl-0.5.5}/src/simplhdl/__main__.py

@@ -5,8 +5,9 @@ import logging
 import traceback
 
 from typing import Sequence
-
 from pathlib import Path
+from rich.logging import RichHandler
+
 from . import __version__
 from .simplhdl import Simplhdl
 from .plugins import load_plugins
@@ -67,7 +68,18 @@ def main():
         args = parse_arguments()
         levels = [logging.INFO, logging.DEBUG, logging.NOTSET]
         level = levels[min(args.verbose, len(levels)-1)]
-        logging.basicConfig(level=level)
+        if level < logging.INFO:
+            show_path = True
+        else:
+            show_path = False
+
+        FORMAT = "[simplhdl.%(module)s] - %(message)s"
+        console = RichHandler(level=level, show_time=False, rich_tracebacks=True, show_path=show_path)
+        logging.basicConfig(
+            level=5,
+            format=FORMAT,
+            handlers=[console]
+        )
         simpl = Simplhdl(args)
         simpl.run()
     except (NotImplementedError, FileNotFoundError, CalledShError,

{simplhdl-0.5.4 → simplhdl-0.5.5}/src/simplhdl/cocotb.py

@@ -41,7 +41,7 @@ class Cocotb:
             return None
         set_ = set()
         try:
-            modules = self.project.DefaultDesign._topLevel.split()
+            modules = self.project.DefaultDesign.TopLevel.split()
         except AttributeError:
             raise FlowError("No top levels found")
 
@@ -58,14 +58,14 @@ class Cocotb:
 
     def hdltoplevels(self) -> str:
         tops = []
-        for t in self.project.DefaultDesign._topLevel.split():
+        for t in self.project.DefaultDesign.TopLevel.split():
             if t != self.top:
                 tops.append(t)
         return ' '.join(tops)
 
     def get_dut(self) -> str:
         try:
-            for top in self.project.DefaultDesign._topLevel.split():
+            for top in self.project.DefaultDesign.TopLevel.split():
                 if top != self.top:
                     return top
         except AttributeError:
@@ -97,7 +97,7 @@ class Cocotb:
                     with open(file.Path, 'r', errors='replace') as f:
                         lines = f.readlines()
                         for line in lines:
-                            if re.search(rf'^\s*module\s+{name}(\s+|\(|;|$)', line):
+                            if re.search(rf'^\s*module\s+{name}(\s+|#\(|\(|;|$)', line):
                                 if file.Library.Name == lib:
                                     logger.info(f"Cocotb dut '{self.dut}' is Verilog")
                                     return VerilogSourceFile

{simplhdl-0.5.4 → simplhdl-0.5.5}/src/simplhdl/flow.py

@@ -1,3 +1,5 @@
+import logging
+
 from argparse import Namespace
 from pathlib import Path
 from typing import Callable, Dict, Set
@@ -16,6 +18,11 @@ class FlowBase(metaclass=ABCMeta):
         self.builddir: Path = builddir
         self.category: FlowCategory = FlowCategory.DEFAULT
         self.tools: Set = set()
+        self.builddir.mkdir(parents=True, exist_ok=True)
+        file = logging.FileHandler(self.builddir.joinpath('simplhdl.log'), mode='w')
+        file.setLevel(logging.NOTSET)
+        file.setFormatter(logging.Formatter('%(asctime)s - %(levelname)s:%(filename)s:%(lineno)d - %(message)s'))
+        logging.getLogger().addHandler(file)
 
     @classmethod
     @abstractmethod

{simplhdl-0.5.4 → simplhdl-0.5.5}/src/simplhdl/flows/flake8/flake8flow.py

@@ -49,7 +49,7 @@ class Flake8Flow(FlowBase):
             self.files = [f.Path for f in self.project.DefaultDesign.DefaultFileSet.Files(PythonSourceFile)]
 
     def generate(self):
-        config = os.getenv('SIMPLHDL_FLAKE8_RULES')
+        config = os.getenv('SIMPLHDL_FLAKE8_CONFIG')
         if config:
             shutil.copy(config, self.builddir.joinpath('setup.cfg'))
         else:

{simplhdl-0.5.4 → simplhdl-0.5.5}/src/simplhdl/flows/verible/veribleflow.py

@@ -8,7 +8,7 @@ from argparse import Namespace
 from pathlib import Path
 from jinja2 import Environment, FileSystemLoader
 
-from simplhdl.flow import FlowFactory, FlowBase
+from simplhdl.flow import FlowFactory, FlowBase, FlowError
 from simplhdl.resources.templates import verible as templates
 from simplhdl.utils import sh, CalledShError, generate_from_template
 from simplhdl.pyedaa import VerilogIncludeFile, VerilogSourceFile, SystemVerilogSourceFile
@@ -51,8 +51,11 @@ class VeribleFlow(FlowBase):
     def generate(self):
         if self.args.rules:
             self.rules = self.args.rules
-        elif os.getenv('SIMPLHDL_VERIBLE_RULES'):
-            self.rules = self.args.rules
+        elif os.getenv('SIMPLHDL_VERIBLE_CONFIG'):
+            rules = os.getenv('SIMPLHDL_VERIBLE_CONFIG')
+            if not Path(rules).exists():
+                raise FlowError(f"Rules file {rules} does not exist")
+            self.rules = rules
         else:
             templatedir = resources_files(self.templates)
             environment = Environment(
@@ -67,7 +70,7 @@ class VeribleFlow(FlowBase):
         command = [
             "verible-verilog-lint",
             "--check_syntax=false",
-            "--ruleset=all",
+            "--ruleset=none",
             "--rules_config", self.rules
         ]
 

{simplhdl-0.5.4 → simplhdl-0.5.5}/src/simplhdl/flows/vsg/vsgflow.py

@@ -74,8 +74,11 @@ class VsgFlow(FlowBase):
                                UsedIn=UsedIn)
         if self.args.rules:
             self.rules = self.args.rules
-        elif os.getenv('SIMPLHDL_VSG_RULES'):
-            self.rules = self.args.rules
+        elif os.getenv('SIMPLHDL_VSG_CONFIG'):
+            rules = os.getenv('SIMPLHDL_VSG_CONFIG')
+            if not Path(rules).exists():
+                raise FlowError(f"Rules file {rules} does not exist")
+            self.rules = rules
         else:
             template = environment.get_template('rules.yml.j2')
             generate_from_template(template, self.builddir)

{simplhdl-0.5.4 → simplhdl-0.5.5}/src/simplhdl/generators/spd.py

@@ -290,14 +290,14 @@ def copy_ipfile(file: QuartusIPSpecificationFile, dest: Path) -> QuartusIPSpecif
     destfile = dest.joinpath(file.Path.name)
     if srcdir.exists():
         if not md5file.exists() or not md5check(file.Path, srcdir, filename=md5file):
-            logger.info(f"Copy {file.Path} to {destfile}")
-            logger.info(f"Copy {srcdir} to {destdir}")
+            logger.debug(f"Copy {file.Path} to {destfile}")
+            logger.debug(f"Copy {srcdir} to {destdir}")
             copy(str(file.Path), str(destfile))
             copytree(str(srcdir), str(destdir), dirs_exist_ok=True)
             md5write(file.Path, srcdir, filename=md5file)
     else:
         if not md5file.exists() or not md5check(file.Path, filename=md5file):
-            logger.info(f"Copy {file.Path} to {destfile}")
+            logger.debug(f"Copy {file.Path} to {destfile}")
             rmtree(destdir, ignore_errors=True)
             copy(str(file.Path), str(destfile))
             md5write(file.Path, filename=md5file)

{simplhdl-0.5.4 → simplhdl-0.5.5}/src/simplhdl/pyedaa/design.py

@@ -39,15 +39,16 @@ class Design(pm.Design):
     @property
     def TopLevel(self) -> str:
         """Property setting or returning the fileset's toplevel."""
-        top = set()
+        top = list()
         if self._topLevel is not None:
             for t in self._topLevel.split():
-                top.add(t)
+                top.append(t)
         for fileset in self._fileSets.values():
             if fileset.TopLevel is not None:
                 for t in fileset.TopLevel.split():
-                    top.add(t)
-        return ' '.join(top)
+                    top.append(t)
+        # remove duplicates without changing the order
+        return ' '.join(list(dict.fromkeys(top)))
 
     @TopLevel.setter
     def TopLevel(self, value: str) -> None:

{simplhdl-0.5.4 → simplhdl-0.5.5}/src/simplhdl/utils.py

@@ -17,7 +17,7 @@ class CalledShError(Exception):
     pass
 
 
-def sh(command: List[str], cwd: Optional[Path] = None, output=False, shell=False, env=None):
+def sh(command: List[str], cwd: Optional[Path] = None, output=False, shell=False, env=None, indent=2):
     if os.name == 'nt':
         shell = True
 
@@ -26,7 +26,7 @@ def sh(command: List[str], cwd: Optional[Path] = None, output=False, shell=False
         if output:
             stdout: str = ''
             for line in p.stdout:
-                sys.stdout.buffer.write(line)
+                sys.stdout.buffer.write(b' '*indent + line)
                 sys.stdout.buffer.flush()
                 stdout += line.decode()
             _, stderr = p.communicate()

simplhdl-0.5.4/src/simplhdl/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.5.4"