-
Notifications
You must be signed in to change notification settings - Fork 169
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
Refine and fix building with zig #369
Conversation
7d8fbd3
to
6c99bad
Compare
ha? So you want to add 0.12 zig support to the webui? |
I think subsequent adjustments to 0.12 are best placed in zig-webui |
Oh, my problem, please ignore what I said above, I just checked the commit record I should also make a new adaptation on |
Yeah, this is just about continuing to support building the webui C library with the zig build system as it's already in there. All zig WRAPPER api code should continue to go to the zig-webui repo. |
I'm gonna finish the PR this likely tomorrow or the day after tomorrow. Unfortunately, there is a lot of work atm |
9eef59d
to
1141dbe
Compare
1141dbe
to
3b3ccdf
Compare
857696a
to
9ca7c3c
Compare
The PR fixes and refines building with Zig.
The changes are also a preparation to allow a simple integration for building with Zig 0.12.0 in a follow-up.
The goal of this PR is to simplify the code while making it more expressive and extending functionality at the same time.
It also allows to zig-build with extended information. E.g. what is currently a silent build, can give info based on the build options, like:
It's not a huge diff, but still fairly big. So just reviewing and testing the new
build.zig
result locally might be an easier approach.