Subject: Re: [PATCH 5/6] t9902: mingw-specific fix for gitfile link files

From: Junio C Hamano <[email-masked]>
Date: Thu, 17 Jul 2014 13:42:28 -0700

Stepan Kasal <[email-masked]> writes:

> From: Pat Thoyts <[email-masked]>
>
> The path in a .git platform independent link file needs to be absolute
> and under mingw we need it to be a windows type path, not a unix style
> path so it should start with a drive letter and not a /.

OK.   We seem to have uses of "pwd -P" in other tests; don't they
need a similar treatment, and if not why not?

>
> Signed-off-by: Pat Thoyts <[email-masked]>
> Signed-off-by: Stepan Kasal <[email-masked]>
> ---
>  t/t9902-completion.sh | 17 +++++++++++++----
>  1 file changed, 13 insertions(+), 4 deletions(-)
>
> diff --git a/t/t9902-completion.sh b/t/t9902-completion.sh
> index 1d1c106..f10a752 100755
> --- a/t/t9902-completion.sh
> +++ b/t/t9902-completion.sh
> @@ -212,9 +212,18 @@ test_expect_success '__gitdir - non-existing $GIT_DIR' '
>  	)
>  '
>  
> +function pwd_P_W () {
> +	if test_have_prereq MINGW
> +	then
> +		pwd -W
> +	else
> +		pwd -P
> +	fi
> +}
> +
>  test_expect_success '__gitdir - gitfile in cwd' '
> -	echo "$(pwd -P)/otherrepo/.git" >expected &&
> -	echo "gitdir: $TRASH_DIRECTORY/otherrepo/.git" >subdir/.git &&
> +	echo "$(pwd_P_W)/otherrepo/.git" >expected &&
> +	echo "gitdir: $(pwd_P_W)/otherrepo/.git" >subdir/.git &&
>  	test_when_finished "rm -f subdir/.git" &&
>  	(
>  		cd subdir &&
> @@ -224,8 +233,8 @@ test_expect_success '__gitdir - gitfile in cwd' '
>  '
>  
>  test_expect_success '__gitdir - gitfile in parent' '
> -	echo "$(pwd -P)/otherrepo/.git" >expected &&
> -	echo "gitdir: $TRASH_DIRECTORY/otherrepo/.git" >subdir/.git &&
> +	echo "$(pwd_P_W)/otherrepo/.git" >expected &&
> +	echo "gitdir: $(pwd_P_W)/otherrepo/.git" >subdir/.git &&
>  	test_when_finished "rm -f subdir/.git" &&
>  	(
>  		cd subdir/subsubdir &&
> -- 
> 2.0.0.9635.g0be03cb
>
> -- 

-- 
-- 
*** Please reply-to-all at all times ***
*** (do not pretend to know who is subscribed and who is not) ***
*** Please avoid top-posting. ***
The msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.

You received this message because you are subscribed to the Google
Groups "msysGit" group.
To post to this group, send email to [email-masked]
To unsubscribe from this group, send email to
[email-masked]
For more options, and view previous threads, visit this group at
http://groups.google.com/group/msysgit?hl=en_US?hl=en

--- 
You received this message because you are subscribed to the Google Groups "msysGit" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email-masked].
For more options, visit https://groups.google.com/d/optout.