From 4eeebf76ae05b1f7a1fba4391bde5c30c2fa74a0 Mon Sep 17 00:00:00 2001
From: Daniel Smith <dbsmith@google.com>
Date: Wed, 21 Sep 2016 18:08:41 -0700
Subject: [PATCH] make verify godeps help you fix your problems

---
 hack/verify-godeps.sh | 98 +++++++++++++++++++++++++++----------------
 1 file changed, 61 insertions(+), 37 deletions(-)

diff --git a/hack/verify-godeps.sh b/hack/verify-godeps.sh
index 44619cfc7d1..f6e0efa5ec7 100755
--- a/hack/verify-godeps.sh
+++ b/hack/verify-godeps.sh
@@ -52,9 +52,14 @@ fi
 
 # Create a nice clean place to put our new godeps
 _tmpdir="$(mktemp -d -t gopath.XXXXXX)"
+KEEP_TMP=false
 function cleanup {
-  echo "Removing ${_tmpdir}"
-  rm -rf "${_tmpdir}"
+  if [ "${KEEP_TMP}" == "true" ]; then
+    echo "Leaving ${_tmpdir} for you to examine or copy. Please delete it manually when finished. (rm -rf ${_tmpdir})"
+  else
+    echo "Removing ${_tmpdir}"
+    rm -rf "${_tmpdir}"
+  fi
 }
 trap cleanup EXIT
 
@@ -67,48 +72,67 @@ _kubetmp="${_kubetmp}/kubernetes"
 
 # Do all our work in the new GOPATH
 export GOPATH="${_tmpdir}"
-cd "${_kubetmp}"
 
-# Build the godep tool
-go get -u github.com/tools/godep 2>/dev/null
-GODEP="${GOPATH}/bin/godep"
-pin-godep() {
-  pushd "${GOPATH}/src/github.com/tools/godep" > /dev/null
-    git checkout "$1"
-    "${GODEP}" go install
-  popd > /dev/null
-}
-# Use to following if we ever need to pin godep to a specific version again
-#pin-godep 'v63'
+pushd "${_kubetmp}" 2>&1 > /dev/null
+  # Build the godep tool
+  go get -u github.com/tools/godep 2>/dev/null
+  GODEP="${GOPATH}/bin/godep"
+  pin-godep() {
+    pushd "${GOPATH}/src/github.com/tools/godep" > /dev/null
+      git checkout "$1"
+      "${GODEP}" go install
+    popd > /dev/null
+  }
+  # Use to following if we ever need to pin godep to a specific version again
+  #pin-godep 'v63'
 
-# Fill out that nice clean place with the kube godeps
-echo "Starting to download all kubernetes godeps. This takes a while"
-"${GODEP}" restore
-echo "Download finished"
+  # Fill out that nice clean place with the kube godeps
+  echo "Starting to download all kubernetes godeps. This takes a while"
+  "${GODEP}" restore
+  echo "Download finished"
 
-# Destroy deps in the copy of the kube tree
-rm -rf ./Godeps ./vendor
+  # Destroy deps in the copy of the kube tree
+  rm -rf ./Godeps ./vendor
 
-# For some reason the kube tree needs to be a git repo for the godep tool to
-# run. Doesn't make sense.
-git init > /dev/null 2>&1
+  # For some reason the kube tree needs to be a git repo for the godep tool to
+  # run. Doesn't make sense.
+  git init > /dev/null 2>&1
 
-# Recreate the Godeps using the nice clean set we just downloaded
-hack/godep-save.sh
+  # Recreate the Godeps using the nice clean set we just downloaded
+  hack/godep-save.sh
+popd 2>&1 > /dev/null
 
-# Test for diffs
-if ! _out="$(diff -Naupr --ignore-matching-lines='^\s*\"GoVersion\":' --ignore-matching-line='^\s*\"GodepVersion\":' --ignore-matching-lines='^\s*\"Comment\":' ${KUBE_ROOT}/Godeps/Godeps.json ${_kubetmp}/Godeps/Godeps.json)"; then
-  echo "Your Godeps.json is different:"
-  echo "${_out}"
-  echo "Godeps Verify failed."
-  exit 1
-fi
+ret=0
 
-if ! _out="$(diff -Naupr ${KUBE_ROOT}/vendor ${_kubetmp}/vendor)"; then
-  echo "Your vendored results are different:"
-  echo "${_out}"
-  echo "Godeps Verify failed."
-  exit 1
+pushd "${KUBE_ROOT}" 2>&1 > /dev/null
+  # Test for diffs
+  if ! _out="$(diff -Naupr --ignore-matching-lines='^\s*\"GoVersion\":' --ignore-matching-line='^\s*\"GodepVersion\":' --ignore-matching-lines='^\s*\"Comment\":' Godeps/Godeps.json ${_kubetmp}/Godeps/Godeps.json)"; then
+    echo "Your Godeps.json is different:"
+    echo "${_out}"
+    echo "Godeps Verify failed."
+    echo "${_out}" > godepdiff.patch
+    echo "If you're seeing this locally, run the below command to fix your Godeps.json:"
+    echo "patch -p0 < godepdiff.patch"
+    echo "(The above output can be saved as godepdiff.patch if you're not running this locally)"
+    KEEP_TMP=true
+    ret=1
+  fi
+
+  if ! _out="$(diff -Naupr vendor ${_kubetmp}/vendor)"; then
+    echo "Your vendored results are different:"
+    echo "${_out}"
+    echo "Godeps Verify failed."
+    echo "${_out}" > vendordiff.patch
+    echo "If you're seeing this locally, run the below command to fix your directories:"
+    echo "patch -p0 < vendordiff.patch"
+    echo "(The above output can be saved as godepdiff.patch if you're not running this locally)"
+    KEEP_TMP=true
+    ret=1
+  fi
+popd 2>&1 > /dev/null
+
+if [[ ${ret} > 0 ]]; then
+  exit ${ret}
 fi
 
 echo "Godeps Verified."