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
darwin.darling: fix build on case-insensitive filesystems #110685
Conversation
4f6d4b1
to
b7a716d
Compare
@GrahamcOfBorg build darwin.Libsystem |
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.
This breaks stdenv https://gist.github.com/veprbl/f94b093aeaaaf879506432add0422f62
It's because the new
@veprbl which one do you prefer? |
I would try to stick to first updating darling if that is possible without breaking anything. |
Use upstream darlinghq/darling@ea5f07d to fix build on case-sensitive filesystems. Also update headers in Libsystem due to path modified. fix NixOS#91480
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.
I think, commit title should state something like "darwin.darling: unstable-2018-02-10 -> unstable-2020-04-26"
os/activity.h | ||
os/log.h | ||
os/trace.h | ||
") |
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.
/nix/store/pawa4j5ywf0s0mm1khk8f45qx8qrj49v-Libsystem-1238.60.2-osx-10.12.6/include/os/trace.h:32:10: fatal error: 'os/trace_base.h' file not found
#include <os/trace_base.h>
^~~~~~~~~~~~~~~~~
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.
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.
Here is a little complicated, os/trace_base.h
required os/availability.h
(not the one below, a new one), while the new os/availability.h
required new Availability.h
in xnu
according to #110685 (review) . We have to do it step by step to keep minimal change in each PR.
@veprbl would you review #109368 first? Then I will try making a pure xnu update, then darling.
I marked this as stale due to inactivity. → More info |
I marked this as stale due to inactivity. → More info |
Motivation for this change
Use upstream darlinghq/darling@ea5f07d to fix build on case-insensitive filesystems.
Also update headers in Libsystem due to path modified.
Tested darling but not Libsystem.
cc @veprbl @thefloweringash
fix #91480 #107431
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)