Skip to content

Commit 0c3d6e7

Browse files
committed
versions.txt: nicer output
if YOSYS_EXE is empty, log it with iffy code. This avoids -V being passed to bash and strange error messages appearing. If YOSYS_EXE is non-empty, log error silently into versions.txt as versions.txt goes into the make issue report and the actual output from this command will not be part of the make issue Signed-off-by: Øyvind Harboe <[email protected]>
1 parent b81e030 commit 0c3d6e7

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

flow/Makefile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -184,9 +184,9 @@ $(foreach block,$(BLOCKS),$(eval $(WORK_HOME)/results/$(PLATFORM)/$(DESIGN_NICKN
184184
.PHONY: versions.txt
185185
versions.txt:
186186
mkdir -p $(OBJECTS_DIR)
187-
@echo "yosys $(shell $(YOSYS_EXE) -V 2>&1)" > $(OBJECTS_DIR)/$@
188-
@echo "openroad $(shell $(OPENROAD_EXE) -version 2>&1)" >> $(OBJECTS_DIR)/$@
189-
@echo "klayout $(shell $(KLAYOUT_CMD) -zz -v 2>&1)" >> $(OBJECTS_DIR)/$@
187+
@echo "yosys $(if $(YOSYS_EXE),$(shell $(YOSYS_EXE) -V 2>&1),not available)" > $(OBJECTS_DIR)/$@
188+
@echo "openroad $(if $(OPENROAD_EXE),$(shell $(OPENROAD_EXE) -version 2>&1),not available)" >> $(OBJECTS_DIR)/$@
189+
@echo "klayout $(if $(KLAYOUT_CMD),$(shell $(KLAYOUT_CMD) -zz -v 2>&1),not available)" >> $(OBJECTS_DIR)/$@
190190

191191
# Pre-process libraries
192192
# ==============================================================================

0 commit comments

Comments
 (0)