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

SunOS fix #9214

Closed
0-wiz-0 opened this issue Sep 3, 2018 · 5 comments
Closed

SunOS fix #9214

0-wiz-0 opened this issue Sep 3, 2018 · 5 comments
Labels
Build issues Issues with building from source, including different choices of architecture, compilers and OS

Comments

@0-wiz-0
Copy link

0-wiz-0 commented Sep 3, 2018

@jperkin added a patch to the pkgsrc package for scipy with the commit message:

Apply a couple of patches to fix SunOS.

I can't evaluate the patch or know details about the problem. Here's the patch:

--- scipy/special/_round.h.orig 2018-05-05 17:10:11.000000000 +0000
+++ scipy/special/_round.h
@@ -9,6 +9,10 @@
 #include "_c99compat.h"
 #include "cephes/dd_idefs.h"
 
+#if defined(__cplusplus)
+using std::isinf;
+using std::isnan;
+#endif
 
 double add_round_up(double a, double b)
 {
@pvanmulbregt pvanmulbregt added the Build issues Issues with building from source, including different choices of architecture, compilers and OS label Sep 3, 2018
@pvanmulbregt
Copy link
Contributor

Is there a URL for the discussion/patch?

@ilayn
Copy link
Member

ilayn commented Sep 3, 2018

I found this but don't know if that's the one.

NetBSD/pkgsrc@81abcb6

@0-wiz-0
Copy link
Author

0-wiz-0 commented Sep 3, 2018

That's the original commit, yes. The HAVE_OPEN_MEMSTREAM part looks like it's solved already.

@rgommers
Copy link
Member

I think that if we want to apply this patch, the correct place to tweak things is in scipy/_lib/_c99compat.h and not in one of the multiple files that uses sc_isnan. It would also be good to know why this is only needed on NetBSD.

@jperkin or @0-wiz-0 a PR would be very welcome. We can't apply this as is though, so I'll close this issue.

@rgommers
Copy link
Member

actually this is a duplicate with gh-9214, which has more discussion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build issues Issues with building from source, including different choices of architecture, compilers and OS
Projects
None yet
Development

No branches or pull requests

4 participants