-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
update zig to 0.11.0 #4233
update zig to 0.11.0 #4233
Conversation
Thank you for starting this To fix the The I remember reading a zig PR I think from @mlugg sometime recently that sounded like it'd fix the missing reference traces for comptime function calls so maybe that error message is better in the HEAD build of zig |
Yeah, in theory there should be a reference trace now - lemme know if it's still bad |
This comment was marked as resolved.
This comment was marked as resolved.
There are a lot of also the error is very verbose, related to ziglang/zig#12344 A fraction of them
|
This sounds like a zig bug. None of those should need to be comptime |
Not sure, I'll file an issue in the Zig repo |
At least some of these appear to be due to this change: https://github.com/oven-sh/bun/pull/4233/files#diff-f2619d0e89ac22f8e6016642173178fea360dd0ea1bddf72f1b9ea078e1db9c9R218 The line in question in case the link above dies: pub const WriterType: type = @TypeOf(WriteStream(Source.StreamType, undefined));
pub const WriterType: type = WriteStream(Source.StreamType, undefined); |
@mlugg These errors in Details
UPDATE:
|
Will try and take a look when I'm briefly at home |
This comment was marked as resolved.
This comment was marked as resolved.
@Jarred-Sumner I think it's ready for review |
Can you search and update 0.11.0-dev.4006+bf827d0b5 to the new version you have compiling? After that it looks good to me. |
@dylan-conway replaced |
This comment was marked as resolved.
This comment was marked as resolved.
@CGQAQ it's |
Oh, thanks, changed |
These failing tests are expected, right? @Jarred-Sumner |
@Jarred-Sumner @dylan-conway Is there anything else I should change before this could land? |
@CGQAQ Some test failures in CI are expected right now. I took a look and nothing is out of the ordinary. Thanks for working on this! |
What does this PR do?
This PR includes:
How did you verify your code works?
zig fmt
on the changed filesThanks to @Hanaasagi , @ianprime0509 and @mlugg for helping me out