-
-
Notifications
You must be signed in to change notification settings - Fork 710
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
Produce Nix expression for easy-install in NixOS #245
Comments
Hi @tomberek ! Thanks for filing this. I don't know anyone in the Sandstorm community yet who knows Nix... but maybe you do? I'd love to help you with this, if you're interested. |
If you don't have the time or interest, I'm also totally OK with leaving this open, as a signal to others that we welcome their contribution! |
I've got a few other projects. I'll try to get to this some time. I would On Sat, Feb 7, 2015 at 2:42 PM, Asheesh Laroia notifications@github.com
|
Cool. I am happy to help or review when the time comes!
|
it appears that NixOS kernels are built with CONFIG_USER_NS |
I had a quick try but the Makefile doesn't follow the common patterns. It tries to download the dependencies with git and doesn't provide a fallback where libraries are provided by the system. Because Nix uses a sandbox it doesn't allow network access during build time. It also makes the build non-reproducible because each git remote can change at any time and the dependencies are not pinned. I tried to package ekam which is sandstorm's custom make replacement which also has similar issues. My WIP is available at https://github.com/zimbatm/nixpkgs/tree/sandstorm |
I've found this to be a common problem with integrating some build systems On Tue, Apr 19, 2016 at 4:39 PM, zimbatm notifications@github.com wrote:
|
I don't believe this is accurate. The dependencies fetched by git are pinned by |
Ekam is now in nixpkgs (NixOS/nixpkgs#141064). I have a work-in-progress nix expression to build sandstorm (but it does not yet compile) at https://github.com/garrison/sandstorm-nix. |
No description provided.
The text was updated successfully, but these errors were encountered: