Re: [Ksummit-discuss] [CORE TOPIC] Recruitment (Reviewers, Testers, Maintainers, Hobbyists)

[Mark Brown <broonie@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1164 bytes --]

On Thu, Jul 16, 2015 at 07:26:23PM +0530, Sudip Mukherjee wrote:

> Then in my opinion, it will be better if every maintainer has a script
> which will reply automatically on receiving a patch that if the patch is
> not applied within xxx days then please send a ping / resend the patch.

That then sets the expectation that you're going to do something with
the patch which is fine if it's relevant but there's lots of moderately
common cases with things like changes that touch multiple subsystems
(eg, for mfds or for adding a device driver and the DT using it
simultaneously) which result in things like resends of patches that
you've already reviewed due to a change earlier in the series and which
could get awfully noisy and/or confusing for submitters as a result.

To be reliable you'd off the top of my head need to start doing things
like parsing the patch to make sure it was for a relevant subsystem and
that it hadn't already got a reviewed/acked/whatever tag, sending a
different message if it wasn't something you'd expect to handle
yourself.  Personally I don't know that I trust my skills in appropriate
scripting languages to write that safely.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]