Re: [PATCH 2/2] submodule: fix handling of supermodules with relative origin URLs

From: Jens Lehmann <jens.lehmann@web.de>
Date: May 19, 2012, 18:56 UTC
Message-ID: <4FB7ECCF.9020403@web.de>
In-Reply-To: <1337402403-7546-2-git-send-email-jon.seymour@gmail.com>

Am 19.05.2012 06:40, schrieb Jon Seymour:

Show 9 quoted lines

> Prior to this change, an operation such as git submodule add, init or
> sync produced the wrong result when the origin URL of the supermodule
> was itself a relative URL.
> 
> The issue arises in these cases, because the origin URL of
> the supermodule needs to be prepended with a prefix that navigates
> from the submodule to the supermodule so that when the submodule
> URL is concatenated, the resulting url is relative to the working tree
> of the submodule.

Just a small nit: I'd prefer to replace the 4 occurrences of the term "supermodule" with "superproject".

BTW, what happened to the following comment in you other email?

>> +                               remoteurl="${up_path%/}/${remoteurl%/*}"
>
> Meant up_path%/ to be up_path%/*

The '*' is not there (but the test suite runs fine no matter if I add a '*' there or not). Thinking about it not adding the '*' should be correct, as you just want to chop off a trailing '/' from $up_path here, right?

So no objection on the code changes from my side.