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

Question about PERL_PATH

From
IAIan Abbott <abbotti@mev.co.uk>
Date
May 6, 2011, 13:30 UTC
Message-ID
<4DC3F804.3080205@mev.co.uk>

The Gentoo ebuilds for git have started invoking 'make' with the option 'PERL_PATH=/usr/bin/env perl'. This doesn't seem to cause any problems except when running git instaweb, which outputs an error similar to the following:

/usr/libexec/git-core/git-instaweb: line 135: /usr/bin/env perl: No such file or directory

There is an easy workaround, which is to modify the invocation of perl in the httpd_is_ready() function in the git-instaweb shell script. It currently invokes perl as "$PERL" (with the quotes). Removing the quotes makes it work.

My question is, should git support Gentoo's unusual setting of PERL_PATH as a multi-word command, or should Gentoo patch around the problem they have created with git-instaweb themselves?

There is one other place where $PERL is used in git-instaweb and that is in the mongoose_conf() function. It has a heredoc that puts the following line in the httpd.conf file (subject to shell variable expansion):

cgi_interp	$PERL
For Gentoo, that would get expanded to the following:
cgi_interp	/usr/bin/env perl
I don't know if Mongoose would choke on this or not.
-- 
-=( Ian Abbott @ MEV Ltd.    E-mail: <abbotti@mev.co.uk>        )=-
-=( Tel: +44 (0)161 477 1898   FAX: +44 (0)161 718 3587         )=-
Next: Michael J Gruber
Message 1 of 7 in “Question about PERL_PATH”
  1. Ian AbbottMay 6, 2011
  2. Michael J GruberMay 6, 2011
  3. Junio C HamanoMay 6, 2011
  4. Ian AbbottMay 6, 2011
  5. Robin H. JohnsonMay 7, 2011
  6. Jeff KingMay 8, 2011
  7. Drew NorthupMay 9, 2011

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.