[GH-ISSUE #465] zig build update - deprecates defineCMacro -> root_module.addCMacro #366

Closed
opened 2026-05-05 12:12:20 -06:00 by gitea-mirror · 1 comment
Owner

Originally created by @zigster64 on GitHub (Dec 24, 2024).
Original GitHub issue: https://github.com/jmcnamara/libxlsxwriter/issues/465

Not a bug in libxlswriter ...

With the zig build, recent compiler update has a breaking change to the way C Macros are passed from the build system to the target platform.

142471fcc4

effect is that usage of defineCMacro should be replaced now with root_module.addCMacro

Added a PR to cover this. PR is tested with a build of zig-libxlswriter, and then end to end tested with my Zig app that generates spreadsheets.

Originally created by @zigster64 on GitHub (Dec 24, 2024). Original GitHub issue: https://github.com/jmcnamara/libxlsxwriter/issues/465 Not a bug in libxlswriter ... With the zig build, recent compiler update has a breaking change to the way C Macros are passed from the build system to the target platform. https://github.com/ziglang/zig/commit/142471fcc46070326526e3976f0150fe734df0b6 effect is that usage of defineCMacro should be replaced now with root_module.addCMacro Added a PR to cover this. PR is tested with a build of zig-libxlswriter, and then end to end tested with my Zig app that generates spreadsheets.
Author
Owner

@jmcnamara commented on GitHub (Dec 24, 2024):

PR merged.

<!-- gh-comment-id:2561348282 --> @jmcnamara commented on GitHub (Dec 24, 2024): PR merged.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/libxlsxwriter#366
No description provided.