Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH] Update diff-highlight

From: Junio C Hamano <hidden>
Date: 2016-06-15 23:08:21

Peter Dave Hello [off-list ref] writes:
From: Peter Dave Hello <redacted>

Use `#!/usr/bin/env perl` instead of `#!/usr/bin/perl`
Even though there are existing examples in contrib/ parts that use
this pattern, we try to avoid use of #!/usr/bin/env in more serious
parts of our system.  This is to control the exact interpreter used
to run our scripts at the build time, and to avoid interference by
end users' $PATH environment.  So adding more use of #!/usr/bin/env
is not quite a welcome move, even if it is to contrib/ part.

Perhaps you can instead mimick the way how contrib/subtree uses the
same SHELL_PATH used in the primary Makefile (and config.mak) to
turn git-subtree.sh into git-subtree command.  Rename the source to
diff-highlight.perl, and use PERL_PATH when build procedure turns it
into diff-highlight?

I think that is more in line with the rest of the system.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help