Re: finding isnan

[Thomas Klausner <wiz%gatalith.at@localhost>]

On Mon, Nov 06, 2023 at 12:26:35AM +0100, Joerg Sonnenberger wrote:
> On Monday, November 6, 2023 12:21:49 AM CET Thomas Klausner wrote:
> > Hi!
> > 
> > math/py-scikit-learn currently doesn't build because it doesn't find
> > 'isnan'.
> > 
> > As I understand it, that symbol should be available for C++ files from
> > C++11 onwards, so I tried adding 'c++11' to USE_LANGUAGES, but that
> > didn't help, even though -std=c++11 ended up on the compiler line.
> > 
> > http://www.ki.nu/pkgsrc/reports/current/NetBSD-9.0/20231104.1950/py310-scikit-learn-1.3.2/build.log
> 
> If it is including cmath, it should be using std::isnan. If it is mixing math.h and cmath,
> it should stop doing that, ideally by including cmath first.

If I understand the build process correctly, it's using math.h only,
but the code is in files generated by cython, for code like this:

sklearn/utils/_isfinite.pyx:from libc.math cimport isnan, isinf
sklearn/utils/_isfinite.pyx:        if isnan(v):

So I'm not sure how to fix that.
 Thomas