On Wed, Jun 1, 2011 at 8:28 AM, Chad <innocentkiller(a)gmail.com> wrote:
I *do* think we should enforce a 48hr "revert if
broken" rule. If you
can't be bothered to clean up your breakages in within 48 hours of
putting your original patch in, it must not have been very important.
Officially speaking, since forever we have had an "it's acceptable to revert
on sight if broken" rule.
If you encounter something broken, correct responses include:
* revert it immediately to a known good state; send an explanation to the
author about what's wrong (usually a CR comment and reopening a 'fixed'
bugzilla entry)
* figure out the problem and fix it; send follow-up explanation to the
author (usually a CR comment and a Bugzilla comment if it's a followup on a
bug fix)
* take a stab at the problem and recommend a solution to someone else who
will fix it straight away; including a note on CR or Bugzilla summarizing
your IRC discussion is wise in case it ends up getting dropped
* tag it fixme and ask someone more familiar with the stuff than you to
handle it
-- brion