Re: [PATCH 1/2] libata: add CFA specific identify data words

[Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>]

On Monday 13 April 2009 22:28:17 Jeff Garzik wrote:
> Bartlomiej Zolnierkiewicz wrote:
> > As I see it, there was absolutely no need to rush things -- I simply didn't
> > get to processing newer submissions yet and I would just push Sergei's patch
> > in the next IDE pull request (which happened on March 13th).
> 
> That is an IDE-centric perspective.  It was a cross-tree depedency, and 
> _I_ should not need to wait for you, particularly when you are 
> constantly complaining about lacking time for IDE maintenance.

The patch had _zero_ IDE specific changes, you didn't have to wait for me.

You could have simply pushed the whole Sergei's patch yourself and indeed
this will make things more efficient for everybody.

> > Simple reminder/query about the patch status would be more than enough but
> > instead you wasted time for everybody by splitting libata part from Sergei's
> > patch (without updating patch summary+description which I had to fix) and
> > later forgotting about it.  Moreover instead of simply pushing forgotten
> > part yourself you requested Sergei to resubmit it and at the same time you
> > tried to put the blame about the whole situation on me.
> 
> The onus is always on the submittor to monitor their patches, resubmit, 
> etc.  It is simply not scalable to constantly ping submittors, keep 
> track of their individual development schedules, etc.

In the end it is your responsibility as the Maintainer to get their patches
merged (== keep the project flourishing).  Plus keeping in touch with people
submitting you patches and knowing (more-or-less) their individual schedules
is a key into properly planning merges or large scale changes.  Though YMMV
as there is no one perfect way of doing things befitting all people.

Thanks,
Bart