[PATCHSET] cwrap for 'make test'

Stefan (metze) Metzmacher metze at samba.org
Wed Apr 16 07:03:42 MDT 2014


Hi,

I think I finally finished my review of this.

Please have a look at
https://git.samba.org/?p=metze/samba/wip.git;a=shortlog;h=refs/heads/master4-asn

>> I'm finally done with migrating our source tree to use the preloadable 
>> wrappers. I check if a system version of the wrappers is installed and 
>> available with the needed version. If none is installed it will use the 
>> bundled version.
>>
>>
>> What are the advantages of the preloadable version?
>> * You can run 'make test' with a optimized build.
>> * It works with each system library and binary.
>> * We can use external projects in future (MIT KDC)
>>
>> You can find the patchset here:
>> https://git.samba.org/?p=asn/samba.git;a=shortlog;h=refs/heads/cwrap
>>
>>
>> I'm looking forward to discuss the patches with you!
> 
> One issue I raised last time that is un-addressed is the change to the
> ldb module code, which worked around issues with samba_dlz.so being
> linked against internal Heimdal, and BIND9 itself being linked against
> the system MIT. 
> 
> I understand (but do not agree with) your position that having both in
> the same binary is a problem, but this is essentially what every single
> Samba installation using BIND9 does, and does sucessfully, and we can't
> break it. 

Sure, but is this really a problem anymore, as we have symbol versioning?

For now we use just disable RTLD_DEEPBIND during make test,
but don't the default for now:
https://git.samba.org/?p=metze/samba/wip.git;a=commitdiff;h=5589b45135418a47d41df188526552fffca99c9e
https://git.samba.org/?p=metze/samba/wip.git;a=commitdiff;h=acd9bf54595e2f6af7a00b9195439de4fa7fb329

I'm currently doing some private autobuilds with this branch.

If won't be new problems, I'd push this tomorrow if nobody objects.

metze


More information about the samba-technical mailing list