Re: [PATCH] t5516-fetch-push: Correct misspelled pushInsteadOf

From: Anders Kaseorg <andersk@mit.edu>
Date: Mar 4, 2015, 09:43 UTC
Message-ID: <alpine.DEB.2.10.1503040439100.60859@buzzword-bingo.mit.edu>
In-Reply-To: <54F6D20D.903@drmicha.warpmail.net>

On Wed, 4 Mar 2015, Michael J Gruber wrote:

Show 13 quoted lines

> If there is no "pushurl":
>   "url" will be used as the "URL for fetch" and as the "URL for push".
>   if we are pushing and there is "pushinsteadof" for (part of) "url"
>     subsitute that within "url"
>   else if there is "insteadof" for (part of) "url"
>     substitute that within "url"
> 
> If there is a "pushurl":
>   "url" will be used as the "URL for fetch" and "pushurl" as the "URL for push".
>   if we are pushing and there is "insteadof" for (part of) "pushurl"
>     substitute that within "pushurl"
>   else if we are fetching and there is "insteadof" for (part of) "url"
>     substiute that within "url"

Speaking of that, I recently had to reimplement most of this logic for the openstack git-review tool (https://review.openstack.org/160152), which is why I was staring too closely at t5516 in the first place. It would be nice to have a ‘git ls-remote --get-push-url’ analogous to the existing ‘git ls-remote --get-url’.

Anders