<div dir="ltr">There's a patch against the coder module waiting to be reviewed (I know it's not core), but this patch will add functionality to coder which will allow you to run a code review on patches. You can paste in the patch, or just provide the URL to it. It might help speed up the process slightly.<br>
<br>Cheers,<br>Stella<br><br><div class="gmail_quote">On Wed, Aug 13, 2008 at 10:55 PM, Yuval Hager <span dir="ltr"><<a href="mailto:yuval@avramzon.net">yuval@avramzon.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Wednesday 13 August 2008, Nathaniel Catchpole wrote:<br>
> So if you already have HEAD installed (I pretty much only clean my<br>
> database before or after reviewing a patch with an upgrade path or<br>
> schema change), you can do this:<br>
><br>
> // clean your HEAD of old changes.<br>
> cvs -q diff | patch -p0 -R<br>
><br>
> wget <a href="http://drupal.org/files/issues/somepatch.patch" target="_blank">http://drupal.org/files/issues/somepatch.patch</a><br>
><br>
> patch -p0 < somepatch.patch<br>
><br>
> 1 out of 1 hunks failed etc.<br>
><br>
> Mark to code needs work with 'needs a re-roll'.<br>
<br>
</div>Can't this process be automated? I think I remember something about a process<br>
that scans for stale patches - does it still exist?. The<br>
close-fixed-after-two-weeks bot seems to be doing a very nice work!<br>
<font color="#888888"><br>
--<br>
Yuval Hager<br>
[@] <a href="mailto:yuval@avramzon.net">yuval@avramzon.net</a><br>
</font></blockquote></div><br></div>