Skip to content

Conversation

@Spadi0
Copy link

@Spadi0 Spadi0 commented Jan 18, 2025

Fixes builds using Zig master after ziglang/zig#22505 was merged by using the updated field names in std.builtin.Type.

Fixes builds using Zig master after ziglang/zig#22505 was merged by
using the updated field names in `std.builtin.Type`.
floooh added a commit to floooh/chipz-kitty that referenced this pull request Jan 18, 2025
@mlarouche
Copy link
Collaborator

@Spadi0 I can't merge this PR until we migrate to the mach nominated Zig that have those changes. In the meanwhile, I suggest you keep a fork on your own if you want to track Zig master

@Spadi0
Copy link
Author

Spadi0 commented Jan 19, 2025

@Spadi0 I can't merge this PR until we migrate to the mach nominated Zig that have those changes. In the meanwhile, I suggest you keep a fork on your own if you want to track Zig master

Sounds good. I confess, I didn't even know about Mach's Nominated Zig system or that this project followed it - it does seem like a good idea, though (keeping up with Zig master can be harrowing). I might even switch my own projects to using it, to be honest.

In keeping with what the Mach project seems to do, I can leave my branch and this PR open (I'll change it to a draft PR) until the next Nominated Zig version picks a cut-off commit. In the meantime, I'll try to remember to check in on things every now and then, and to rebase/update the branch as necessary to keep it building on Zig master.

Hopefully that gives you a place to direct those like myself who use Zig master and means upgrading to the next Nominated Zig should be fairly easy when the time comes.

@Spadi0 Spadi0 marked this pull request as draft January 19, 2025 20:51
@mlarouche mlarouche closed this May 27, 2025
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