[PATCH 0/3] Fix mixing of excludes/includes in abi_match
Andrew Bartlett
abartlet at samba.org
Mon Nov 5 13:11:28 MST 2012
On Mon, 2012-11-05 at 19:36 +0100, 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(-)
This looks good, and I'm really glad to see it has tests. Do we need to
bump the ldb version and re-release, with a forced --abi-update?
Andrew Bartlett
--
Andrew Bartlett http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
More information about the samba-technical
mailing list