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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux