search

sam701 / zig-toml

Zig TOML (v1.0.0) parser
MIT License
35 stars 13 forks source link

Fix for latest Zig master #4

Closed abhinav closed 9 months ago

abhinav commented 9 months ago

The build.zig file needed to be updated as a result of some recent build system changes,

On top of that, it is now a compile-time error to declare a var and then not mutate it. These cases must use const instead.

sam701 commented 9 months ago

@abhinav thank you! zig 0.11 fails to build because of the changes in build.zig. I'll drop support for zig 0.11 in the following commit.

abhinav commented 9 months ago

@abhinav thank you! zig 0.11 fails to build because of the changes in build.zig. I'll drop support for zig 0.11 in the following commit.

👍 I assumed Zig 0.11 wasn't supported already because CI wasn't testing against it. It's possible to support both if you'd prefer that with a little comptime code. Example: https://github.com/abhinav/temp.zig/blob/main/build.zig

abhinav commented 9 months ago

@sam701 https://github.com/sam701/zig-toml/pull/5 adds back 0.11 support and tests for both