Add fips-recast to registry #68
Conversation
|
Awesome :) On OSX I'm still getting a compile error because GL/glu.h doesn't exist, as far as I can see only one GLU function is used in the tools: gluProject. One fips-friendly solution would be to replace this with glm (https://github.com/floooh/fips-glm), but I'm not sure whether the recast maintainer wants to bring in too many dependencies. |
|
One solution would be to select between gluProject and glm::project using a C define, and fips-projects could then simply set some RECAST_USE_GLM in the fips.yml or cmake files. |
|
PS: I've also written a ticket that fips needs some sort of import filter for cases like this where imports are only needed in specific situations (such as when building sample programs, but not when importing as lib): #69 |
|
I pinged Mikko about this and he prefers a direct copy-paste to replace the dependency on GLU (if its really just that function): https://twitter.com/mikkomononen/status/577406986029035521 |
|
Nice! I will do this change and then do a little more work on the demo late today, thank you. |
|
Done. Removed GLU dependency, hope now it will work on MacOS. Still not tested on Windows, will try tomorrow at work. |
|
I'm now having trouble with GL_TEXTURE_RECTANGLE_ARB not being defined on OSX, seems to be some include problem with glfw3.h. I'll try to find out more this evening. |
|
I have created a pull request fungos/fips-recast#1 to fix the OSX issues (and some minor CMakeLists.txt tweaks), looking at Windows next. |
Add recast to registry.
I've converted recast demo to glfw3 as I'd said it would be easier than get a fipsified SDL, but at the same time I've not tested on MacOS (I do not have one) or Windows (but will do soon).
Note: The glfw3 conversion is currently hosted at my fork, but the changes may get accepted upstream as the maintainer already confirmed it prefers glfw. I must only verify that premake4 builds correctly on MacOS and Windows before doing the PR there.