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

Re: [PATCH] perl shebangs: Use /usr/bin/env, not /usr/bin/perl

From
Jeff King <peff@peff.net>
Date
May 21, 2010, 16:01 UTC
Message-ID
<20100521160143.GA29031@sigill.intra.peff.net>
In-Reply-To
<86tyq1gti2.fsf@red.stonehenge.com>
On Fri, May 21, 2010 at 08:32:53AM -0700, Randal L. Schwartz wrote:
> The official Perl should be in /usr/bin or /usr/local/bin, and that's
> the one you should use with Git.  *Or*, give the installer a knob to
> configure.
Don't we already provide a knob? The Makefile has:
  $(patsubst %.perl,%,$(SCRIPT_PERL)): % : %.perl
  [...]
          sed -e '1{' \
              -e '        s|#!.*perl|#!$(PERL_PATH_SQ)|' \
  [...]

So just setting PERL_PATH should be enough (from which PERL_PATH_SQ is derived).

-Peff
Previous: Randal L. SchwartzNext: Ævar Arnfjörð Bjarmason
Message 8 of 10 in “perl libs: perl -w + use warnings is redundant”
  1. perl libs: perl -w + use warnings is redundantÆvar Arnfjörð Bjarmason, May 21, 2010
  2. perl shebangs: Use /usr/bin/env, not /usr/bin/perlÆvar Arnfjörð Bjarmason, May 21, 2010
  3. Peter KjellerstedtMay 21, 2010
  4. Ævar Arnfjörð BjarmasonMay 21, 2010
  5. perl shebangs: Use /usr/bin/env, not /usr/bin/perlÆvar Arnfjörð Bjarmason, May 21, 2010
  6. Tor ArntsenMay 21, 2010
  7. Randal L. SchwartzMay 21, 2010
  8. Jeff KingMay 21, 2010
  9. 1/2 perl libs: perl -w + use warnings is redundantÆvar Arnfjörð Bjarmason, May 21, 2010
  10. Ævar Arnfjörð BjarmasonMay 21, 2010

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.