Naz Gassiep <[EMAIL PROTECTED]> writes: > Andrew Dunstan wrote: >> How do we know in advance of reviewing them that they are sane?
> Same way as happens now. I would assume this mechanism would only be > applied to patches that had already been approved to contrib, or some > other measure that can be used to isolate only those patches that we > *expect* to already be working. What is "approved to contrib"? The problem here is that having reasonable certainty that a patch is not malicious requires having gone over it in some detail; at which point you might as well apply the thing. Or if you didn't apply it, you bounced it for reasons that are unlikely to have anything to do with needing more automated testing. ISTM this idea can only work if we have a "second tier" of reviewers who are considered good enough to vet patches as safe, but not quite good enough to certify them as commitable. I'm not seeing a large pool of people volunteering to hold that position --- at best it'd be a transitory state before attaining committerdom. If you are relying on a constant large influx of new people, you are doomed to failure (see "Ponzi scheme" for a counterexample). regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 5: don't forget to increase your free space map settings