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

[Mark Brown <broonie@kernel.org>]

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

On Wed, Jul 15, 2015 at 09:20:43PM -0400, Steven Rostedt wrote:
> Sudip Mukherjee <sudipm.mukherjee@gmail.com> wrote:

> > Another problem, when a newbie tries to move out of staging to some other
> > subsystem he likes, the maintainer may not be that much responsive. Just
> > for example, i submitted a patch on November, 2014 and I am yet to receive
> > a reply or review to that and the patch was not a style correction patch.

> BTW, it should always be OK to ping the maintainer if they ignore a
> patch. I believe one week is a good time to wait. And again in another
> week if they still do not reply. I know a few maintainers that think if
> they get to a patch that is old and the author never pinged them, they
> think the author doesn't think that patch is too important and they
> just delete it.

Please don't encourage people to send content free pings bit instead
resend it - a content free ping mostly just adds to mail volume which is
pretty much the original problem.  If the patch has actually been lost
then a resend is going to be needed anyway and if not then it's mostly
just adding to mail volume.  With a lot of mail clients (including mutt
which I use) the nag will get threaded in with the original patch buried
back in the mailbox and not even be seen if that's still sitting waiting
for handling.

> I'm sure all maintainers are fine with a friendly ping after a week.
> Don't send a ping before then, because maintainers are busy, and may
> get annoyed by being too pushy.

Not me, they tend to just annoy me for the reasons above.  Resends are
useful but just straight up pings not so much.

I'd say a couple of weeks is a better lower limit than a single week
unless it's a very serious bugfix, a week could easily be a holiday or
whatever and larger patches or patch serieses do take time to review.

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