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

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

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

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:43:08

Bryan Larsen [off-list ref] 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?

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

From: Bryan Larsen <hidden>
Date: 2016-06-15 22:43:08

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?
I haven't used gitweb.cgi.  All the tests run, including the 
git-send-email test.  (t4200 patch was sent a couple of days ago).  I 
know there are several people using git very successfully from MacPorts.

The biggest problem in OS X has been that the BSD versions of standard 
programs such as xargs and sed are used instead of the GNU versions. 
There are currently no problems of that sort.

cheers,
Bryan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help