From 425c3491846461f7ce6aaadc558d3fa0b3adf050 Mon Sep 17 00:00:00 2001 From: JunghwanNA <70629228+shaun0927@users.noreply.github.com> Date: Thu, 16 Apr 2026 12:26:50 +0900 Subject: [PATCH] fix(ci): use PATH-based iverilog/vvp discovery for cross-layer tests The default IVERILOG and VVP paths were hardcoded to macOS Homebrew locations (/opt/homebrew/bin/iverilog). On Ubuntu CI runners, apt installs iverilog to /usr/bin/, so the Path.exists() check returns False and all Tier 2 Verilog cosim tests are silently skipped. Change defaults to bare command names so the existing which-based fallback at line 57-58 discovers the binary via PATH on any platform. --- 9_Firmware/tests/cross_layer/test_cross_layer_contract.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/9_Firmware/tests/cross_layer/test_cross_layer_contract.py b/9_Firmware/tests/cross_layer/test_cross_layer_contract.py index c3b950e..b24d2a3 100644 --- a/9_Firmware/tests/cross_layer/test_cross_layer_contract.py +++ b/9_Firmware/tests/cross_layer/test_cross_layer_contract.py @@ -49,8 +49,8 @@ sys.path.insert(0, str(cp.GUI_DIR)) # Helpers # =================================================================== -IVERILOG = os.environ.get("IVERILOG", "/opt/homebrew/bin/iverilog") -VVP = os.environ.get("VVP", "/opt/homebrew/bin/vvp") +IVERILOG = os.environ.get("IVERILOG", "iverilog") +VVP = os.environ.get("VVP", "vvp") CXX = os.environ.get("CXX", "c++") # Check tool availability for conditional skipping