[PATCH] Re: Help wanted on AD DC winbindd work
asn at samba.org
Thu Jun 5 06:43:38 MDT 2014
On Wednesday 04 June 2014 22:35:07 Andrew Bartlett wrote:
> On Tue, 2014-06-03 at 13:45 +0200, Andreas Schneider wrote:
> > On Monday 26 May 2014 16:05:11 Andreas Schneider wrote:
> > > > > Jeremy, I know how much you prefer attached patches, so here is the
> > > > > set
> > > > > I'm looking to get into master.
> > > > >
> > > > > git://git.samba.org/abartlet.git ad-dc-winbindd-pre
> > > > > https://git.samba.org/abartlet/samba.git/?p=abartlet/samba.git/.git;
> > > > > a=sh
> > > > > or
> > > > > tlog;h=refs/heads/ad-dc-winbindd-pre
> > > > >
> > > > > Please review/push
> > > >
> > > > I updated the patches and branches, and they now pass a full
> > > > autobuild.
> > >
> > > I've started with a review of these patches.
> > Hi,
> > I've looked through the patches. What is marked with TODO needs some
> > changes. Patches marked with REVIEW I need to look at them again. I
> > haven't understood them in detail yet.
> > https://git.samba.org/?p=asn/samba.git;a=shortlog;h=refs/heads/REVIEW_ad-d
> > c-winbindd-pre
> I've fixed up the issues you raised, see
> Hopefully this addresses your concerns.
> This code order follows what metze has done for the SamLogon code, on
> which it was based.
> Andrew Bartlett
I've added more RB+ and updated the branch. I'm sorry but I'm not able to
finish the review. I'm leaving for vacation. Maybe someone else can jump in
and finish it:
4 patches need review.
Andreas Schneider GPG-ID: CC014E3D
Samba Team asn at samba.org
More information about the samba-technical