Your Tombstone Reanimation branch
kamenim at samba.org
Mon Dec 15 19:06:33 MST 2014
On Tue, Dec 16, 2014 at 12:47 AM, Andrew Bartlett <abartlet at samba.org>
> On Tue, 2014-12-16 at 09:25 +1300, Andrew Bartlett wrote:
> > On Mon, 2014-12-15 at 17:46 +1300, Andrew Bartlett wrote:
> > > Kamen,
> > >
> > > Garming and I did some work to review your tombstone-reanimation-wip
> > > brnach. I've pushed it back to
> > >
> with some TODO markers (metze style).
> > >
> > > We need to go over it again, and to run the tests, but it is an
> > > impressive lump of work and I look forward to seeing it landed!
> > I've run a full make test, and the ldap.secdesc.python test fails. I
> > attach the summary file, and the below is the failing subunit. Can you
> > look into this please?
> The attached patch fixes the issue. See the markers in the branch.
> I'm now re-running a full make test.
> Some of the patches need you to sign off on them again, can you look
Thank you very much for your review and fixing the 'make test' issue!
I am building a dev machine (kind of) so tomorrow, hopefully, I will be
able to address your comments.
As a stretch goal, I will address your concerns for env variable names.
As you suggested, I will prefix env variables used by tests with TEST_
> Andrew Bartlett
> Authentication Developer, Samba Team http://samba.org
> Samba Developer, Catalyst IT
More information about the samba-technical