Thread (7 messages) flat view 7 messages, 3 authors, 2016-06-15

Re: [PATCH] help.c: Pull cmd_version out of this file.

From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:49:55

Thiago Farina wrote:
On Wed, Oct 27, 2010 at 1:18 PM, Jonathan Nieder [off-list ref] wrote:
quoted
I thought there was some discussion about another binary in
/usr/lib/git-core being a bad thing?
I don't recall, why it's a bad thing?
Because those hardlinks for builtins are present for historical reasons
only.  They slow down filename lookup by some miniscule amount and tempt
authors of scripts to try the dashed form of commands.

Ah, found it.[1]

| At this point, adding the dashed-form "git-version" to the filesystem is
| going backwards.  These files are only to help people who have old scripts
| from pre 1.6.0 days with "PATH=$(git --exec-path):$PATH" with their
| environment.
| 
| We should instead try to not adding them (which would probably need tweak
| in the command list generation code in "git help"), as nobody from pre
| 1.6.0 era would have called "git-version" binary.

Hope that helps.

[1] 4a7a62f3:whats-cooking.txt
http://thread.gmane.org/gmane.comp.version-control.git/155073
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help