search

kpu / kenlm

KenLM: Faster and Smaller Language Model Queries
http://kheafield.com/code/kenlm/
Other
2.46k stars 514 forks source link

Explicitly try to compile with header inclusion in Python setup #446

Closed jacobkahn closed 5 months ago

jacobkahn commented 5 months ago

There are some platforms/cases where the compiler sees a header with -ifoo but that #include <foo.h> still fails.

Without investigating these, since #include <foo.h> is a compile time requirement for Python bindings, change the compile test to include those headers as well.

Example of failure: https://github.com/flashlight/text/actions/runs/8039155921/job/21955863757 which succeeds from test branch when the test is added: https://github.com/flashlight/text/actions/runs/8039715499/job/21956995528 (doesn't enable HAVE_ZLIB falsely)