From 7bcd6ecea625cbe528633617c51e09f376368202 Mon Sep 17 00:00:00 2001 From: Alex Thiessen Date: Thu, 4 Jan 2018 15:16:23 +0000 Subject: util/gitconfig: Use `make` in git hooks As the code was moved from the Makefile.inc to a separate file in commit 9ab8ae6a (util/gitconfig: Make gitconfig a bash script),`$(MAKE)` was replaced by `remake`, introducing dependency on this tool which is basically a `make` with debugging capabilities. Many developers don't have `remake` installed, leading to pre-commit hooks being not executed properly. Apparently this was an unintentional change. Furthermore, special treatment of `make` tool via the `%MAKE%` substitution performed during hooks' deployment is still desired. Use case is calling `remake gitconfig` to set `remake` as the `make` tool in the hooks. To accomplish this, add a parameter that is passed from the Makefile.inc to gitconfig.sh. Change-Id: Ia78e06567b904b342dc9b7778569201fe02e6897 Signed-off-by: Alex Thiessen Reviewed-on: https://review.coreboot.org/23096 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Martin Roth --- util/gitconfig/gitconfig.sh | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'util') diff --git a/util/gitconfig/gitconfig.sh b/util/gitconfig/gitconfig.sh index c2c4a69bbd..edfb6df742 100755 --- a/util/gitconfig/gitconfig.sh +++ b/util/gitconfig/gitconfig.sh @@ -15,6 +15,10 @@ ## GNU General Public License for more details. ## +# In the hooks, use the same `make` tool as used when running `make gitconfig`, +# e.g. `remake gitconfig` will set `remake` to be run by the hooks. +MAKE="${1:-make}" + if ! { cdup="$(git rev-parse --show-cdup 2>/dev/null)" && [ -z "${cdup}" ]; } then echo "Error: Not in root of a git repository" @@ -26,7 +30,7 @@ for hook in commit-msg pre-commit ; do if [ util/gitconfig/${hook} -nt "${coreboot_hooks}/${hook}" ] \ || [ ! -x "${coreboot_hooks}/${hook}" ] then - sed -e "s,%MAKE%,remake,g" util/gitconfig/$hook \ + sed -e "s,%MAKE%,${MAKE},g" util/gitconfig/$hook \ > "${coreboot_hooks}/${hook}" chmod +x "${coreboot_hooks}/${hook}" fi @@ -39,7 +43,7 @@ for submodule in 3rdparty/blobs libhwbase libgfxinit; do if [ util/gitconfig/commit-msg -nt "${hooks}/commit-msg" ] \ || [ ! -x "${hooks}/commit-msg" ] then - sed -e "s,%MAKE%,remake,g" util/gitconfig/commit-msg \ + sed -e "s,%MAKE%,${MAKE},g" util/gitconfig/commit-msg \ > "${hooks}/commit-msg" chmod +x "${hooks}/commit-msg" fi -- cgit v1.2.3