[development] Re-check "code needs work" patches
Angela Byron
drupal-devel at webchick.net
Mon Nov 17 07:39:05 UTC 2008
HEAD was broken today for about 4 hours after the commit of
http://drupal.org/node/74645#comment-1110500. Because this patch was
missing a hunk from install.inc, testing bot correctly flagged it as
"code needs work" with a reason of "Failed to install HEAD."
Unfortunately, because the patch then got committed to HEAD, that meant
that *every* patch the testing bot tried thereafter failed for the same
reason. So over 180 patches got marked "code needs work" as their turns
came up in the testing queue. :\
The *good* news is that boombatower has some code ready that will test
vanilla HEAD after each commit and immediately stop processing more
testing results until this situation remedies itself, should it happen
again (and it probably will). It will also ping the dev list and the
core maitainers via email so that they can fix it ASAP.
The *bad* news is that we don't have a real easy way to undo these CNWs
so this will have to be a manual process. :\ http://pastebin.ca/1258476
has a list of all of the affected issues. If your patch is on the list,
please set it back to CNR and click the "Submit for re-testing" link on
the erroneous failure to get it re-tested.
A good rule of thumb in the future is never set a patch RTBC until it's
been pinged back from the testing bot, which will ensure:
a) The patch applies
b) The patch doesn't cause Drupal to fail to install
c) The patch doesn't have a PHP syntax error
d) All of the automated tests pass
-Angie
More information about the development
mailing list