Thanks, I'll update the docs explaining the *-By: a bit better and send it out for review.
George > -----Original Message----- > From: mesa-dev [mailto:[email protected]] On > Behalf Of Ilia Mirkin > Sent: Wednesday, November 23, 2016 10:59 AM > To: Kyriazis, George <[email protected]> > Cc: ML mesa-dev <[email protected]>; Emil Velikov > <[email protected]> > Subject: Re: [Mesa-dev] [PATCH v4 09/10] gallium: swr: Added swr build for > windows > > On Wed, Nov 23, 2016 at 11:10 AM, Kyriazis, George > <[email protected]> wrote: > > > >> -----Original Message----- > >> From: mesa-dev [mailto:[email protected]] On > >> Behalf Of Emil Velikov > >> Sent: Wednesday, November 23, 2016 10:02 AM > >> To: Ilia Mirkin <[email protected]> > >> Cc: Kyriazis, George <[email protected]>; ML mesa-dev <mesa- > >> [email protected]> > >> Subject: Re: [Mesa-dev] [PATCH v4 09/10] gallium: swr: Added swr > >> build for windows > >> > >> On 23 November 2016 at 15:39, Ilia Mirkin <[email protected]> wrote: > >> > On Wed, Nov 23, 2016 at 10:20 AM, Kyriazis, George > >> > <[email protected]> wrote: > >> >> Thanks Emil, > >> >> > >> >> One thing that I am clear about is: what is the difference between > >> Reviewed-by:, Acked-by:, etc. It sounds like they do the same thing, > >> however in the documentation they are worded as if they mean > >> different things. > >> > > >> > Reviewed-by = I am familiar with this code, and I have carefully > >> > looked over your change and am fairly sure that it will work > >> > properly Acked-by = I glanced at the change and it seems reasonable > >> > OR I looked carefully at the change, but am not overly familiar > >> > with the surrounding code > >> > > >> Exactly this. Thanks Ilia! > >> > > Thanks! In the docs there is also a reference to "Signed-off-by". Thoughts > on that one? > > That one's a little trickier - in the Linux Kernel, it has real significance - > > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Document > ation/SubmittingPatches#n420 > > However as we have no comparable document in the mesa tree, it loses > some of its luster. I think a lot of mesa hackers are also kernel hackers, and > the practice has sort of flowed over. There's no hard rule about it though. > > Cheers, > > -ilia > _______________________________________________ > mesa-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
