[PATCH 0/3] Fix mixing of excludes/includes in abi_match

Stephen Gallagher 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 mailing list