tech-userlevel archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: Private symbols in libpcap, libmagic, libexpat



At Tue, 17 Dec 2024 23:52:11 +0000, Taylor R Campbell <riastradh%NetBSD.org@localhost> wrote:
Subject: Private symbols in libpcap, libmagic, libexpat
>
> Currently we build libpcap, libmagic, and libexpat with various
> private symbols exported accidentally because our build system was
> missing -fvisibility=hidden and associated CPPFLAGS.
>
> The attached patch fixes this (and as a side effect should unbreak the
> MKINET6=no build).

Awesome!  Thanks!


But also libldap now fails because of, I think, MKKERBEROS=no (and thus
USE_KERBEROS=no):

--- /Volumes/work/woods/g-NetBSD-src/external/bsd/openldap/lib/libldap_r/ldap_r.expsym  2024-12-08 17:36:22
+++ libldap_r.so.6.0.actsym     2024-12-17 17:25:06
@@ -164,11 +164,6 @@
 ldap_int_error_init
 ldap_int_flush_request
 ldap_int_global_options
-ldap_int_gssapi_close
-ldap_int_gssapi_config
-ldap_int_gssapi_get_option
-ldap_int_gssapi_mutex
-ldap_int_gssapi_set_option
 ldap_int_hostname
 ldap_int_hostname_mutex
 ldap_int_inet4or6
libldap_r.so.6.0: error: actual symbols differ from expected symbols
--- libldap_r.so.6.0.diffsym ---

*** Failed target: libldap_r.so.6.0.diffsym
*** In directory: /Volumes/work/woods/g-NetBSD-src/external/bsd/openldap/lib/libldap_r


I'm guessing (with the help of ldap-int.h) that the "ldap_int_" prefix
on those symbols mean "internal to ldap" though I also see "ldap_pvt_"
prefixes in the code....


> So it might be worthwhile to pull up the change _without_ a major bump
> even though that technically violates the rules.

I think not doing version major bumps for this is the right choice,
probably all-around, but certainly for libpcap and libexpat.

It is arguably the more conservative choice as well since all it is
doing is to nails down what's really supposed to be in a library's ABI
to just what's in its API, and as you point out, at least for both
libpcap and libexpat, this is the way upstream was already building
anyway.


--
					Greg A. Woods <gwoods%acm.org@localhost>

Kelowna, BC     +1 250 762-7675           RoboHack <woods%robohack.ca@localhost>
Planix, Inc. <woods%planix.com@localhost>     Avoncote Farms <woods%avoncote.ca@localhost>

Attachment: pgp8zu8sOfvW2.pgp
Description: OpenPGP Digital Signature



Home | Main Index | Thread Index | Old Index