gh-141226: correctly support Py_HASH_EXTERNAL for Unix platforms

[picnixz]

I will need to test this manually. However, do we have give docs for embedders that would provide custom implementations? It's really "figure how to link your embedded implementation but here's what you need to provide". Is it sufficient? PyHash_Func is defined in pyhash.c:

#ifPy_HASH_ALGORITHM==Py_HASH_EXTERNALexternPyHash_FuncDefPyHash_Func; #elsestaticPyHash_FuncDefPyHash_Func; #endif

I guess it's sufficient if we use the correct linker flags but I'm not entirely sure whether I should document this as well. It's a really niche thing (and I'm not aware of anyone doing this; everyone uses the default hash functions that is siphash13 and this hash function is used by most programming languages and the Linux kernel so I don't see why someone would need to change this except for performance reasons such as SIMD implementations).

Note that we cannot just remove this feature because it's part of PEP-456. I can write a PEP to officially deprecate it but I don't think it's worth SC's or my time.

cc @encukou@vstinner

• Issue: Deprecate Py_HASH_EXTERNAL as it cannot be used for --with-hash-algorithm #141226

---

📚 Documentation preview 📚: https://cpython-previews--141245.org.readthedocs.build/

[picnixz]

Let's deprecate this as it's a tricky feature to document and to use.

[emmatyping]

Let's deprecate this as it's a tricky feature to document and to use.

Agreed, and while it was defined in a PEP, I don't think it would take a PEP to deprecate it unless people feel one is necessary IMO.