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
vampire: portability fixes #43071
vampire: portability fixes #43071
Conversation
|
||
|
||
-#elif defined(__APPLE__) | ||
+#elif defined(defined(__APPLE__)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this defined(defined(...))
incantation intentional?
Eeep! Nope, dishonor on me and my family. This is wrong, silly sed. Thanks for catching, will update shortly. Kinda surprised that's not an error.... |
Please try to submit the other fixes upstream as well. Otherwise looks good to me. |
d5c4bdc
to
0395d3b
Compare
Great, will do! Thanks! Turns out I actually had an old version of the patch here, whoops. Updated to what was submitted upstream and with the |
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: vampire Partial log (click to expand)
|
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: vampire Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: vampire Partial log (click to expand)
|
These have been submitted upstream, but seconds before writing this message so naturally no upstream response "yet" :). Regardless of response, fetch patches from the corresponding PR's for context and of course hopefully they'll be included in next version.
0395d3b
to
7353d83
Compare
All changes submitted upstream, patches are now fetched from corresponding PR's. Apologies for the rebase'ing but hopefully this is much nicer :). Thanks for pushing me to submit upstream 👍. Also enabled parallel building 😁. |
Oh, and more complete fpu/fenv fix :). |
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: vampire Partial log (click to expand)
|
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: vampire Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: vampire Partial log (click to expand)
|
Fix problems encountered using libc++, clang, and/or musl.
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)