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

Re: [PATCH] Try 2: Allow PERL_PATH="/usr/bin/env perl"

From
Junio C Hamano <junkio@cox.net>
Date
May 4, 2007, 00:43 UTC
Message-ID
<7vy7k5xvt4.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<463A71D7.5060506@larsen.st>
Bryan Larsen <bryan@larsen.st> writes:
> Maybe PERL_PATH should be renamed PERL_SHEBANG or something.  Because
> if you pass in something that doesn't work on a shebang line (longer
> than 32 characters, say), it just won't work.

I think I see what problem you are trying to solve better now. Probably more relevant example on MacOS would be whitespace in the pathname.

I think using the bare $(PERL_PATH) in perl/Makefile is a reasonable solution.

Are they any other issues on MacOS? For example, gitweb.cgi is built by replacing the shebang with $(PERL_PATH); I presume that you already are successfully working around the whitespace in the pathname with your "env perl" on MacOS?

Previous: Bryan LarsenNext: Bryan Larsen
Message 4 of 6 in “Try 2: Allow PERL_PATH="/usr/bin/env perl"”
  1. Try 2: Allow PERL_PATH="/usr/bin/env perl"Bryan Larsen, May 3, 2007
  2. Junio C HamanoMay 3, 2007
  3. Bryan LarsenMay 3, 2007
  4. Junio C HamanoMay 4, 2007
  5. Bryan LarsenMay 4, 2007
  6. Andrew RuderMay 4, 2007

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.