-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Update to latest zig and simplify build.zig #3905
Conversation
@freakmangd I'm not an expert on Zig building but it looks good to me. Thanks for the review! |
This breaks #3834 again |
@freakmangd please, could you review this issue? |
@sykire Hi, sorry about the breakage, I neglected to test building raylib as a local dependency, I'll get that working asap |
It builds successfully using
I think that's due to an unrelated thing. I'm new to zig and raylib so don't much about them. I'm using the configuration from this site: https://siddharthroy.com/posts/how-to-use-raylib-with-zig-without-any-bindings P.D. This is what is has showing before when doing
|
@sykire I'm using the same configuration to build and run raylib in my testing. Were you able to build and run it before this PR was merged? Maybe try installing the required libraries for your system if you haven't: https://github.com/raysan5/raylib/wiki/Working-on-GNU-Linux I'd think it'd break during the build process if you don't have X11, but I could be wrong Edit: Looks like you're getting a GLFW_PLATFORM_ERROR https://www.glfw.org/docs/latest/group__errors.html |
I couldn't build it, I got until the error in the ticket #3834 because of failed path resolution that was solved but then regressed. The other error I think I'm missing to build some library. I'm still in the process of running Raylib + Zig successfully once 😅. |
installHeader
's signature was changed recently, so a versioned function was added for itsrcdir
isn't needed, just adding "src/" before each path is sufficientprocess.Child.run
andprocess.Child.exec
was coerced into a single functionTested on ubuntu x64, on both 0.12.0-dev.3580+e204a6edb and 0.11.0