From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Philipp Stephani
On Mon, 09 Nov 2015 16:16:31 +0000 Philipp Stephani <p.stephani2@gmail.com= a>> wrote:
PS> I don't need to sign the papers because my employer (Google) has= a special
PS> agreement with the FSF. Please use my work address phst@google.com and
PS> "Copyright Google Inc." for my contributions. I already ha= ve a few patches
PS> in master under that address.
Thank you.=C2=A0 I can see that your existing patches didn't need the "Copyright Google Inc." in the commit message, so I think we can = skip it here.
On Mon, 9 Nov 2015 20:58:59 +0100 Aur=C3=A9lien Aptel <aurelien.aptel+emacs@g= mail.com> wrote:
AA> I've made a clean new branch 'dynamic-modules-clean-1' o= n top of a
AA> recent master which you should be able to rebase easily. I've sp= lit my
AA> "working" branch into 6 Changelog-ready commits. I'm t= he author of the
AA> commits but a significant part of the code was also written by Phili= p
AA> Stefani (big thanks to him). Don't really know how to we can ref= lect
AA> that since it's all mixed up now.
AA> It's on github. You can rework the commit messages if you want a= s I'm
AA> not very good at this.
Aur=C3=A9lien, it looks great.=C2=A0 Thank you for the work.
I have only two requests before I merge:
1) Can you add this header line to the commits that need it (it may
require some checking, unfortunately):
Co-authored-by: Philipp Stephani <phst@google.com>
I *think* that's sufficient.
2) Since you're modifying some commits, can you remove the final period=
from the first line?=C2=A0 It's a suggestion in CONTRIBUTE but not requ= ired.
Otherwise I think it's ready to merge.
On Tue, 10 Nov 2015 20:07:36 +0000 Philipp Stephani <p.stephani2@gmail.com> wrote= :
PS> Thanks! Since the freeze is only days away, can we now start merging= these
PS> changes into the Savannah master?
Yes, I'll do the merge as soon as (1) above is resolved. Technically th= e
patch looks good.