On Thu, 2015-04-16 at 15:37 -0600, Jason Gunthorpe wrote:
> On Fri, Apr 17, 2015 at 12:26:47AM +0300, Or Gerlitz wrote:
> > On Thu, Apr 16, 2015 at 11:13 PM, Jason Gunthorpe
> > <[email protected]> wrote:
> > > On Wed, Apr 15, 2015 at 08:39:13AM -0400, Doug Ledford wrote:
> > >
> > >> Let’s take this as a starting point.  If there are other patches
> > >> people think should make 4.1, please bring them to my attention and
> > >> I’ll add them for review and possible addition to the list.
> > >>
> > >> git://github.com/dledford/linux.git for-4.1
> > >
> > > I noticed a few patches didn't get Reviewed-By's etc from the mailing
> > > list annotated..
> > 
> > Every patch need not have a Reviewed-By annotation, it's perfectly up
> > to the maintainer judgement to make sure there was either consensus,
> 
> More clearly: There were several patches where a Reviewed-By/etc was
> *provided* on-list but did not get captured in the git commit.

Sure, that's what I figured you meant.  That most likely means that I
pulled the bundle from patchworks before their review was captured.  It
means I would need to either re-pull the bundle or manually add the
review.

> AFAIK, capturing provided tags is considered best practice as a way to
> recognize and retain contributors.

Agreed.  I'll double check them tomorrow.

-- 
Doug Ledford <[email protected]>
              GPG KeyID: 0E572FDD


Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to