Thread (17 messages) flat view 17 messages, 8 authors, 2016-06-15

Re: Commit cce8d6fdb introduces file t/t5100/nul, git tree is now incompatible with Cygwin (and probably Windows)

From: Wincent Colaiuta <hidden>
Date: 2016-06-15 22:44:39

Possibly related (same subject, not in this thread)

El 29/5/2008, a las 15:22, Johannes Schindelin escribió:
Hi,

On Wed, 28 May 2008, Wincent Colaiuta wrote:
quoted
El 28/5/2008, a las 8:12, Junio C Hamano escribió:
quoted
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.
No, that's not actually the case. In reality I was pleasantly  
surprised when Junio commented that "perhaps we should remove" that  
file; I had always gotten the impression from this list that such a  
change would be unwelcome because it's easier to just blame the users  
of bad filesystems for choosing those filesystems. I also remember a  
comment from Linus to the effect that that file was kept in the tree  
precisely _because_ it helped us discover such file systems.  
Unfortunately I can't find that message right now but I think it was  
about 6 months ago.

If the powers that be will accept a change that removes Märchen I'll  
be more than happy to whip up a patch.

Wincent
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help