Skip to content

BLD: fix compilation on non glibc-Linuxes #7087

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

Merged
merged 1 commit into from
Jan 21, 2016
Merged

Conversation

xdmiodz
Copy link
Contributor

@xdmiodz xdmiodz commented Jan 21, 2016

Non-glibc Linuxes dont have the __GLIBC_PREREQ function and compilation of numpy
fails on such platforms. To avoid this the TRIG_OK check should be done only in
the glibc environment

The patch is taken from AlpineLinux repository
http://git.alpinelinux.org/cgit/aports/tree/testing/py-numpy/numpy-1.10.0-musl.patch?id=2e5c4bfcf1c9746edd58a8e684d01403f234e71d

Non-glibc Linuxes dont have the __GLIBC_PREREQ function and compilation of numpy
fails on such platforms. To avoid this the TRIG_OK check should be done only in
the glibc environment

The patch is taken from AlpineLinux repository
http://git.alpinelinux.org/cgit/aports/tree/testing/py-numpy/numpy-1.10.0-musl.patch?id=2e5c4bfcf1c9746edd58a8e684d01403f234e71d
@charris charris added this to the 1.11.0 release milestone Jan 21, 2016
charris added a commit that referenced this pull request Jan 21, 2016
BLD: fix compilation on non glibc-Linuxes
@charris charris merged commit f970038 into numpy:master Jan 21, 2016
@charris
Copy link
Member

charris commented Jan 21, 2016

Thanks @xdmiodz .

@charris
Copy link
Member

charris commented Jan 21, 2016

Wait. Is this correct? AFAICT, __GLIBC__ is defined in features.h Which is not imported before the test. So this check works for you because it is always false but is not correct in general.

@juliantaylor
Copy link
Contributor

why not just check for ifdef __GLIBC_PREREQ?

@xdmiodz
Copy link
Contributor Author

xdmiodz commented Jan 21, 2016

@juliantaylor you're right, I'll rework the patch

@xdmiodz xdmiodz deleted the musl-build branch January 21, 2016 21:18
@charris charris removed this from the 1.11.0 release milestone Jan 23, 2016
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.

3 participants