Re: Those ruddy punctuation fixes

From: Steven Cole (elenstev@mesatop.com)
Date: Thu Mar 06 2003 - 23:02:49 EST


On Thu, 2003-03-06 at 20:33, Martin J. Bligh wrote:
> >> Wait, this sounds like a conversation with the Mafia:
> >>
> >> "Pay us protection money."
> >> "Why do we need to pay you for protection?"
> >> "So we can protect you from criminals like ourselves."
> >
> > That's a ridiculous comparison and it weakens your argument. Leaving a
>
> Reductio ad absurdum is often enlightening.
>
> > potential problem in place rather than fixing it as I did would be the
> > passive-aggressive approach, not the other way around.
>
> But that's not exactly what you're doing - you're replacing one
> (very small) problem with another (very real) problem, the breakage
> of people's patches. Fixing up patches because of spelling
> errors is a total waste of developer's time.

Agreed.

>
> >> I'd rather solve this problem by making standalone spelling fixes and
> >> other cosmetic changes taboo. Cosmetic changes combined with actual
> >> useful code changes are fine with me. If you're risking breaking the
> >> build, there should be some benefit that justifies the risk.
> >
> > Breaking the build is a low probability (many hundreds of fixes and one
> > build break AFAIK) and low consequence failure (a build fix of that
> > nature is obvious and quickly and easily done).
>
> Breaking the build is indeed a low probability (assuming you compile
> test your tree). Breaking other people's patches is a high probablility.

So another approach to this is to offer cleanup services to willing
maintainers who don't have the time to do it themselves. If anyone
wants their section of the tree spell-corrected, I can do that with the
help of Dan's scripts (the easy part), and review the resulting diff for
inappropriate fixes (broken puns, changed meaning, broken compiles),
send that around to a small group (the spelling police squad) for
further review, and then send it back to the requesting maintainer, who
can /dev/null it entirely, or hack it up to his delight before sending
it Linuswards.

How's that?

Steven

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Fri Mar 07 2003 - 22:00:35 EST