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.

Sign up for GitHub

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

Jump to bottom

error: undefined reference to 'frexp', 'modf' #93

Closed
jfichtner opened this issue Aug 14, 2017 · 5 comments
Closed

error: undefined reference to 'frexp', 'modf' #93

jfichtner opened this issue Aug 14, 2017 · 5 comments

Comments

@jfichtner
Copy link
Contributor

When I try to build the project (from current master branch) I get errors in multiple test projects related to the math.h functions frexp and modf. Adding "-lm" (linking libm.so) to the CFLAGS variable in each of the respective makefiles fixes the issue. I'm not familiar enough with GNU make to know if this fix is appropriate for a pull request or not, or if perhaps this could have other side effects on other build environments / targets. Please advise.
@j123b567
Copy link
Owner

What compiler are you using? (and what build environment).

@jfichtner
Copy link
Contributor Author

I'm using arm-linux-androideabi-gcc (GCC) 4.8 on Ubuntu 14.04, targeting android.

@j123b567
Copy link
Owner

It seems to be an "issue" of Bionic C library. Anyway, these functions are from libm and it should be linked by '-lm'. I will try it with current build on linux.

j123b567 added a commit that referenced this issue Aug 16, 2017
@j123b567
Add dependency on libm as required by #93
4f960d2
@j123b567
Copy link
Owner

Can you try latest version if it fixes all problems for you?

@jfichtner
Copy link
Contributor Author

Yes, that fixes my build issues, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@j123b567 @jfichtner