Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Jul 18, 2016

Conversation

oswaldo4
Copy link
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
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
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
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