[PATCH 0/3] Fix mixing of excludes/includes in abi_match
sgallagh at redhat.com
Mon Nov 5 12:31:38 MST 2012
On 11/05/2012 01:36 PM, Jelmer Vernooij wrote:
> This patchset adds a couple of unit tests for the write_vscript_abi() function,
> and fixes a recent regression in that function.
> ldb uses a abi_match of '!ldb_*module_ops !ldb_*backend_ops ldb_*'.
> After a recent change (9c3e294400234ebdf9b98031bae583524fd0b0ac) to wafsamba
> this line causes symbols with the prefixes rep_ and map_ to be public. This
> patchset fixes wafsamba to no longer expose these symbols, as it did prior
> to September.
> Note that ldb 1.1.12 was released with the broken version of samba_abi,
> and thus includes some public symbols that shouldn't actually be public.
> Jelmer Vernooij (3):
> wafsamba.samba_abi: Refactor abi_write_vscript to take file argument.
> wafsamba.samba_abi: Add basic unit tests.
> wafsamba.abi: Fix abi_match with both excludes and includes.
> buildtools/wafsamba/samba_abi.py | 30 +++++++++-------
> buildtools/wafsamba/tests/test_abi.py | 67 +++++++++++++++++++++++++++++++++++
> 2 files changed, 84 insertions(+), 13 deletions(-)
Ack to all three patches.
I've tested on Fedora that this eliminates the unexpected public ABI.
More information about the samba-technical