From e392b9a18b146312446c7073ad2b4956b3bcfc35 Mon Sep 17 00:00:00 2001 From: Ben Kochie Date: Tue, 27 Feb 2018 11:06:38 +0100 Subject: [PATCH] Cleanup licence check script (#3893) * Cleanup licence check script * `echo -e` is not supported by POSIX shell. * Add a license to the check script. * Split out license check loop into function for clarity. * Exit 1 --- scripts/check_license.sh | 31 +++++++++++++++++++++++++------ 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/scripts/check_license.sh b/scripts/check_license.sh index d3882c6b6..8b0fff550 100755 --- a/scripts/check_license.sh +++ b/scripts/check_license.sh @@ -1,10 +1,29 @@ #!/bin/sh +# +# Copyright 2018 The Prometheus Authors +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. + +check_license() { + local file="" + for file in $(find . -type f -iname '*.go' ! -path './vendor/*'); do + head -n3 "${file}" | grep -Eq "(Copyright|generated|GENERATED)" || echo " ${file}" + done +} + +licRes=$(check_license) -licRes=$( -for file in $(find . -type f -iname '*.go' ! -path './vendor/*'); do - head -n3 "${file}" | grep -Eq "(Copyright|generated|GENERATED)" || echo -e " ${file}" -done;) if [ -n "${licRes}" ]; then - echo -e "license header checking failed:\n${licRes}" - exit 255 + echo "license header checking failed:" + echo "${licRes}" + exit 1 fi