Re: Commit cce8d6fdb introduces file t/t5100/nul, git tree is now incompatible with Cygwin (and probably Windows)
- From
Steffen Prohaska <prohaska@zib.de>
- Date
- May 31, 2008, 17:37 UTC
- Message-ID
- <28EDBE43-6E55-4EC9-97C6-8AC8D949E82D@zib.de>
- In-Reply-To
- <29FDE86E-5B83-4736-8CC6-9C5697B91B2C@wincent.com>
On May 29, 2008, at 4:58 PM, Wincent Colaiuta wrote:
Show 25 quoted lines
> El 29/5/2008, a las 15:22, Johannes Schindelin escribió: >> >> >> On Wed, 28 May 2008, Wincent Colaiuta wrote: >> >>> El 28/5/2008, a las 8:12, Junio C Hamano escribió: >>> >>>> Perhaps we should remove the infamous gitweb/test/Märchen file >>>> while we >>>> are at it? I do not think the file is ever used. >>> >>> I for one would love to see it go, seeing as I live in the ghetto >>> that >>> is HFS+ and am constantly annoyed by it cluttering up my status >>> output >>> with spurious content. >>> >>> I understand that the reason it lives in the tree is precisely to >>> discover problems with such filesystems, but the problem is well and >>> truly discovered by now and I'd much rather see this kind of thing >>> tested from within the test suite rather than every time I do "git >>> status" or "git checkout". >> >> Probably you are not enoyed enough to just go and fix it. >
[...]
> > If the powers that be will accept a change that removes Märchen I'll > be more than happy to whip up a patch.
Unicode normalization is tested in t/t0050-filesystem.sh, which reports on HFS+:
* still broken 8: rename (silent unicode normalization) * still broken 9: merge (silent unicode normalization)
I believe there is no value in keeping gitweb/test/Märchen for the reason of testing HFS+, so I vote for removing it, unless there is another good reason for keeping it.
Steffen