Skip to content
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

Use b.graph.host instead of b.host #1

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Conversation

AndrewKraevskii
Copy link
Contributor

b.host was deprecated in 0.13.0 and removed in ziglang/zig#20388

b.host was deprecated in 0.13.0 and removed in ziglang/zig#20388
@floooh
Copy link
Owner

floooh commented Dec 19, 2024

Just looking through the zig changes, man... I really don't like this change at all, I was actually hoping it goes into the opposite direction and methods which are currently on the root module would go up into the build step :/

image

@floooh
Copy link
Owner

floooh commented Dec 19, 2024

Ok, I'm using builtin.version to decide whether to use the old or new code. Assuming that everybody who's on the current dev version runs the latest HEAD version:

floooh/sokol-zig@a26bb7c

@floooh
Copy link
Owner

floooh commented Dec 19, 2024

Erm, previous comments are for the other PR :)

Thanks for the PR, merging now...

@floooh floooh merged commit 6d054aa into floooh:main Dec 19, 2024
3 checks passed
@floooh
Copy link
Owner

floooh commented Dec 19, 2024

Ok, all my 'leaf projects tested and working again (also the WASM builds). Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants