back.verilog: add workaround for evaluation Verific behavior.
authorTeguh Hofstee <5227572+hofstee@users.noreply.github.com>
Thu, 23 Apr 2020 21:46:10 +0000 (14:46 -0700)
committerGitHub <noreply@github.com>
Thu, 23 Apr 2020 21:46:10 +0000 (21:46 +0000)
The evaluation version of Verific prints its license information to stdout,
and since it is against the EULA to change that in any way, this behavior
is not possible to fix in Yosys. Add a workaround in nMigen instead.

nmigen/back/verilog.py

index 6e2185abcdbe8a0df63b12f23d60ed497ab22502..04e19b6a682b3da1c59be4a7f532d6face510b90 100644 (file)
@@ -1,6 +1,7 @@
 import os
 import re
 import subprocess
+import itertools
 
 from .._toolchain import *
 from . import rtlil
@@ -17,6 +18,7 @@ def _yosys_version():
     yosys_path = require_tool("yosys")
     version = subprocess.check_output([yosys_path, "-V"], encoding="utf-8")
     # If Yosys is built with Verific, then Verific license information is printed first.
+    # See below for details.
     m = re.search(r"^Yosys ([\d.]+)(?:\+(\d+))?", version, flags=re.M)
     tag, offset = m[1], m[2] or 0
     return tuple(map(int, tag.split("."))), offset
@@ -65,6 +67,13 @@ write_verilog -norename {write_verilog_opts}
     if popen.returncode:
         raise YosysError(error.strip())
     else:
+        # If Yosys is built with an evaluation version of Verific, then Verific license information 
+        # is printed first. It consists of empty lines and lines starting with `--`, which are not
+        # valid at the start of a Verilog file, and thus may be reliably removed.
+        verilog_text = "\n".join(itertools.dropwhile(
+            lambda x: x == "" or x.startswith("--"),
+            verilog_text.splitlines()
+        ))
         return verilog_text