libgnutlsxx link problems

[Simon Josefsson]

Boyan Kasarov <bkasarov at gmail.com> writes:

> Hello,
> I am sending two patches to add symbol visibility support.
>
> [PATCH 1/2] Import lib-symbol-visibility from gnulib
> [PATCH 2/2] Use lib-symbol-visibility for the C++ library

Thanks!  The patch looks good, so I'll push it once your FSF copyright
papers have arrived (if I didn't send these off-list, please remind me).

> The second patch adds the necessary changes as explained in
> lib-symbol-visibility manual to add support for libgnutlsxx. I also
> removed the "local: *;" directive from libgnutlsxx.map, because it is
> no longer needed to hide symbols using it.

I'm not sure, maybe the 'local: *' is still needed: what if you use a
compiler that doesn't support symbol visibility but supports linker
version scripts?  Then 'local: *' would be useful.  Is there any harm in
keeping it?

/Simon