Re: [PATCH] add ls-remote --get-push-url option

2015-07-31 Thread Ben Boeckel
On Fri, Jul 31, 2015 at 12:16:46 -0700, Junio C Hamano wrote: > Or even "git remote get url [$there]", "git remote get push-url [$there]". Looking at `git remote`'s existing subcommands, consistency there would be something like: git remote get-url $there git remote get-url --push $there

Re: [PATCH] add ls-remote --get-push-url option

2015-07-31 Thread Junio C Hamano
Ben Boeckel writes: > On Fri, Jul 31, 2015 at 12:02:14 -0700, Junio C Hamano wrote: >> Ben Boeckel writes: >> >> > With some sed, yes, but then so would `git remote show` just as useful >> > too (and in that case, "why does --get-url exist either?" comes to >> > mind). >> >> Either carelessnes

Re: [PATCH] add ls-remote --get-push-url option

2015-07-31 Thread Ben Boeckel
On Fri, Jul 31, 2015 at 12:02:14 -0700, Junio C Hamano wrote: > Ben Boeckel writes: > > > With some sed, yes, but then so would `git remote show` just as useful > > too (and in that case, "why does --get-url exist either?" comes to > > mind). > > Either carelessness let it slip in, or it came be

Re: [PATCH] add ls-remote --get-push-url option

2015-07-31 Thread Junio C Hamano
Ben Boeckel writes: > With some sed, yes, but then so would `git remote show` just as useful > too (and in that case, "why does --get-url exist either?" comes to > mind). Either carelessness let it slip in, or it came before 'git remote show'. -- To unsubscribe from this list: send the line "uns

Re: [PATCH] add ls-remote --get-push-url option

2015-07-31 Thread Ben Boeckel
On Fri, Jul 31, 2015 at 11:40:12 -0700, Junio C Hamano wrote: > Probably get-url makes (some) sense because ls-remote is a "fetch > that does not actually fetch anything". But "get-push-url" to > ls-remote makes _no_ sense whatsoever. ls-remote and fetch do not > have to know or care about push-u

Re: [PATCH] add ls-remote --get-push-url option

2015-07-31 Thread Junio C Hamano
Ben Boeckel writes: > Not sure if it would be better to make a new variable or to reuse the existing > one. I'm reusing it currently because it makes it easier to ensure they > are mutually exclusive. > > Please keep me CC'd to the list; I'm not subscribed. > > Thanks, > > --Ben > > Ben Boeckel (

[PATCH] add ls-remote --get-push-url option

2015-07-31 Thread Ben Boeckel
Not sure if it would be better to make a new variable or to reuse the existing one. I'm reusing it currently because it makes it easier to ensure they are mutually exclusive. Please keep me CC'd to the list; I'm not subscribed. Thanks, --Ben Ben Boeckel (1): ls-remote: add --get-push-url opti