Re: [PATCH] net/sunrpc/xprtsock.c: some common code found

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

 



On Sat, 2009-02-07 at 18:56 +0200, Ilpo Järvinen wrote:
> Can you please tell if you're still under impression that there wasn't
> any changelog? What about those other guys who raised their concerns?
> (I can certainly understand why this impression might develop but it's a 
> problem in the canonial patch format, not in my actual changelog, I even 
> need to circumvent ^--- by adding a space the to prevent tools for making 
> a mistake). It will make a lot more sense after getting applied.
> ...Because there certainly was a changelog, and I didn't have any 
> diffstat output there in the changelog?!? The output that *is* in the 
> changelog is supposed to *help* the reviewer to see what will be still 
> necessary to keep in the separate functions, but I guess people missed 
> that because the changelog looks too similar to what you will categorize 
> as patch (diff-funcs uses output which is familiar to all who who deal 
> with patches, and that's for purpose!).

If Chuck and Bruce are happy with it, then there is no problem, however
I have yet to see an acked-by from either one of them. My point is that
the process for accepting patches into the kernel involves ensuring that
there is consensus. It does not allow for one group to ride roughshod
over the objections of another.

   Trond
-- 
Trond Myklebust
Linux NFS client maintainer

NetApp
Trond.Myklebust@xxxxxxxxxx
www.netapp.com
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux