From d4d99deb97034aa7122a8aec2059555cff189da8 Mon Sep 17 00:00:00 2001
From: Mike Danese <mikedanese@gmail.com>
Date: Wed, 15 Jul 2015 09:49:21 -0700
Subject: [PATCH] make mungedoc exit 1 if manual changes are needed and wire up
 erro message.

---
 cmd/mungedocs/mungedocs.go | 10 +++++++---
 hack/run-gendocs.sh        | 10 ++++------
 hack/verify-gendocs.sh     | 16 ++++++----------
 3 files changed, 17 insertions(+), 19 deletions(-)

diff --git a/cmd/mungedocs/mungedocs.go b/cmd/mungedocs/mungedocs.go
index 410ac1ca5b6..e452b0d2292 100644
--- a/cmd/mungedocs/mungedocs.go
+++ b/cmd/mungedocs/mungedocs.go
@@ -187,7 +187,7 @@ func main() {
 	// - If verify is true: exit 0 if no changes needed, exit 1 if changes
 	//   needed.
 	// - If verify is false: exit 0 if changes successfully made or no
-	//   changes needed.
+	//   changes needed, exit 1 if manual changes are needed.
 	var changesNeeded bool
 
 	err := filepath.Walk(leaf, newWalkFunc(&fp, &changesNeeded))
@@ -195,8 +195,12 @@ func main() {
 		fmt.Fprintf(os.Stderr, "ERROR: %v\n", err)
 		os.Exit(2)
 	}
-	if changesNeeded && *verify {
-		fmt.Fprintf(os.Stderr, "FAIL: changes needed but not made due to --verify\n")
+	if changesNeeded {
+		if *verify {
+			fmt.Fprintf(os.Stderr, "FAIL: changes needed but not made due to --verify\n")
+		} else {
+			fmt.Fprintf(os.Stderr, "FAIL: some manual changes are still required.\n")
+		}
 		os.Exit(1)
 	}
 }
diff --git a/hack/run-gendocs.sh b/hack/run-gendocs.sh
index c7c22cdc182..6b71a7177ae 100755
--- a/hack/run-gendocs.sh
+++ b/hack/run-gendocs.sh
@@ -45,22 +45,20 @@ kube::util::gen-doc "${genman}" "${KUBE_ROOT}" "docs/man/man1"
 kube::util::gen-doc "${genbashcomp}" "${KUBE_ROOT}" "contrib/completions/bash/"
 kube::util::gen-analytics "${KUBE_ROOT}"
 
-"${mungedocs}" "--root-dir=${KUBE_ROOT}/docs/"
-ret=$?
+"${mungedocs}" "--root-dir=${KUBE_ROOT}/docs/" && ret=0 || ret=$?
 if [[ $ret -eq 1 ]]; then
   echo "${KUBE_ROOT}/docs/ requires manual changes.  See preceeding errors."
   exit 1
-elif [[ $ret -eq 2 ]]; then
+elif [[ $ret -gt 1 ]]; then
   echo "Error running mungedocs."
   exit 1
 fi
 
-"${mungedocs}" "--root-dir=${KUBE_ROOT}/examples/"
-ret=$?
+"${mungedocs}" "--root-dir=${KUBE_ROOT}/examples/" && ret=0 || ret=$?
 if [[ $ret -eq 1 ]]; then
   echo "${KUBE_ROOT}/examples/ requires manual changes.  See preceeding errors."
   exit 1
-elif [[ $ret -eq 2 ]]; then
+elif [[ $ret -gt 1 ]]; then
   echo "Error running mungedocs."
   exit 1
 fi
diff --git a/hack/verify-gendocs.sh b/hack/verify-gendocs.sh
index c4bda4bd5f9..e2a6981744b 100755
--- a/hack/verify-gendocs.sh
+++ b/hack/verify-gendocs.sh
@@ -49,24 +49,22 @@ cp -a "${DOCROOT}" "${TMP_DOCROOT}"
 
 # mungedocs --verify can (and should) be run on the real docs, otherwise their
 # links will be distorted. --verify means that it will not make changes.
-"${mungedocs}" "--verify=true" "--root-dir=${DOCROOT}"
-ret=$?
+"${mungedocs}" "--verify=true" "--root-dir=${DOCROOT}" && ret=0 || ret=$?
 if [[ $ret -eq 1 ]]; then
   echo "${DOCROOT} is out of date. Please run hack/run-gendocs.sh"
   exit 1
 fi
-if [[ $ret -eq 2 ]]; then
+if [[ $ret -gt 1 ]]; then
   echo "Error running mungedocs"
   exit 1
 fi
 
-"${mungedocs}" "--verify=true" "--root-dir=${EXAMPLEROOT}"
-ret=$?
+"${mungedocs}" "--verify=true" "--root-dir=${EXAMPLEROOT}" && ret=0 || ret=$?
 if [[ $ret -eq 1 ]]; then
   echo "${EXAMPLEROOT} is out of date. Please run hack/run-gendocs.sh"
   exit 1
 fi
-if [[ $ret -eq 2 ]]; then
+if [[ $ret -gt 1 ]]; then
   echo "Error running mungedocs"
   exit 1
 fi
@@ -75,8 +73,7 @@ kube::util::gen-doc "${genman}" "${_tmp}" "docs/man/man1/"
 kube::util::gen-doc "${gendocs}" "${_tmp}" "docs/user-guide/kubectl/" '###### Auto generated by spf13/cobra'
 
 echo "diffing ${DOCROOT} against freshly generated docs"
-ret=0
-diff -Naupr "${DOCROOT}" "${TMP_DOCROOT}" || ret=$?
+diff -Naupr "${DOCROOT}" "${TMP_DOCROOT}" && ret=0 || ret=$?
 rm -rf "${_tmp}"
 needsanalytics=($(kube::util::gen-analytics "${KUBE_ROOT}" 1))
 if [[ ${#needsanalytics[@]} -ne 0 ]]; then
@@ -96,8 +93,7 @@ COMPROOT="${KUBE_ROOT}/contrib/completions"
 TMP_COMPROOT="${KUBE_ROOT}/contrib/completions_tmp"
 cp -a "${COMPROOT}" "${TMP_COMPROOT}"
 kube::util::gen-doc "${genbashcomp}" "${TMP_COMPROOT}" "bash/"
-ret=0
-diff -Naupr "${COMPROOT}" "${TMP_COMPROOT}" || ret=$?
+diff -Naupr "${COMPROOT}" "${TMP_COMPROOT}" && ret=0 || ret=$?
 rm -rf ${TMP_COMPROOT}
 if [ $ret -eq 0 ]
 then