Skip to content

Update for Zig 0.17.0#220

Merged
robbielyman merged 4 commits into
natecraddock:mainfrom
AnErrupTion:zig-0.17
Jul 9, 2026
Merged

Update for Zig 0.17.0#220
robbielyman merged 4 commits into
natecraddock:mainfrom
AnErrupTion:zig-0.17

Conversation

@AnErrupTion

@AnErrupTion AnErrupTion commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Hi! This PR updates the code base to work on Zig 0.17.0. Specifically, this has been tested on Zig 0.17.0-dev.1282+c0f9b51d8.

All tests run except the void table ones, because void{} is no longer valid syntax and throws this at compile-time:

error: type 'void' does not support array initialization syntax

I'm not sure if this can be fixed or not, so in doubt, I have commented these tests out.
EDIT: Fixed!

Also, all examples run and work except the multi-threading one, which compiles but doesn't work most of the time and I'm not sure why. Help appreciated here!

Signed-off-by: AnErrupTion <anerruption@disroot.org>
Signed-off-by: AnErrupTion <anerruption@disroot.org>
@natecraddock

natecraddock commented Jul 9, 2026

Copy link
Copy Markdown
Owner

I haven't looked closely, but I think {} is the replacement for void{}

Thanks for working on this!

This will also need updates to the CI files to use zig 0.17

@AnErrupTion

Copy link
Copy Markdown
Contributor Author

I haven't looked closely, but I think {} is the replacement for void{}

Thanks for working on this!

This will also need updates to the CI files to use zig 0.17

Indeed, this seems to do the trick!

AnErrupTion and others added 2 commits July 9, 2026 19:59
@robbielyman robbielyman merged commit 409ab60 into natecraddock:main Jul 9, 2026
3 checks passed
@natecraddock

Copy link
Copy Markdown
Owner

Also, all examples run and work except the multi-threading one, which compiles but doesn't work most of the time and I'm not sure why. Help appreciated here!

Was this merged before fixing the multi threading example? If that is the case, that is fine, but if it's broken I'd like to make sure an issue is opened to track that

@AnErrupTion

Copy link
Copy Markdown
Contributor Author

Also, all examples run and work except the multi-threading one, which compiles but doesn't work most of the time and I'm not sure why. Help appreciated here!

Was this merged before fixing the multi threading example? If that is the case, that is fine, but if it's broken I'd like to make sure an issue is opened to track that

Yes, the multi-threading example is still broken.

@robbielyman

robbielyman commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

ope, my bad, i wasn't following closely enough. in that case i'd like to revert, actually. i'd happily accept a follow-up to this that fixes the multi-threading test!

@AnErrupTion

Copy link
Copy Markdown
Contributor Author

ope, my bad, i wasn't following closely enough. in that case i'd like to revert, actually. i'd happily accept a follow-up to this that fixes the multi-threading test!

I'll re-open it tomorrow, right now it's getting late for me.

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.

3 participants