git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] scalar: fix wrong shell hashbang

From
ZheNing Hu <adlternative@gmail.com>
Date
Sep 20, 2022, 12:43 UTC
Message-ID
<CAOLTT8Rxh7dVJL6gGe1KPnTNOgPdQky06deOe0L5qczE0eh64g@mail.gmail.com>
In-Reply-To
<433bf552-db20-9989-92c5-ee625ce9337a@github.com>
Victoria Dye <vdye@github.com> 于2022年9月19日周一 04:08写道:
Show 34 quoted lines
>
> ZheNing Hu via GitGitGadget wrote:
> > From: ZheNing Hu <adlternative@gmail.com>
> >
> > $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 <adlternative@gmail.com>
> > ---
> >     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

Previous: Junio C Hamano
Message 4 of 4 in “scalar: fix wrong shell hashbang”
  1. scalar: fix wrong shell hashbangZheNing Hu via GitGitGadget, Sep 17, 2022
  2. Victoria DyeSep 18, 2022
  3. Junio C HamanoSep 19, 2022
  4. ZheNing HuSep 20, 2022

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.