Re: [PATCH] get_remote_url(): use the same data source as ls-remote to get remote urls

From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Date: Mar 1, 2011, 08:41 UTC
Message-ID: <20110301084110.GT22310@pengutronix.de>
In-Reply-To: <7v4o7nhgr1.fsf@alter.siamese.dyndns.org>

On Mon, Feb 28, 2011 at 03:38:26PM -0800, Junio C Hamano wrote:

Show 12 quoted lines

> Uwe Kleine-König  <u.kleine-koenig@pengutronix.de> writes:
> 
> > with this patch git-request-pull.sh (== the only in-tree user of
> > git-parse-remote.sh:get_remote_url()) could directly use
> >
> > 	git ls-remote --get-url
> >
> > .  I guess you wouldn't want to remove git-parse-remote.sh:get_remote_url()
> > though?!
> 
> If nobody uses it, why not?  I was actually hoping for the day that nobody
> uses any of the defined function so taht we can remove that whole file.

Ok for me. Probably I'm a bit conservative here because I usually work on the kernel and people are picky here to remove interfaces.

I can followup with a patch.

Best regards Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |