[development] What about reviewing patches?

Khalid Baheyeldin kb at 2bits.com
Thu Aug 14 00:56:32 UTC 2008


On Wed, Aug 13, 2008 at 5:55 PM, Yuval Hager <yuval at avramzon.net> wrote:

> On Wednesday 13 August 2008, Nathaniel Catchpole wrote:
> > So if you already have HEAD installed (I pretty much only clean my
> > database before or after reviewing a patch with an upgrade path or
> > schema change), you can do this:
> >
> > // clean your HEAD of old changes.
> > cvs -q diff | patch -p0 -R
> >
> > wget http://drupal.org/files/issues/somepatch.patch
> >
> > patch -p0 < somepatch.patch
> >
> > 1 out of 1 hunks failed etc.
> >
> > Mark to code needs work with 'needs a re-roll'.
>
> Can't this process be automated? I think I remember something about a
> process
> that scans for stale patches - does it still exist?. The
> close-fixed-after-two-weeks bot seems to be doing a very nice work!
>

This is supposed to go here at some point http://testing.drupal.org/
-- 
Khalid M. Baheyeldin
2bits.com, Inc.
http://2bits.com
Drupal optimization, development, customization and consulting.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.drupal.org/pipermail/development/attachments/20080813/d379bdad/attachment.htm 


More information about the development mailing list