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

BLD: functions that differ only in their return type cannot be overloaded (on FreeBSD) #3459

Closed
yurivict opened this issue Aug 2, 2021 · 3 comments · Fixed by #3605
Closed
Assignees
Labels
bug build related to the build process

Comments

@yurivict
Copy link

yurivict commented Aug 2, 2021

Bug report

Bug summary
Build fails on FreeBSD 13:

In file included from yt/utilities/lib/image_samplers.cpp:677:
yt/utilities/lib/platform_dep_math.hpp:15:13: error: functions that differ only in their return type cannot be overloaded
inline bool __isnormal(double x) {
       ~~~~ ^
/usr/include/math.h:189:5: note: previous declaration is here
int     __isnormal(double) __pure2;
~~~     ^
@welcome
Copy link

welcome bot commented Aug 2, 2021

Hi, and welcome to yt! Thanks for opening your first issue. We have an issue template that helps us to gather relevant information to help diagnosing and fixing the issue.

@neutrinoceros
Copy link
Member

Hi @yurivict, it seems that this could (maybe ?) be solved by patching the preprocessor directives logic in yt/utilities/lib/platform_dep_math.hpp, though I don't know much about FreeBSD (or c++). @cphyc, @matthewturk, maybe you guys have ideas where to start ?

@cphyc cphyc self-assigned this Oct 18, 2021
@cphyc
Copy link
Member

cphyc commented Oct 18, 2021

Hi @yurivict, I have just created a fix (#3605) that works for me on Free BSD 13 w/ Python 3.8 (in a virtual box though).

Could you test this on your side?

@neutrinoceros neutrinoceros changed the title 4.0.1: error: functions that differ only in their return type cannot be overloaded (on FreeBSD) BLD: functions that differ only in their return type cannot be overloaded (on FreeBSD) Oct 18, 2021
@neutrinoceros neutrinoceros added the build related to the build process label Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug build related to the build process
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants