On Wed, Oct 07, 2015 at 10:21:33PM +0800, Fengguang Wu wrote: > On Wed, Oct 07, 2015 at 03:55:29PM +0200, Peter Zijlstra wrote: > > How about not building when there's no "^Signed-off-by:" at all? > > That's a good idea: no need to test quick demo-of-idea patches. > > > Even private build fails for patches like this -- esp. 3+ -- gets > > annoying real quick. > > > > Also note that this 'patch' has: $subject ~ /^Re:/, nor did it have > > "^Subject:" like headers in the body. > > That's good clues, too. So how about make the rule > > Skip test if no "^Signed-off-by:" and Subject =~ /^Re:/ > > For a patch posted inside a discussion thread, as long as it have > "^Signed-off-by:", I guess the author is serious and the patch could > be tested seriously.
Works for me; now hoping I will abide by my own suggested rules ;-) Thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
