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

Continuing the work you've done so far #1

Merged
merged 5 commits into from
Dec 20, 2023

Conversation

AdjectiveAllison
Copy link

Hey Dave, I also am interested in getting capy working on zig master.

First thing I did was add iterate = true to the build.zig openDir change due to later in the build file needing iterator access and the 0.11.0 code beforehand had that.

Then I just started manually digging into the errors of the atomic.Atomic stuff as well as leftover meta.trait stuff laying around from your change.

There are 5 more meta.trait references left in the codebase as of now but they are using methods that you didn't build into your trait.zig yet so I left them be temporarily. isZigString, hasUniqueRepresentation, and isPtrTo are the ones remaining.

Looking forward to getting this working as well :)

I'm on zig version 0.12.0-dev.1830+779b8e259 which is just a bit ahead of where you originally targeted and it's current master.

@dvmason
Copy link
Owner

dvmason commented Dec 20, 2023

Thanks, AdjectiveAllison! I've updated my master

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