On Sun, 2005-03-06 at 23:50 -0800, Paul Jackson wrote: > Andres wrote: > > An obvious fix is an obvious fix. > > Perhaps in theory. But in practice, any fix bears some risk. >
Of course; no one's arguing that (things that depend on broken behavior, corner cases, etc). In practice, however; an obvious fix is still an obvious fix. :) What I mean is, if there's an unknown (ie, due to hardware behavior, user input, etc), and it's not adding sanity checking or somehow ensuring that the data it's dealing with is of a certain form, then it's not really an obvious fix. Of course, part of the problem is that "obvious" is subjective. I know what *I* consider obvious; someone who works for a hardware company, and has access to hardware, specifications, and errata would have different criteria for what they consider obvious. Here's where that signed-off-by-5-people thing comes into play. > They have nothing against "obvious" fixes. But unless additional > criteria are also met, such fixes are for someone else to apply. > That's fine; I agree w/ the additional criteria of "it must be a build, oops, hang, or race fix". > > >> - It can not contain any "trivial" fixes in it (spelling changes, > > >> whitespace cleanups, etc.) > > > > This and the "it must fix a problem" are basically saying the same thing. > > Not at all. Let me put it this way. > > If a change that fixes a problem is included in a patch with another > change that makes trivial changes (typo fix, say), the patch will > be rejected. > That's fine as well; that's covered by the "it must only fix one thing" rule, which I also agree w/. The "no trivial fixes" thing is still redundant; a) a patch must contain only one fix, and b) a patch may only fix a build error, oops, hang, or race. > The statement: > > "It must fix a problem and it must _not_ contain anything else, > such as 'trivial' fixes." > > is _obviously_ not the same as: > > "It must fix a problem." > > (Notice how quickly even the obvious becomes unobvious ...;). > -- Andres Salomon <[EMAIL PROTECTED]>
signature.asc
Description: This is a digitally signed message part
