search

justinwoo / spago2nix

Generate a derivation of (old) Spago dependencies, and use them to install them into the directory structure used by Spago.
MIT License
42 stars 22 forks source link

Fix install and build commands #36

Closed dhess closed 3 years ago

dhess commented 3 years ago

Please see the commit messages for details.

dhess commented 3 years ago

Please note that it appears the tests failed because the nix-build command failed on the GitHub Action runner, not because there's an issue with the patch:

633
copying path '/nix/store/f0f9dgy6r5mgqmjb7yzml0px729p93lv-nss-cacert-3.56' from 'https://cache.nixos.org'...
634
copying path '/nix/store/4ywspa1bnlmasnk6jkphyr3x0ff0qxif-git-2.28.0' from 'https://cache.nixos.org'...
635
building '/nix/store/hsyim37asq4qz1nai74makxnjsgl8wi2-purescript-aff-390857f.drv'...
636
building '/nix/store/2gzqwdw8naxx6n0kwhxdvndk706s7cri-purescript-arraybuffer-types-505aa3d.drv'...
637
building '/nix/store/n35k5yll7qkgj6vj3kg3lpqw4ylwmh85-purescript-arrays-1bca4c0.drv'...
638
building '/nix/store/ar8kys7il7q0hay29vcqg0bwid5wmwqd-purescript-bifunctors-1062425.drv'...
639
error: all build users are currently in use; consider creating additional users and adding them to the 'nixbld' group
640
Error: the 'spago2nix install' command failed to build the 'installSpagoStyle' Nix attribute

This is precisely the kind of error that was previously going undetected, for the reasons explained in the commit messages.

dhess commented 3 years ago

I pushed another commit to fix the test by dialing down the number of simultaneous nix-build jobs that were causing the issue on the runner.

justinwoo commented 3 years ago

thanks