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.hfs: remove from Libsystem and mark broken #111468
base: staging
Are you sure you want to change the base?
Conversation
@GrahamcOfBorg eval |
hfs was introduced in e6f7f29 (NixOS#47678), but it seems nobody will needs its pure headers. We drop it from Libsystem.
@matthewbauer Do you remember why we needed hfs headers? |
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.
LGTM, provided that it doesn't break any builds. But perhaps we haven't needed it because of a lack of darwin-only packages?
So I think hfs required at some point for some packages. I would bet Apple is not using them much internally any more. Regardless, it's not too hard to add it as an additional dependency where it is. |
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:
localauth_k5login.c:41:10: fatal error: 'hfs/hfs_mount.h' file not found
#include <hfs/hfs_mount.h>
^~~~~~~~~~~~~~~~~
1 error generated.
make[2]: *** [Makefile:918: localauth_k5login.so] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/private/tmp/nix-build-libkrb5-1.18.drv-0/krb5-1.18/src/lib/krb5/os'
make[1]: *** [Makefile:1154: all-recurse] Error 1
make[1]: Leaving directory '/private/tmp/nix-build-libkrb5-1.18.drv-0/krb5-1.18/src/lib/krb5'
make: *** [Makefile:1004: all-recurse] Error 1
builder for '/nix/store/qqhkxy6c8n3a67cfqbivmx89p5pgh0hg-libkrb5-1.18.drv' failed with exit code 2
I marked this as stale due to inactivity. → More info |
I marked this as stale due to inactivity. → More info |
Motivation for this change
hfs was introduced in e6f7f29 (#47678), but it seems nobody will needs its pure headers. We drop it from Libsystem.
For those really need it, they should input hfs themselves.
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)