Re: [PATCH 3/6] Glean libexec path from argv[0] for git-upload-pack and git-receive-pack.
- From
Steffen Prohaska <prohaska@zib.de>
- Date
- Jan 10, 2009, 15:55 UTC
- Message-ID
- <81783342-81AA-415A-8EF3-1CD9F5390483@zib.de>
- In-Reply-To
- <alpine.DEB.1.00.0901101532430.30769@pacific.mpi-cbg.de>
On Jan 10, 2009, at 3:34 PM, Johannes Schindelin wrote:
Show 34 quoted lines
> On Sat, 10 Jan 2009, Steffen Prohaska wrote: > >> From: Steve Haslam <shaslam@lastminute.com> >> >> If the user specified the full path to git-upload-pack as the -u >> option to >> "git clone" when cloning a remote repository, and git was not on >> the default >> PATH on the remote machine, git-upload-pack was failing to exec >> git-pack-objects. >> >> By making the argv[0] path (if any) available to setup_path(), this >> will >> allow finding the "git" executable in the same directory as >> "git-upload-pack". The default built in to exec_cmd.c is to look >> for "git" >> in the ".../libexec/git-core" directory, but it is not installed >> there (any >> longer). >> >> Much the same applies to invoking git-receive-pack from a non-PATH >> location >> using the "--exec" argument to "git push". >> >> [ spr: split Steve's original commit into two commits. ] > > I think you do not want to have that [ ... ] in the commit message, > but > after the "--". > > Further, it would make sense to have these 2 patches independently, > provided that a test is added with which we could verify that the > patches > are actually necessary for upload-pack/receive-pack.
Steve, could you comment on this. You are the original author of the patch.
Steffen