From a460c3ca2c0c4d1d3e62b60c1fa04b9f5c66b84f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Max=20K=C3=B6hler?= <max.koehler@ni.com>
Date: Tue, 24 Sep 2024 01:36:14 +0200
Subject: [PATCH] fpga: tools: add diamond build sanity checks

Original-commit: aa10241c7f034739da8905f862beaf0f31d286ed
---
 tools/make/diamond_design_builder.mak | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/tools/make/diamond_design_builder.mak b/tools/make/diamond_design_builder.mak
index abd8748..fb55eb7 100644
--- a/tools/make/diamond_design_builder.mak
+++ b/tools/make/diamond_design_builder.mak
@@ -33,9 +33,18 @@ BUILD_DIAMOND_DESIGN = \
 	cd $(4); \
 	echo "BUILDER: Implementating design..."; \
 	$(DIAMOND_EXE) build.tcl > $(1)_log.txt ; \
+	if [ $$? -ne 0 ]; \
+	then \
+		echo "BUILDER: Build failed. See $(1)_log.txt for details."; \
+		exit 1; \
+	fi; \
 	echo "BUILDER: Parsing reports..."; \
-	grep "Cumulative negative slack: 0 (0+0)" impl1/$(1)_impl1.twr ; \
-		if [ $$? -ne 0 ]; then exit 1; fi; \
+	grep "Cumulative negative slack: 0 (0+0)" impl1/$(1)_impl1.twr > /dev/null 2>&1; \
+		if [ $$? -ne 0 ]; \
+		then \
+			echo "BUILDER: Timing failed. See $(1)_impl1.twr for details."; \
+			exit 1; \
+		fi; \
 	echo "BUILDER: Generating bitfile..."; \
 	ddtcmd -oft -svfsingle -if $(5)/$(1)_$(5).jed			 	\
 		-dev $(2) -op "FLASH Erase,Program,Verify" -revd	\