Skip to content
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

xattrs not available in nix builds #548

Closed
pmahoney opened this issue Feb 3, 2019 · 7 comments · Fixed by #641
Closed

xattrs not available in nix builds #548

pmahoney opened this issue Feb 3, 2019 · 7 comments · Fixed by #641

Comments

@pmahoney
Copy link

pmahoney commented Feb 3, 2019

This is more of a heads-up than an issue or feature request.

I wanted to try packaging "please" for nixpkgs, but it's going to be difficult since nix disables xattrs within builds It appears the reasons for disabling them would not apply to "please" (other than desire to support filesystems that lack xattrs), but they are disabled at any rate.

@peterebden
Copy link
Member

Hmmmmm OK. Not sure what a good alternative would be; we used to use a separate file but moved to xattrs to avoid risks of it getting out of sync with the real ones.

Thanks for the heads up!

@georgegiovanni
Copy link

What about a way to at least create output (plz-out) elsewhere (not in the same dir as src)? AWS EFS also doesn't support xattrs.

@peterebden
Copy link
Member

Have discussed that a bit; seems legit but just quite a bit of work to make everything aware of it.

@swills
Copy link

swills commented Apr 4, 2019

Also experiencing this issue on FreeBSD using tmpfs, a memory based filesystem which doesn't support xattrs.

@katzdm
Copy link
Contributor

katzdm commented Jun 4, 2019

This also impacts various MacOS virtualization systems (docker-on-mac, virtualbox).

@hzeller
Copy link

hzeller commented Aug 1, 2023

@pmahoney Did you consider packaging please for nix after this was resolved ?

@pmahoney
Copy link
Author

pmahoney commented Aug 1, 2023

@hzeller No, and I do not currently have plans to package please.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants