# [PATCH 0/14] misc test-suite patches

3 messages from 2010-12-14 to 2010-12-18. Participants: Ramsay Jones, Junio C Hamano.
Thread: https://gitlist.dev/t/26053

## Ramsay Jones, 2010-12-14 18:17

Subject: [PATCH 0/14] misc test-suite patches
Message-ID: <4D07B4C1.2050007@ramsay1.demon.co.uk>
URL: https://gitlist.dev/e/4D07B4C1.2050007%40ramsay1.demon.co.uk

```
Hi Junio,

I've had many of these test-suite related patches lying around for
a while, so I'm passing these along now in the hope I can remove
some old branches...

[PATCH 01/14] t3600-rm.sh: Don't pass a non-existent prereq to test #15
[PATCH 02/14] t9142: Move call to start_httpd into the setup test
[PATCH 03/14] lib-git-svn.sh: Avoid setting web server variables unnecessarily
[PATCH 04/14] lib-git-svn.sh: Add check for missing mod_dav_svn module
[PATCH 05/14] lib-git-svn.sh: Add check for mis-configured web server variables
[PATCH 06/14] t9501-*.sh: Fix a test failure on Cygwin
[PATCH 07/14] t7800-difftool.sh: Fix a test failure on Cygwin
[PATCH 08/14] help.c: Fix detection of custom merge strategy on cygwin
[PATCH 09/14] t1301-*.sh: Fix the 'forced modes' test on cygwin
[PATCH 10/14] t6038-*.sh: Pass the -b (--binary) option to sed on cygwin
[PATCH 11/14] t3032-*.sh: Pass the -b (--binary) option to sed on cygwin
[PATCH 12/14] t3032-*.sh: Do not strip CR from line-endings while grepping on MinGW
[PATCH 13/14] t4135-*.sh: Skip the "backslash" tests on cygwin
[PATCH 14/14] t{3032,6038}-*.sh: Allow SED_BIN_OPT to override the -b sed option

These patches don't really have too much in common, well 2-5 and 10-12 are
somewhat related, other than fixing tests! (These were mostly done on Linux
and Cygwin but, since I also test on MinGW, one or two are just for MinGW)

ATB,
Ramsay Jones

```

## Junio C Hamano, 2010-12-14 19:44

Subject: Re: [PATCH 0/14] misc test-suite patches
Message-ID: <7v4oagt9zu.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7v4oagt9zu.fsf%40alter.siamese.dyndns.org
In-Reply-To: <4D07B4C1.2050007@ramsay1.demon.co.uk>

```
Ramsay Jones <ramsay@ramsay1.demon.co.uk> writes:

> I've had many of these test-suite related patches lying around for
> a while, so I'm passing these along now in the hope I can remove
> some old branches...

Thanks.

> [PATCH 08/14] help.c: Fix detection of custom merge strategy on cygwin
> [PATCH 09/14] t1301-*.sh: Fix the 'forced modes' test on cygwin

I sense that there is something fundamentally wrong with the tricky l/stat
emulation set-up code there, but I lack an expertise to judge if these two
patches is going in the right direction.  Help from people savvier on
Windows issues than I is very much appreciated.

```

## Ramsay Jones, 2010-12-18 20:08

Subject: Re: [PATCH 0/14] misc test-suite patches
Message-ID: <4D0D14D0.5020809@ramsay1.demon.co.uk>
URL: https://gitlist.dev/e/4D0D14D0.5020809%40ramsay1.demon.co.uk
In-Reply-To: <7v4oagt9zu.fsf@alter.siamese.dyndns.org>

```
Junio C Hamano wrote:
>> [PATCH 08/14] help.c: Fix detection of custom merge strategy on cygwin
>> [PATCH 09/14] t1301-*.sh: Fix the 'forced modes' test on cygwin
> 
> I sense that there is something fundamentally wrong with the tricky l/stat
> emulation set-up code there, but I lack an expertise to judge if these two
> patches is going in the right direction.  Help from people savvier on
> Windows issues than I is very much appreciated.

Just as a matter of interest, since I put some extra effort into writing
them, did you find the commit messages to be insufficient/unhelpful/confusing?
What could I have done better?

ATB,
Ramsay Jones

```
