Granularity
Myk Melez
myk at mozilla.org
Thu Oct 24 20:48:18 UTC 2002
MattyT wrote:
>What really concerns me here is not the per-component issue, but the
>fact that we're discussing it *after* it's been checked in. And this is
>hardly the first time it's happened for me.
>
It's common for me, too, but that's the price of working for a large,
active project and not spending my life reading bug mail. I regularly
notice check-ins with flaws I would have corrected, but the patches are
almost always good enough (TM), and I'd rather they go in than be
shunted through another bottleneck that I still won't have enough time
to keep up with.
>Currently everyone is working on their own enhancements, and they only
>get one review. This means a large number of people are potentially in
>the dark about the implementation of features, and complaints arise when
>it's essentially too late.
>
Even patches that only need one review frequently get several sets of
experienced eyes on them (and lack of a comment doesn't mean lack of
attention). It doesn't matter how many people are in the dark as long
as a sufficient number aren't, which could be as few as one if that
person is an accomplished Bugzilla hacker.
>Many freed software projects have a single mind who ultimately controls
>what goes in and what doesn't - Linux for example. We don't have this,
>Dave probably does not have the time to design review all patches. But
>it's still important that we have vision and don't let Bugzilla become
>any more crazy and unorthogonal than it already is in some places.
>
I agree, but our code quality has been consistently increasing, so I
don't think we have a "crazy and unorthogonal" problem, and making flags
component-specific certainly isn't an example of it, since intentional
deviation from a norm for a good reason is a good thing.
>What I think needs to be done, is to have ALL enhancements thoroughly
>summarised and the summary posted to this list.
>
In other words, design by committee. No thanks.
>We have certainly done this sort of thing before, but not for everything
>(or even most things), and I think we should expect it to be.
>
>
I disagree. Design discussions on this list have had some very bad
results, f.e. the "sending mail" patch which got bogged down on design
so much that nothing has been done, even though all the proposed
solutions were good enough and any of them would have been better than
no change at all, which is what we have now.
I'd much rather have Dave approve patches or delegate such approval to
the component owners, with discussions on the list reserved for cases
where developers need help or can't agree.
-myk
More information about the developers
mailing list