Releases, locking and ldb

Stefan Metzmacher metze at
Sat Jul 1 22:18:08 UTC 2017

Am 30.06.2017 um 23:47 schrieb Andrew Bartlett:
> On Fri, 2017-06-30 at 23:00 +0200, Stefan Metzmacher wrote:
>> Am 30.06.2017 um 22:23 schrieb Andrew Bartlett via samba-technical:
>>> On Fri, 2017-06-30 at 23:03 +1200, Andrew Bartlett via samba-technical
>>> wrote:
>>>> Perhaps I'm tempting fate too much making changes after we got it all
>>>> working, but I've attached my current thoughts.  I wish I could do more
>>>> to actually remove the nested transactions, but now is not the time.
>>> Indeed, fate tempted.  Even this smaller 'safe' set fails autobuild,
>>> which honestly scares me a little.  However we can't go down this rat-
>>> hole today. 
>> I've added the tdb_transaction_active(), but I'm not sure
>> it will be the way to fix things in ldb. There's more discussion
>> required. If we get something useful in the next weeks it can
>> be backported as bugfix, it's not required to fix all bugs before
>> 4.7.0rc1.
>> passed ' samba samba-systemkrb5' twice. I'll some
>> complete autobuilds now.
> I've fired off 5 more of that branch in the Catalyst Cloud.
>> I'll fix up the commit message and comments around the tests.
> Thanks!
>> The important thing that's not tested yet is that the
>> commit release works to reject builds with older samba versions.
> Understood.  I'm going to try hard to spend the weekend with the
> family, but perhaps Jeremy gets a chance before my Monday.

Ok, with the "ldb:wscript: define
the autobuild target "samba-libs" triggers this:

In file included from ../source4/dsdb/common/util.c:27:0:
/m/metze/W/b863444/prefix/samba-libs/include/ldb_module.h:68:2: error:
#error "Samba < 4.7 is not compatible with this version of ldb due to
assumptions around read locks"
 #error "Samba < 4.7 is not compatible with this version of ldb due to
assumptions around read locks"
Waf: Leaving directory `/memdisk/metze/W/b863444/samba-libs/bin'
Build failed:  -> task failed (err #1):
        {task: cc util.c -> util_6.o}

So it seems to work.

I'll finish up this stuff based on your ldb-safe-locking-13 branch.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <>

More information about the samba-technical mailing list