Messages in this thread | | | From | Stefan Monnier <monnier+lists/linux/kernel/news/@tequila.cs.yale.edu> | Subject | Re: 2.1.131: some quality thoughts | Date | 17 Dec 1998 19:06:18 -0500 |
| |
>>>>> "Linus" == Linus Torvalds <torvalds@transmeta.com> writes: > As far as I can tell, everybody agrees that CVS is being stupid in not > checking at least that "core" is a regular file before ignoring it.
Not quite: CVS defines the cvsignore patterns as applying to files and directories. It was meant to behave that way because you also want to ignore some directories (like RCS, CVS, CM (for SML/NJ users out there), ...). The real fix is hence to force the use of different patterns for dirs and for files (a trailing slash would be an obvious candidate for making such a distinction). But this is not backward compatible.
> My opinion is that you shouldn't work around bugs in your tool-chain, > you should fix them. Either by fixing CVS, or by using something else.
The problem is that cvsignore tweaks are a fact of life. You cannot expect a default .cvsignore setting to work everywhere for everyone, so a CVS user has to be aware of this and setup .cvsignore files as needed. Once this is done, dealing with the rare cases where a file pattern wrongly applies to a directory (as is the case here for `core') is really not a big deal. Fixing CVS as suggested above would only reduce the amount of work to setup linux under CVS by a really tiny fraction. That's why I never bothered to fix it and that's probably why noone else bothered to fix it.
Stefan
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/
| |