<div dir="ltr">Hey Kendy,<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 3, 2017 at 10:27 AM, Jan Holesovsky <span dir="ltr"><<a href="mailto:kendy@collabora.com" target="_blank">kendy@collabora.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
I've heard already from 2 people that it is not possible for them to<br>
push other people's patches to gerrit; they get an error like:<br>
<br>
remote: ERROR:  In commit XXXXXXXXXXXXXXXXXXXXXXXXXXXXXX<wbr>XXX<br>
remote: ERROR:  author email address <a href="mailto:YYY@YYY.ORG">YYY@YYY.ORG</a><br>
remote: ERROR:  does not match your user account.<br>
<br>
I wonder - what setting needs to be set for them so that they can push<br>
other people patches?  Or do they need to get the full push rights?<br>
<br>
And in general - what's the reason for such a restriction?  Why somebody<br>
cannot eg. backport somebody else's patch to a stable branch, even if<br>
they don't have full commit access yet (?) - the patch will be reviewed<br>
anyway...<br>
<br></blockquote><div><br><br></div><div>That should be the <span class="gmail-com-google-gerrit-client-admin-PermissionEditor_BinderImpl_GenCss_style-name">"Forge Author Identity" part in <a href="https://gerrit.libreoffice.org/#/admin/projects/Dev-ACL-Template,access">https://gerrit.libreoffice.org/#/admin/projects/Dev-ACL-Template,access</a><br><br><br></span></div><div><span class="gmail-com-google-gerrit-client-admin-PermissionEditor_BinderImpl_GenCss_style-name">At least for me the interpretation was always that it simplifies the lives of reviewers. If a patch has been committed by someone who is not the author and I can not be 100% sure that the license is correct I need to spend a lot of time researching the license situation myself. Additionally I'm often not very thrilled when people who have not understood a patch are back porting them to a stable branch. IMHO backporting should always either be done by someone who has the experience in that part of the code, has written the patch or has reviewed the patch for master.<br><br></span></div><div><span class="gmail-com-google-gerrit-client-admin-PermissionEditor_BinderImpl_GenCss_style-name">Regards,<br></span></div><div><span class="gmail-com-google-gerrit-client-admin-PermissionEditor_BinderImpl_GenCss_style-name">Markus<br></span></div></div></div></div>