-
Notifications
You must be signed in to change notification settings - Fork 99
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
Test in both C and C++ #58
Conversation
@DanielGibson Well, this is annoying. I'm explicitly using (see https://travis-ci.org/StrangeZak/Handmade-Math/jobs/217910297) |
looks like it can't find the functions at link time, try adding |
Also, not sure what |
|
@DanielGibson Well, clang is happy now but gcc is not. I'm not sure why, since the https://travis-ci.org/StrangeZak/Handmade-Math/jobs/218082438 |
Yep. The problem is basically described at http://stackoverflow.com/questions/45135/why-does-the-order-in-which-libraries-are-linked-sometimes-cause-errors-in-gcc and http://www.network-theory.co.uk/docs/gccintro/gccintro_18.html TL;DR: put the -lfoo after the source/object file that needs it, in your case make |
There it is! Thanks for the advice on that. I'm still rather new to compiling C projects, as you can see... @StrangeZak What do you think? Shall I squash and merge? |
no problem, I'm glad I could help :) |
Is this good to merged just noticed this sorry. @bvisness |
An attempt to fix #57 forever. I wrote a small, custom unit test setup that meets our needs without extraneous features.
This should build in both C and C++ modes with both clang and gcc/g++. But we shall see if Travis behaves...