On Thu, Jan 09, 2014 at 12:07:38PM -0800, Junio C Hamano wrote:
> Having said all that, the difference between the full URL shown by
> "remote --verbose" (which is used to interact with the remote in
> this repository) and the abbreviated URL (which is shown by "fetch"
> and is designed to be sharable with others with a simple cut&paste)
> matters only when there are a pair of ambiguously configured
> repositories (e.g. there are two repositories "git://host/a.git/"
> and "git://host/a/.git") that serve different things and you are
> debugging the situation.  And to me, "remote --verbose" looks more
> or less a debugging aid, nothing more.  So another alternative that
> may be to leave everything as-is.
> Thanks.

I like the alterantive option of "leave everything as-is", especially
after the arguments you've presented. There is still the problem of the
logic that has been duplicated. I think it should be put in a function,
but if you are ok with leaving it duplicated that is fine by me.

        if (raw_url)
                url = transport_anonymize_url(raw_url);
                url = xstrdup("foreign");

        url_len = strlen(url);
        for (i = url_len - 1; url[i] == '/' && 0 <= i; i--)
        url_len = i + 1;
        if (4 < i && !strncmp(".git", url + i - 3, 4))
                url_len = i - 3;

Tom Miller
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to