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

Updates for zig 0.11.0 build system and package manager #27

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Trevor-Strong
Copy link

Just some very minor fixes/updates.

Changes

  • Rename source_file option to the new name root_source_file in build.zig
  • Add target and optimize options, as those are now per-module instead of per-compilation
  • Add build.zig.zon for compatibility with the builtin zig package manager.
  • Rename usages of Endian fields to new name (.Big -> .big, .Little -> .little)
  • Add a test step to build.zig
  • Restore the root.UNICODE option since the referenced issue is now closed.

@Trevor-Strong Trevor-Strong marked this pull request as ready for review January 27, 2024 16:13
@marler8997 marler8997 force-pushed the main branch 2 times, most recently from 592549a to eb06c90 Compare February 22, 2024 14:47
@marler8997
Copy link
Contributor

These changes should be done in the zigwin32gen repo, otherwise they'd get overwritten in the next release. I think most, possibly all of them have been addressed at this point, if not feel free to open a PR in that repo.

@marler8997 marler8997 force-pushed the main branch 2 times, most recently from 852ee48 to c778640 Compare April 17, 2024 01:22
@dgellow
Copy link

dgellow commented Apr 17, 2024

The file build.zig.zon seems to be missing

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.

3 participants