search

nine-lives-later / zzmq

Zig Binding for ZeroMQ
Mozilla Public License 2.0
25 stars 4 forks source link

Updated code to be compliant with Zig 0.12 #3

Closed 7Zifle closed 7 months ago

7Zifle commented 7 months ago

This pull request makes necessary changes to get the codebase compliant with Zig 0.12. The test.Dockerfile builds musl 1.2.4 because Alpine Linux is using musl 1.2.5 which is incompatible with Zig 0.12. I wasn't able to find a simpler way of getting musl 1.2.4 in the testing environment.

fkollmann commented 7 months ago

Thanks for the PR! I just realized the new Zig 0.12 got released. Please give me a few days to work through the changes (both this PR and Zig).

PS: Saw your previous PR. GitHub will automatically update a PR, if you push new changes. PPS: Actually musl-dev in Alpine 3.19 is v1.2.4 (https://pkgs.alpinelinux.org/package/v3.19/main/x86_64/musl-dev).

7Zifle commented 7 months ago

Thank you for following up on this. I've updated the test.Dockerfile so it no longer includes building musl. It appears the problem I was having with the docker unit tests was related to a dependency on clang. I also went ahead and updated the examples build.zig files. Just a heads up, The build.zon files are still using zzmq-v0.2.1-zig0.11

7Zifle commented 7 months ago

I apologize for the number of commits in this update. While reading over the examples, I noticed that several standard library functions referenced have been either relocated or modified slightly.

fkollmann commented 7 months ago

FYI, since the PR contains various aspects, I will merge it into a new pr3_stage branch and work out the details from there.