Seems reasonable, thanks, but I think it should ideally be in LDFLAGS.

Seems reasonable, thanks, but I think it should ideally be in LDFLAGS.

do you mean so? env ['LDFLAGS'] + = f ' -lm'

Yes, but no need to make this one a format string, skip the f.

That said, looking at the recipe it looks like it uses env['LIBS'] for this a few lines above. Could you check adding -lm in that line works?

Yes, but no need to make this one a format string, skip the f.

That said, looking at the recipe it looks like it uses env['LIBS'] for this a few lines above. Could you check adding -lm in that line works?

Hmm and really, it should work, should I build a package using this recipe and then create a new pull request?

you could update this pull request instead of creating a new one, but aside that, yes.

@tshirtman @inclement I did a test build, everything works