[PATCH] Add ntlm_auth require-membership-of tests

Kamen Mazdrashki kamenim at samba.org
Tue Jul 15 05:33:58 MDT 2014


Hi Andrew,



On Tue, Jul 15, 2014 at 11:28 AM, Andrew Bartlett <abartlet at samba.org>
wrote:

> On Thu, 2014-07-10 at 13:40 +0200, Kamen Mazdrashki wrote:
> > Hi Garmin,
> >
> > Could you please fix leading white spaces in test_ntlm_auth_s3.sh
> > As far as I can see, we are using TAB in sh files also.
> >
> > Other that that, patch looks good to be
> > Reviewed-by: Kamen Mazdrashki <kamenim at samba.org>
> >
> > Cheers,
> > Kamen
> >
>
> Thanks,
>
> I had to tweak the second patch slightly (since the patch was generated,
> we got the winbindd in AD DC into master, so this works in one more
> configuration).  Please indicate your review on the second patch.
>
> Thanks for formatting the script :)
Reviewed and pushed to autobuild.


> Andrew Bartlett
>
> --
> Andrew Bartlett                       http://samba.org/~abartlet/
> Authentication Developer, Samba Team  http://samba.org
> Samba Developer, Catalyst IT
> http://catalyst.net.nz/services/samba
>
>


More information about the samba-technical mailing list