Argh!
The header file puts it in an #ifdef ZDICT_STATIC_LINKING_ONLY block, saying it's experimental. But, configure looks for the exported function symbol.

cc @encukou @vstinner I'm confused by this error as the struct has existed with the members that we need since v0.7 (2016) of libzstd. Do you have any suggestions?

A

Is it worth setting the ZDICT_STATIC_LINKING_ONLY define? The other option is to remove the fallback symbol detection and have autoconf just check for >=1.4.5.

IMO, the >=1.4.5 check is safer.

For reference, v1.4.4 and earlier guarded the symbol behind a define. It was stabilised (facebook/zstd#2111) in v1.4.5.

free-threaded tsan failure is spurious: 1 test failed: test_asyncio.test_free_threading

test_datetime's [24, 24, 24] leaks are also spurious: this PR doesn't include #133498. I checked:

• buildbot/AMD64 CentOS9 NoGIL Refleaks PR
• buildbot/ARM64 MacOS M1 Refleaks NoGIL PR
• buildbot/PPC64LE RHEL8 Refleaks PR

The buildbots don't seem to have scheduled

The buildbots don't seem to have scheduled

Same thing happened at #133497 (comment). Trying again worked.

I'm still not seeing e.g. 'AMD64 RHEL8 LTO PR' in the pending checks list?

buildbot/PPC64LE RHEL8 Refleaks PR failures are the leaks:

test_datetime leaked [24, 24, 24] memory blocks, sum=72
FAIL: test_interrupt (test.test_multiprocessing_spawn.test_processes.WithProcessesTestProcess.test_interrupt)
Re-running test.test_multiprocessing_spawn.test_processes in verbose mode (matching: test_interrupt)
Re-running test_datetime in verbose mode
test_datetime leaked [24, 24, 24] memory blocks, sum=72

AMD64 RHEL8 FIPS No Builtin Hashes is unstable; failed in the usual way here.

Looking good so far, let's merge. Thanks both!