-
Notifications
You must be signed in to change notification settings - Fork 7
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
ImGui feature fails to build #19
Comments
Yes not unexpected, there was some confusion around whether sokol_imgui.h should be part of the Rust bindings or not (for instance it's currently not in the other bindings). The errors look like that the sokol_imgui.h and sokol_gfx_imgui.h C headers are outdated. I will probably need to set aside some time to clean up the Rust bindings, but as I said, Rust is the only binding which even includes those headers, so a quick fix would be to remove those two headers for the time being and at a later time add ImGui support to all language bindings. |
@floooh If you want a reference, I've included everything but the kitchen sink in Toxoid Engine (a polyglot WASM scripting game engine written in Rust). Spine is up next. https://github.com/toxoidengine/toxoid/tree/master/crates/toxoid_sokol 2024-01-06.20-40-50.mp4It's just using raw bindings for now though. High-level bindings are a bit of an ask to maintain, as I'm sure you know. |
It would be ideal to have separate libs for all utils as well as sokol_gp for convenient 2D rendering. From what I remember though it's a little difficult to get the build.rs / LLVM to play nice with duplicate C implementations. |
@troyedwardsjr What about the changes in PR #9? Do they help you? |
Oops, that PR totally fell through the cracks. If anybody can test if this works I'm happy to merge. |
Slight change of plans: #9 (comment) ...sorry for the inconvenience. |
@floooh
Without feature:
With ImGui feature:
With Emscripten:
With Emscripten + ImGUI feature:
The text was updated successfully, but these errors were encountered: