Progress on GitHub and alternatives
Andrew Bartlett
abartlet at samba.org
Mon Sep 7 00:35:18 UTC 2015
On Fri, 2015-09-04 at 11:07 -0700, Jeremy Allison wrote:
> On Fri, Sep 04, 2015 at 06:12:11PM +1200, Andrew Bartlett wrote:
> > On Tue, 2015-08-18 at 17:28 +1200, Andrew Bartlett wrote:
> > > G'Day,
> > >
> > > For the past while it has concerned me that we need a better
> > > review
> > > system and that we should have a mirror and be accepting
> > > contributions
> > > via github.
> > >
> > > To be clear, github isn't really a review system, but it has a
> > > massive
> > > user community that we want to try to better connect with.
> > >
> > > The plan is to make every github pull request turn into a mail on
> > > the
> > > mailing list, so that we keep our mailing-list based workflow.
> >
> > > The URL for the official Samba Team repo is:
> > > https://github.com/samba-team/samba
> >
> > I've done this now, and we have some test notification scripts
> > sending mail to testlist.
> > You can see how this looks at
> > https://lists.samba.org/archive/testlist/2015-September/thread.html
> > It needs a lot of work to be really nice, but it is a start. Pull
> > request on https://github.com/abartlet/gh-mailinglist-notifications
> > are
> > very, very welcome.
> > I'm also trying to get another tool working:
> > https://github.com/abartlet/pull-request-mailer
> > This will post patches to the list, but for now it times out trying
> > to
> > clone the samba repo, so needs some love.
>
> Great work Andrew, thanks for doing this !
I have both scripts up and running. What I need now is some feedback
from you and the rest of the community.
I need to know which of the scripts (or both) to run long term. Look
here:
https://lists.samba.org/archive/testlist/2015-September/thread.html
and tell me what you like, and what you don't. If you want to see the
actual mail output, the subscription page is here:
https://lists.samba.org/mailman/listinfo/testlist
I also need folks to push their own patches to github, make pull
requests and comment on the workflow.
Currently, the service running git-format-patch (pull-request-mailer)
*will* automatically CC mails to those listed as From and Signed-off-by
in patches in pull requests. Let me know if that would be an issue.
The other script, gh-mailinglist-notifications attaches the patches to
just one mail, which I understand may be a preference.
Finally, we need to figure out how we will communicate back to the
author, as sadly replies to these mails won't make it back to github.
Either a few of us become members of the project and get the direct
pull request mails as well (that can be replied to), or we can leave
pull-request-mailer to write a message like this one
https://github.com/samba-team/samba/pull/7
Thanks,
Andrew Bartlett
--
Andrew Bartlett
https://samba.org/~abartlet/
Authentication Developer, Samba Team https://samba.org
Samba Development and Support, Catalyst IT
https://catalyst.net.nz/services/samba
More information about the samba-technical
mailing list