From 45cdecdea9c5368f419e7bdcbd78a669ce88a47a Mon Sep 17 00:00:00 2001
From: mrg <mrg@ucsc.edu>
Date: Tue, 3 Nov 2020 15:47:04 -0800
Subject: [PATCH] Improve error message about missing DRC/LVS tools.

---
 compiler/globals.py         |  7 +++----
 compiler/verify/__init__.py | 10 ++++++----
 2 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/compiler/globals.py b/compiler/globals.py
index 32137437..5d553175 100644
--- a/compiler/globals.py
+++ b/compiler/globals.py
@@ -257,8 +257,8 @@ def get_tool(tool_type, preferences, default_name=None):
     if default_name:
         exe_name = find_exe(default_name)
         if exe_name == None:
-            debug.error("{0} not found. Cannot find {1} tool.".format(default_name,
-                                                                      tool_type),
+            debug.error("{0} not found. Cannot find {1} tool.".format(default_name, tool_type)
+                        + "Disable DRC/LVS with check_lvsdrc=False to ignore.",
                         2)
         else:
             debug.info(1, "Using {0}: {1}".format(tool_type, exe_name))
@@ -271,8 +271,7 @@ def get_tool(tool_type, preferences, default_name=None):
                 return(name, exe_name)
             else:
                 debug.info(1,
-                           "Could not find {0}, trying next {1} tool.".format(name,
-                                                                              tool_type))
+                           "Could not find {0}, trying next {1} tool.".format(name, tool_type))
         else:
             return(None, "")
 
diff --git a/compiler/verify/__init__.py b/compiler/verify/__init__.py
index 4bcf93cc..8da102aa 100644
--- a/compiler/verify/__init__.py
+++ b/compiler/verify/__init__.py
@@ -15,7 +15,6 @@ run_pex, repsectively. If there is an error, they should abort and report the er
 If not, OpenRAM will continue as if nothing happened!
 """
 
-import os
 import debug
 from globals import OPTS
 from globals import get_tool
@@ -49,7 +48,8 @@ elif "assura"==OPTS.drc_exe[0]:
 elif "magic"==OPTS.drc_exe[0]:
     from .magic import run_drc, print_drc_stats
 else:
-    debug.warning("Did not find a supported DRC tool.")
+    debug.error("Did not find a supported DRC tool."
+                + "Disable DRC/LVS with check_lvsdrc=False to ignore.", 2)
 
 if not OPTS.lvs_exe:
     from .none import run_lvs, print_lvs_stats
@@ -60,7 +60,8 @@ elif "assura"==OPTS.lvs_exe[0]:
 elif "netgen"==OPTS.lvs_exe[0]:
     from .magic import run_lvs, print_lvs_stats
 else:
-    debug.warning("Did not find a supported LVS tool.")
+    debug.warning("Did not find a supported LVS tool."
+                  + "Disable DRC/LVS with check_lvsdrc=False to ignore.", 2)
 
 
 if not OPTS.pex_exe:
@@ -70,7 +71,8 @@ elif "calibre"==OPTS.pex_exe[0]:
 elif "magic"==OPTS.pex_exe[0]:
     from .magic import run_pex,print_pex_stats
 else:
-    debug.warning("Did not find a supported PEX tool.")
+    debug.warning("Did not find a supported PEX tool."
+                  + "Disable DRC/LVS with check_lvsdrc=False to ignore.", 2)
 
 if OPTS.tech_name == "sky130":
     if OPTS.magic_exe and "magic"==OPTS.magic_exe[0]: