merge request 187 on GitLab

Rowland Penny rpenny at samba.org
Thu Jan 17 18:02:11 UTC 2019


On Thu, 17 Jan 2019 09:54:46 -0800
Jeremy Allison via samba-technical <samba-technical at lists.samba.org>
wrote:

> On Thu, Jan 17, 2019 at 03:57:26PM +0100, Martin Krämer via
> samba-technical wrote:
> > Hi all,
> > 
> > some days ago I have created a merge request for a samba-tool patch
> > on GitLab.
> > I created this based on instructions from:
> > https://wiki.samba.org/index.php/Samba_CI_on_gitlab
> > and
> > https://wiki.samba.org/index.php/Using_Git_for_Samba_Development
> > 
> > Never the less this is my first contribution to samba and my first
> > work with GitLab - I am not sure if I have done everything correct
> > or if there are further actions required by me to be merged.
> > 
> > The merge request is:
> > https://gitlab.com/samba-team/samba/merge_requests/187
> > 
> > Thanks for instructions
> 
> Most of the Samba Team members still look for new patches
> as attachments on the samba-technical mailing list. It's
> always good to send them there as well as on gitlab (which
> we use mostly for CI work, not merging).
> 
> Having said that your patch looks good to me !
> 
> https://gitlab.com/samba-team/samba/merge_requests/187.patch
> 
> Reviewed-by: Jeremy Allison <jra at samba.org>
> 
> Can I get a second Team reviewer ?
> 
> Thanks a *lot* for helping us with Samba !
> 
> Jeremy.
> 

+RB

Just don't expect me to push it, we don't want things breaking :)

Rowland



More information about the samba-technical mailing list