Felipe Contreras <felipe.contre...@gmail.com> writes:
> On Fri, Aug 9, 2013 at 5:15 PM, Junio C Hamano <gits...@pobox.com> wrote:
>> Felipe Contreras <felipe.contre...@gmail.com> writes:
>>>> OK, I think I see why you are puzzled.
>>>> Cloning works fine because we "fix the path" *after* the clone is done
>>>> successfully, for the following reason:
>>> So if we didn't store a different path, it would work. So instead of
>>> expanding '~' ourselves, it would be better to don't expand anything,
>>> and leave it as it is, but how to detect that in fix_path()?
>> I think that the patch relies on that os.path.expanduser(), if
>> url.path is such a path that begins with "~" (or "~whom"), returns
>> an absolute path. When given an absolute path, or "~whom/path",
>> fix_path returns without running 'git config' on remote.<alias>.url
> I think ~whom/path would run 'git config'.
Hmph, do you mean the third example of this?
>>> import os
which will give "~felipe/repo" that is _not_ an absolute repository
because no such user exists on this box?
It is true that in that case fix_path() will not return early and
will throw a bogus path at "git config", but if the "~whom" does not
resolve to an existing home directory of a user, I am not sure what
we can do better than what Antoine's patch does.
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