Patchwatcher online

Adam Petaccia adam at tpetaccia.com
Tue Aug 12 15:39:03 CDT 2008


On Mon, 2008-08-11 at 22:24 -0700, Dan Kegel wrote:
> Dmitry Timoshkov <dmitry at codeweavers.com> wrote:
> > "Zachary Goldberg" <zgold550 at gmail.com> wrote:
> >
> >> Policy is that all patches should be independent, no?
> >
> > There is no such a policy. Dependent patches are marked as
> > 1/xx, 2/xx, ... xx/xx.
> 
> That's a patch series, and patchwatcher handles that ok.
> 
> There's another kind of dependent patch, where somebody
> says "This requires Harold's patch from yesterday".
> Patchwatcher probably isn't going to handle that ever.

What about checking for a string in the email like "patchwatchignore";
if for some reason the patch is known to cause a failure the e-mail
might read like:
	patchwatchignore
	This depends on Harald's patch from yesterday...
	[patch]

This also might be useful for patches which update comments or README
files, etc.




More information about the wine-devel mailing list