git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH js/submodule-relative] t7400: avoid path mangling issues

From
Jon Seymour <jon.seymour@gmail.com>
Date
Jun 14, 2012, 13:11 UTC
Message-ID
<CAH3AnrpCJwXNBOL75mBkfF69RK_TVUsnPx7m8reShR2g75bAWQ@mail.gmail.com>
In-Reply-To
<4FD9D4B3.9060805@viscovery.net>
On Thu, Jun 14, 2012 at 10:10 PM, Johannes Sixt <j.sixt@viscovery.net> wrote:
Show 49 quoted lines
> From: Johannes Sixt <j6t@kdbg.org>
>
> A recently introduced test uses an absolute path. But when run on Windows
> using the MSYS bash, such a path is mangled into a Windows style path when
> it is passed to 'git config'. The subsequent 'test' then compares the
> mangled path to the unmangled version and reports a failure.
>
> A path beginning with two slashes denotes a network directory
> (//server/share path) and is not mangled. Use that trick to side-step the
> issue. Just in case that 'git submodule init' regresses in such a way that
> it accesses the URL, use a path name that is unlikely to exist on POSIX
> systems, and that cannot be a server name on Windows.
>
> Signed-off-by: Johannes Sixt <j6t@kdbg.org>
> ---
>  Sorry, Jon, Junio, for the resend. Forgot to Cc the list.
>
>  t/t7400-submodule-basic.sh | 11 ++++++++---
>  1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/t/t7400-submodule-basic.sh b/t/t7400-submodule-basic.sh
> index a899e6d..c73bec9 100755
> --- a/t/t7400-submodule-basic.sh
> +++ b/t/t7400-submodule-basic.sh
> @@ -510,14 +510,19 @@ test_expect_success '../subrepo works with port-qualified URL - ssh://hostname:2
>        )
>  '
>
> -test_expect_success '../subrepo path works with local path - /foo/repo' '
> +# About the choice of the path in the next test:
> +# - double-slash side-steps path mangling issues on Windows
> +# - it is still an absolute local path
> +# - there cannot be a server with a blank in its name just in case the
> +#   path is used erroneously to access a //server/share style path
> +test_expect_success '../subrepo path works with local path - //somewhere else/repo' '
>        (
>                cd reltest &&
>                cp pristine-.git-config .git/config &&
>                cp pristine-.gitmodules .gitmodules &&
> -               git config remote.origin.url /foo/repo &&
> +               git config remote.origin.url "//somewhere else/repo" &&
>                git submodule init &&
> -               test "$(git config submodule.sub.url)" = /foo/subrepo
> +               test "$(git config submodule.sub.url)" = "//somewhere else/subrepo"
>        )
>  '
>
> --
> 1.7.11.rc3.1188.ga03e3fc
Thank you for the report and the test case.
jon.
Previous: Johannes Sixt
Message 2 of 2 in “t7400: avoid path mangling issues”
  1. t7400: avoid path mangling issuesJohannes Sixt, Jun 14, 2012
  2. Jon SeymourJun 14, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.