
On Tue, Apr 26, 2016 at 10:23:36AM -0600, Stephen Warren wrote:
On 04/25/2016 05:22 PM, Tom Rini wrote:
Lawyers can argue, but projects have guidelines. I mean heck, I've see you remind people to fix the include order in new patches. Do they need to add an NVIDIA copyright notice too? No, of course not.
Well honestly all patch review does contribute to the creative process and affect the final result, so I don't actually think it'd be that much of a stretch that a reviewer's copyright applies to the final patch; they were involved in the creation of it. Perhaps Reviewed-by tags or the mailing list history would be enough evidence of this though, if such were needed later.
And this is why I love patchwork and hope that as it starts to handle series more directly it will also handle "For the series, ..." messages in addition to collecting up Acked/Reviewed/Tested by lines. These are important for history of changes to the codebase.