Merge pull request #84 from riscv/83-improve-passfail-message-of-norm-rule-test-makefile

Clear pass/fail messages in makefile

Author: james-ball-qualcomm

.pre-commit-config.yaml

@@ -8,7 +8,7 @@ repos:
       - id: end-of-file-fixer
         exclude: |
           \.svg$|
-          ^schemas/.*\.json$
+          \.json$
       - id: trailing-whitespace
         args: [--markdown-linebreak-ext=md]
         exclude: \.svg$
@@ -17,4 +17,4 @@ repos:
     rev: 0.2.3
     hooks:
       - id: yamlfmt
-        args: [--mapping, '2', --sequence, '4', --offset, '2']
+        args: [--mapping, '2', --sequence, '4', --offset, '2', --preserve-quotes]

Makefile

@@ -124,16 +124,12 @@ build-test-norm-rules: $(BUILT_NORM_RULES)
 compare-tests: compare-test-tags compare-test-norm-rules
 
 compare-test-tags: $(EXPECTED_NORM_TAGS) $(BUILT_NORM_TAGS)
-	@echo XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
-	@echo Comparing $(EXPECTED_NORM_TAGS) to $(BUILT_NORM_TAGS)
-	@echo XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
-	diff $(EXPECTED_NORM_TAGS) $(BUILT_NORM_TAGS)
+	@echo "CHECKING BUILT TAGS AGAINST EXPECTED TAGS"
+	diff $(EXPECTED_NORM_TAGS) $(BUILT_NORM_TAGS) && echo "diff PASSED" || (echo "diff FAILED"; exit 1)
 
 compare-test-norm-rules: $(EXPECTED_NORM_RULES) $(BUILT_NORM_RULES)
-	@echo XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
-	@echo Comparing $(EXPECTED_NORM_RULES) to $(BUILT_NORM_RULES)
-	@echo XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
-	diff $(EXPECTED_NORM_RULES) $(BUILT_NORM_RULES)
+	@echo "CHECKING BUILT NORM RULES AGAINST EXPECTED NORM RULES"
+	diff $(EXPECTED_NORM_RULES) $(BUILT_NORM_RULES) && echo "diff PASSED" || (echo "diff FAILED"; exit 1)
 
 # Update expected files from built files
 .PHONY: update-expected