-
Notifications
You must be signed in to change notification settings - Fork 96
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
errors when compiling for iOS #944
Comments
Yeah, some of that is actually in process now. But we have a fair number of those warnings silenced already as they can get pretty pedantic. |
Probably need |
I had a look at this, there are simply too many changes required, and we have to change the internal halfedges to 64-bit which can affect performance and require further evaluation. I think for now the only way is to silence the warning. |
@pca006132 what about adding casts in the code? Also that particular warning above can be fixed by changing the |
This requires a lot of changes. For each one it is simple, but there are many of them... |
* cleanup files * hide some public APIs * merged CMakeLists.txt for headers * fix build * include cleanup * include cleanup again * address #944 * python cmake install * iostream is still needed * fix nix
I'm trying to compile manifold as an xcframework for iOS.
Doing this:
(I'm using this toolchain file https://github.com/leetal/ios-cmake)
I get various errors like:
Probably can silence those warnings (though I'm lousy with CMake and
-DCMAKE_COMPILE_WARNING_AS_ERROR=OFF
didn't do anything), but might be good to fix the underlying thing.The text was updated successfully, but these errors were encountered: