From 692b20f1b2ec619ee9471ee070bc6b43c17284ee Mon Sep 17 00:00:00 2001
From: Jakob Unterwurzacher
Date: Sun, 30 Apr 2017 13:14:54 +0200
Subject: golint.bash: fix exit codes

We want to exit with 1 only if we we have complaints left after
the greps.
---
 golint.bash | 23 +++++++++++++++++------
 1 file changed, 17 insertions(+), 6 deletions(-)

(limited to 'golint.bash')

diff --git a/golint.bash b/golint.bash
index 6da72c8..5040c12 100755
--- a/golint.bash
+++ b/golint.bash
@@ -1,7 +1,18 @@
-#!/bin/bash
+#!/bin/bash -u
 
-golint ./... | \
- grep -v "don't use an underscore in package name" | \
- grep -v "don't use ALL_CAPS in Go names; use CamelCase" |
- grep -v "struct field allow_other should be allowOther" |
- grep -v "struct field serialize_reads should be serializeReads"
+OUTPUT=$(
+	golint ./... | \
+	 grep -v "don't use an underscore in package name" | \
+	 grep -v "don't use ALL_CAPS in Go names; use CamelCase" |
+	 grep -v "struct field allow_other should be allowOther" |
+	 grep -v "struct field serialize_reads should be serializeReads"
+)
+
+# No output --> all good
+if [[ -z "$OUTPUT" ]] ; then
+	exit 0
+fi
+
+echo "golint.bash:"
+echo $OUTPUT
+exit 1
-- 
cgit v1.2.3