Yeah, I have expected both sign it, if they worked together.<div><br></div><div>Daniel, in the future, add a Signed-off-by line to your patches.</div><div><br></div><div>Gonzalo<br><br><div class="gmail_quote">On Thu, Jul 5, 2012 at 3:33 AM, Sascha Silbe <span dir="ltr"><<a href="mailto:silbe@activitycentral.com" target="_blank">silbe@activitycentral.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">Gonzalo Odiard <<a href="mailto:gonzalo@laptop.org">gonzalo@laptop.org</a>> writes:<br>
<br>
> Why is<br>
><br>
> Signed-off-by: Caspar Bothmer <<a href="mailto:caspar@activitycentral.com">caspar@activitycentral.com</a>><br>
><br>
> if the patch was done by Daniel?<br>
<br>
</div>Because Caspar changed the patch. See the kernel patch submission guide<br>
[1], section 12) Sign your work:<br>
<br>
>> [...] Rule (b) allows you to adjust the code, but then it is very<br>
>> impolite to change one submitter's code and make him endorse your<br>
>> bugs. To solve this problem, it is recommended that you add a line<br>
>> between the last Signed-off-by header and yours, indicating the<br>
>> nature of your changes.<br>
<br>
<br>
It looks odd in this case because Daniel didn't sign off his own<br>
patch. The kernel maintainers simply wouldn't accept a patch that isn't<br>
signed off by the author, but as even Daniel Drake's and Simon's patches<br>
aren't always signed off, I can't really enforce that rule for other<br>
Sugar contributors.<br>
<br>
Sascha<br>
<br>
[1] <a href="https://www.kernel.org/doc/Documentation/SubmittingPatches" target="_blank">https://www.kernel.org/doc/Documentation/SubmittingPatches</a><br>
<div class="HOEnZb"><div class="h5">--<br>
<a href="http://sascha.silbe.org/" target="_blank">http://sascha.silbe.org/</a><br>
<a href="http://www.infra-silbe.de/" target="_blank">http://www.infra-silbe.de/</a><br>
</div></div></blockquote></div><br></div>