On Tue, Aug 14, 2018 at 11:59 AM Ævar Arnfjörð Bjarmason
<ava...@gmail.com> wrote:
>
> The tests added in 63e95beb08 ("submodule: port resolve_relative_url
> from shell to C", 2016-04-15) didn't do a good job of testing various
> up-path invocations where the up-path would bring us beyond even the
> URL in question without emitting an error.
>
> These results look nonsensical, but it's worth exhaustively testing
> them before fixing any of this code, so we can see which of these
> cases were changed.Yeah. Please look at the comment in builtin/submodule--helper.c in that commit, where I described my expectations. I should have put them into tests instead with the expectations spelled out there. Thanks for this patch! Stefan > > Signed-off-by: Ævar Arnfjörð Bjarmason <ava...@gmail.com> > --- > So I think these tests are worthwihle in themselves, The reason I put it in the comment instead of tests was the ease of spelling out both the status quo and expectations. > but would like > some advice on how to proceed with that from someone more familiar > with submodules. So ideally we'd also error out as soon as the host name is touched?
