Fix verification scripts to actually look at all go files.

This commit is contained in:
Daniel Smith 2014-08-12 17:16:57 -07:00
parent 74a7179e2e
commit 713a9488a2
2 changed files with 11 additions and 12 deletions

View File

@ -14,19 +14,16 @@
# See the License for the specific language governing permissions and # See the License for the specific language governing permissions and
# limitations under the License. # limitations under the License.
REPO_ROOT="$(realpath "$(dirname $0)/..")" REPO_ROOT="$(cd "$(dirname "$0")/../" && pwd -P)"
result=0 result=0
dirs=("pkg" "cmd") gofiles="$(find ${REPO_ROOT} -type f | grep "[.]go$" | grep -v "third_party/\|release/\|output/\|target/")"
for file in ${gofiles}; do
for dir in ${dirs[@]}; do if [[ "$(${REPO_ROOT}/hooks/boilerplate.sh "${file}")" -eq "0" ]]; then
for file in $(grep -r -l "" "${REPO_ROOT}/${dir}/" | grep "[.]go"); do echo "Boilerplate header is wrong for: ${file}"
if [[ "$(${REPO_ROOT}/hooks/boilerplate.sh "${file}")" -eq "0" ]]; then result=1
echo "Boilerplate header is wrong for: ${file}" fi
result=1
fi
done
done done
dirs=("cluster" "hack" "hooks") dirs=("cluster" "hack" "hooks")

View File

@ -24,9 +24,11 @@ if [[ ${GO_VERSION[2]} != "go1.2" && ${GO_VERSION[2]} != "go1.3" ]]; then
exit 0 exit 0
fi fi
REPO_ROOT="$(cd "$(dirname "$0")/../" && pwd -P)"
bad=$(gofmt -s -l pkg/ cmd/) files="$(find ${REPO_ROOT} -type f | grep "[.]go$" | grep -v "third_party/\|release/\|output/\|target/")"
if [ "$?" != "0" ]; then bad=$(gofmt -s -l ${files})
if [[ -n "${bad}" ]]; then
echo "$bad" echo "$bad"
exit 1 exit 1
fi fi