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

uCLibc may not have complete fenv.h support even if header present #1229

Open
aperezdc opened this issue Oct 22, 2024 · 3 comments
Open

uCLibc may not have complete fenv.h support even if header present #1229

aperezdc opened this issue Oct 22, 2024 · 3 comments

Comments

@aperezdc
Copy link

This is related to bug #279971 in WebKit.

I am not 100% sure how things work when building the uClibc C library, but my understanding is that the only a few architectures provide all the functions from fenv.h, and even on architectures where support is partial (or missing) the configuration program will allow setting UCLIBC_HAS_FENV=y regardless.

So it seems that when building against uClibc more smarts than only checking __has_include(<fenv.h>) may be needed.

@mr-c
Copy link
Collaborator

mr-c commented Oct 22, 2024

Hello @aperezdc and thank you for the report. Is webkit using uClibc or https://uclibc-ng.org/ ? Which version(s)?

@mr-c
Copy link
Collaborator

mr-c commented Oct 22, 2024

A PR to add a better check would be very welcome, of course

@ThomasDevoogdt
Copy link

Hello @aperezdc and thank you for the report. Is webkit using uClibc or https://uclibc-ng.org/ ? Which version(s)?

The uClibc was forked in 2014, see https://uclibc-ng.org/. So I don't think that this one is very relevant. The compile issue was seen by using buildroot which ships with uClibc-ng v1.0.50. See https://github.com/buildroot/buildroot/blob/2024.08.1/package/uclibc/uclibc.mk.

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

No branches or pull requests

3 participants