Re: [PATCH] t1800: correct test to handle Cygwin

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2022-09-01 11:32:12-0700, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> In any case, thanks for a fix.  Will queue.
> 
> >  t/t1800-hook.sh | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/t/t1800-hook.sh b/t/t1800-hook.sh
> > index 64096adac7..fae8b2faf9 100755
> > --- a/t/t1800-hook.sh
> > +++ b/t/t1800-hook.sh
> > @@ -159,7 +159,7 @@ test_expect_success 'git hook run a hook with a bad shebang' '
> >  	# TODO: We should emit the same (or at least a more similar)
> >  	# error on Windows and !Windows. See the OS-specific code in

Please also change the comment here too.
I'm not sure what should be put in the comment here, though.
Maybe on MINGW and other platforms?

> >  	# start_command()
> > -	if test_have_prereq !WINDOWS
> > +	if test_have_prereq !MINGW
> >  	then
> >  		cat >expect <<-\EOF
> >  		fatal: cannot run bad-hooks/test-hook: ...

-- 
Danh



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux