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

BUG: Fix build for glibc on ARC and uclibc. #13137

Merged
merged 2 commits into from Mar 16, 2019
Merged

BUG: Fix build for glibc on ARC and uclibc. #13137

merged 2 commits into from Mar 16, 2019

Conversation

ffontaine
Copy link
Contributor

fenv.h is not available on uclibc, arc architecture on glibc does not support FPU exceptions

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
[Retrieved (and slightly updated) from:
https://git.buildroot.net/buildroot/tree/package/python-numpy/0001-no-fenv-on-uclibc.patch]
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
The FPU exceptions bits are missing in fenv.h in glibc for ARC
architecture.

Signed-off-by: Evgeniy Didin <Evgeniy.Didin@synopsys.com>
[Retrieved (and slightly updated) from:
https://git.buildroot.net/buildroot/tree/package/python-numpy/0003-no-FPU-exceptions-bits-on-ARC-glibc.patch]
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
@charris charris changed the title Fix build on uclibc and glibc arc BUG: Fix build on uclibc and the glibc arc platform. Mar 16, 2019
@charris charris changed the title BUG: Fix build on uclibc and the glibc arc platform. BUG: Fix build for glibc on ARC and uclibc. Mar 16, 2019
@charris
Copy link
Member

charris commented Mar 16, 2019

@ffontaine I assume you have tested this on the relevant platforms, is that correct?

@ffontaine
Copy link
Contributor Author

Nope, it was only build-tested, not runtime tested. However, I assume that the original patches that we have on buildroot on python-numpy in version 1.15.2 was runtime tested by their original authors. I tried to backport these patches to make them work with latest master. However, if you think the changes are incorrect, I can also disable python-numpy on uclibc and glibc arc on buildroot.

@charris
Copy link
Member

charris commented Mar 16, 2019

Build testing is probably adequate. I assume the problem with testing is that you are cross compiling.

I assume that the original patches that we have on buildroot on python-numpy in version 1.15.2 was runtime tested by their original authors

What project is that?

@ffontaine
Copy link
Contributor Author

Buildroot (https://buildroot.org) is an open source project that allows users to easily build their embedded Linux systems through cross-compilation.

@charris
Copy link
Member

charris commented Mar 16, 2019

OK, let's give it a shot. The only thing that concerns me is that ARC or uclibc might update in the future and we will have no way to know. Guess if that happens we will need a version check in anycase. LGTM seems stalled, so merging this instead of waiting. Thanks @ffontaine .

@charris charris merged commit 06da877 into numpy:master Mar 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants