Skip to content

Not all Linux have fpu_control.h (that's a glibc specific header)#102

Merged
vitaut merged 1 commit intoampl:masterfrom
oswaldo4:patch-1
Jul 18, 2016
Merged

Not all Linux have fpu_control.h (that's a glibc specific header)#102
vitaut merged 1 commit intoampl:masterfrom
oswaldo4:patch-1

Conversation

@oswaldo4
Copy link
Copy Markdown
Contributor

A portable alternative could be fenv.h? I'm not sure if it's enough.

A portable alternative could be fenv.h? I'm not sure if it's enough.
@vitaut
Copy link
Copy Markdown
Contributor

vitaut commented Jul 13, 2016

Thanks for the PR. For the reference, what system and standard library is causing the problem?

@oswaldo4
Copy link
Copy Markdown
Contributor Author

Alpine Linux, which uses the musl libc by Rich Felker.
The musl libc presents many many security and performance advantages, so that was why I opted for it for all the local computers here on the beginning of July.
Unfortunately, there are a few software that use unportable glibc-specific functions.

The good thing is that most of those software are open-source, so we can see where it fails exactly and solve the issues. I'm glad AMPL's mp is such a software.

Unfortunately AMPL itself isn't and there are no binaries for musl libc. I understand though that an open-source model for a company is hard to make work and profit from. In the future, who knows, maybe if musl starts getting more famous AMPL could provide musl binaries... Or even open source it all, although I think that's less likely in the short term.

Anyway, thanks for the help.

@vitaut vitaut merged commit 47d2421 into ampl:master Jul 18, 2016
@vitaut
Copy link
Copy Markdown
Contributor

vitaut commented Jul 18, 2016

Merged, thanks!

glebbelov added a commit that referenced this pull request Aug 1, 2023
Normalize conditional comparisons for better unification, makes positive tolerance unnecessary in current tests
glebbelov added a commit that referenced this pull request Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants