# [PATCH] scalar: fix wrong shell hashbang 4 messages from 2022-09-17 to 2022-09-20. Participants: ZheNing Hu via GitGitGadget, Victoria Dye, Junio C Hamano, ZheNing Hu. Thread: https://gitlist.dev/t/58450 ## ZheNing Hu via GitGitGadget, 2022-09-17 13:11 Subject: [PATCH] scalar: fix wrong shell hashbang Message-ID: URL: https://gitlist.dev/e/pull.1355.git.1663420281187.gitgitgadget%40gmail.com ``` From: ZheNing Hu $SHELL_PATH_SQ haven't been set in scalar's Makefile, bin-wrappers/scalar will begin with wrong hashbang "#!", fix it by setting $SHELL_PATH and $SHELL_PATH_SQ in scalar's Makefile. Signed-off-by: ZheNing Hu --- scalar: fix wrong shell hashbang The bin-wrappers/scalar seems like missing setting $SHELL_PATH_SQ which lead to I can't execute bin-wrappers/scalar correctly, which output error: zsh: exec format error: scalar (this bug will not turn out in bash) The bin-wrappers/scalar begin with wrong hashbang "#!", which cannot figure out by zsh. So this patch want to fix this problem. v1: Setting $SHELL_PATH and $SHELL_PATH_SQ in scalar/Makefile. Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1355%2Fadlternative%2Fzh%2Fscalar-fix-hashbang-v1 Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1355/adlternative/zh/scalar-fix-hashbang-v1 Pull-Request: https://github.com/gitgitgadget/git/pull/1355 contrib/scalar/Makefile | 3 +++ 1 file changed, 3 insertions(+) diff --git a/contrib/scalar/Makefile b/contrib/scalar/Makefile index 37f283f35d7..b2a81d9eb45 100644 --- a/contrib/scalar/Makefile +++ b/contrib/scalar/Makefile @@ -8,6 +8,9 @@ include ../../config.mak.uname -include ../../config.mak.autogen -include ../../config.mak +SHELL_PATH ?= $(SHELL) +SHELL_PATH_SQ = $(subst ','\'',$(SHELL_PATH)) + TARGETS = scalar$(X) scalar.o GITLIBS = ../../common-main.o ../../libgit.a ../../xdiff/lib.a base-commit: d3fa443f97e3a8d75b51341e2d5bac380b7422df -- gitgitgadget ``` ## Victoria Dye, 2022-09-18 20:08 Subject: Re: [PATCH] scalar: fix wrong shell hashbang Message-ID: <433bf552-db20-9989-92c5-ee625ce9337a@github.com> URL: https://gitlist.dev/e/433bf552-db20-9989-92c5-ee625ce9337a%40github.com In-Reply-To: ``` ZheNing Hu via GitGitGadget wrote: > From: ZheNing Hu > > $SHELL_PATH_SQ haven't been set in scalar's Makefile, > bin-wrappers/scalar will begin with wrong hashbang "#!", > fix it by setting $SHELL_PATH and $SHELL_PATH_SQ in > scalar's Makefile. > > Signed-off-by: ZheNing Hu > --- > scalar: fix wrong shell hashbang > > The bin-wrappers/scalar seems like missing setting $SHELL_PATH_SQ which > lead to I can't execute bin-wrappers/scalar correctly, which output > error: > > zsh: exec format error: scalar > > (this bug will not turn out in bash) > > The bin-wrappers/scalar begin with wrong hashbang "#!", which cannot > figure out by zsh. So this patch want to fix this problem. > > v1: Setting $SHELL_PATH and $SHELL_PATH_SQ in scalar/Makefile. > Thanks for finding this! However, while your patch fixes the error you've identified (I was able to recreate both the original issue and apply your patch to fix it), I believe it has already been fixed in 'next' by 7b5c93c6c6 (scalar: include in standard Git build & installation, 2022-09-02) (archive: [1]). [1] https://lore.kernel.org/git/bc2092a7a7aae640b547bde8db0c0a26ce31278a.1662134210.git.gitgitgadget@gmail.com/ ``` ## Junio C Hamano, 2022-09-19 18:44 Subject: Re: [PATCH] scalar: fix wrong shell hashbang Message-ID: URL: https://gitlist.dev/e/xmqqillj9pbc.fsf%40gitster.g In-Reply-To: <433bf552-db20-9989-92c5-ee625ce9337a@github.com> ``` Victoria Dye writes: > Thanks for finding this! However, while your patch fixes the error you've > identified (I was able to recreate both the original issue and apply your > patch to fix it), I believe it has already been fixed in 'next' by > 7b5c93c6c6 (scalar: include in standard Git build & installation, > 2022-09-02) (archive: [1]). Yup, thanks. The topic that contains that commit should graduate to the master branch before -rc1, like this afternoon ;-) ``` ## ZheNing Hu, 2022-09-20 12:43 Subject: Re: [PATCH] scalar: fix wrong shell hashbang Message-ID: URL: https://gitlist.dev/e/CAOLTT8Rxh7dVJL6gGe1KPnTNOgPdQky06deOe0L5qczE0eh64g%40mail.gmail.com In-Reply-To: <433bf552-db20-9989-92c5-ee625ce9337a@github.com> ``` Victoria Dye 于2022年9月19日周一 04:08写道： > > ZheNing Hu via GitGitGadget wrote: > > From: ZheNing Hu > > > > $SHELL_PATH_SQ haven't been set in scalar's Makefile, > > bin-wrappers/scalar will begin with wrong hashbang "#!", > > fix it by setting $SHELL_PATH and $SHELL_PATH_SQ in > > scalar's Makefile. > > > > Signed-off-by: ZheNing Hu > > --- > > scalar: fix wrong shell hashbang > > > > The bin-wrappers/scalar seems like missing setting $SHELL_PATH_SQ which > > lead to I can't execute bin-wrappers/scalar correctly, which output > > error: > > > > zsh: exec format error: scalar > > > > (this bug will not turn out in bash) > > > > The bin-wrappers/scalar begin with wrong hashbang "#!", which cannot > > figure out by zsh. So this patch want to fix this problem. > > > > v1: Setting $SHELL_PATH and $SHELL_PATH_SQ in scalar/Makefile. > > > > Thanks for finding this! However, while your patch fixes the error you've > identified (I was able to recreate both the original issue and apply your > patch to fix it), I believe it has already been fixed in 'next' by > 7b5c93c6c6 (scalar: include in standard Git build & installation, > 2022-09-02) (archive: [1]). > > [1] https://lore.kernel.org/git/bc2092a7a7aae640b547bde8db0c0a26ce31278a.1662134210.git.gitgitgadget@gmail.com/ It seems that this problem has been solved, thanks! -- ZheNing Hu ```