Control: severity -1 normal Hi Sebastian,
On Sat, Sep 05, 2020 at 09:03:20PM +0200, Sebastian Andrzej Siewior wrote: > The symbol file records for instance the following symbols: > ZSTD_minCLevel > ZSTD_compressStream2 > > as present in 1.3.8. This isn't entirely correct. Both symbols were > present in 1.3.8 but they were hidden behind ZSTD_STATIC_LINKING_ONLY > and only available for static linking. > They are available for dynamic linking since 1.4.0, please see commit > d7d89513d6a21 ("Stabilize advance API") That was no doubt the intention, however in practice the symbol visibility wasn’t as expected: looking at the .so build in version 1.3.8, common/pool.c includes common/zstd_internal.h which defines ZSTD_STATIC_LINKING_ONLY before including zstd.h, and as a result the symbols are visible. (It’s unfortunate that the build hides the exact commands used, so they’re not visible in the build logs, but that’s another issue. Easy enough to fix in a local build to see exactly what’s going on...) So the cat is out of the bag, and the symbols are present and visible in the .so. The symbols file is generated and only reflects the reality of what is present in the file (apart from the version numbers which are added manually). Regards, Stephen
signature.asc
Description: PGP signature