-
Notifications
You must be signed in to change notification settings - Fork 5
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
Changes for sokol-gfx bindings cleanup. #31
Conversation
Hmm, btw, with a current Zig version I can't run the Does that sound familiar somehow? I'll need to merge the PR ahead of the sokol merge though (which I'll do within the next few hours). E.g.:
|
@floooh ,
Last time I saw this problem was emsdk_install. fixing by adding bash to the command, instead of chmod for posix. What do you think about running the shdc (together bindgen) via CI/CD to update the shaders when there is a change? |
Not a fan :) We should find out whether not setting the exec flags is intended behaviour by the Zig package manager (I seem to recall some Zig issue or discussion about that - because this will also be a problem when Zig moves the Clang toolchain into a package). Maybe we can somehow set the exec flags in build.zig as part of the shaders build job for now? |
Could do. |
DO NOT MERGE!
This PR contains the necessary updates for the upcoming sokol-gfx resource bindings update (which is a breaking change, thus this PR).
I'll take care of the merge when everything on the sokol side is done :)
For details see: floooh/sokol#1111
Reminder: after the sokol- and sokol-tools merge, and sokol-tools-bin having the latest binaries, also update build.zig.zon: