fuzzywuzzy icon indicating copy to clipboard operation
fuzzywuzzy copied to clipboard

process.hpp is missing to include #include <inttypes.h>

Open RoyBellingan opened this issue 2 years ago • 3 comments

Else intmax_t will not be found in gcc13

Maybe in the past was some kind of transitive include ?

RoyBellingan avatar Jul 31 '23 13:07 RoyBellingan

Not a header I remember using. Nor do I remember the gcc version I targeted. Is this a dependency required by gcc13 only?

tmplt avatar Aug 01 '23 09:08 tmplt

Sorry, I wrote a poor bug report, The intmax_t used here https://github.com/tmplt/fuzzywuzzy/blob/b62013235a15c484203b6ff6c369c813a284cf6e/include/process.hpp#L27

is defined in the header inttypes.h which is not included.

What I think is that in the past it was included in a transitive way (IE was included by another include), but recent cleanup in recent GCC dropped it, so is now required.

Soon I will check if recent clang too requires it.

RoyBellingan avatar Aug 01 '23 11:08 RoyBellingan

I had the same problem and I could fix it in 5.15.130-1-MANJARO adding #include "inttypes.h" to that file. Thanks

gcc version 13.2.1 20230801 (GCC) GNU Make 4.4.1 cmake version 3.27.1

d3fstyle avatar Sep 15 '23 08:09 d3fstyle