Thread (9 messages) flat view 9 messages, 4 authors, 2016-06-15

Re: [Fundamental problem with relative system paths] [PATCH 2/2] run-command (Windows): Run dashless "git <cmd>"

From: Johannes Sixt <hidden>
Date: 2016-06-15 22:45:04

Zitat von Junio C Hamano [off-list ref]:
Johannes Sixt [off-list ref] writes:
quoted
Zitat von Steffen Prohaska [off-list ref]:
...
quoted
The patch below might fix the problem by always calling 'bin/git'
for builtin commands.  The computation in system_path() would
always start from 'bin' and thus yields predictable results.  I
am not sure however if it fully solves the problem because other
code paths might run the dashed forms directly.
This paragraph should go into the commit message.
quoted
...
Your patches make a lot of sense.
I was almost going to suggest doing this everywhere not just on Windows,
but execv_git_cmd() on the POSIX side already runs "git" wrapper, so this
patch makes them in line, finally.
For this reason I'm in favor of these patches. I didn't run the full test suite
with them, yet, (you know, that takes a while on Windows), but "make *clone*
*fetch* *pack*" worked out OK.

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