Skip to content
Snippets Groups Projects
Ralf Jung's avatar
commented on merge request !40 "Added NixOS Docs" at Iris / lambda-rust

gmp is something that comes in via opam, we don't even use it directly -- it's somewhere in Rocq's dependency tree. If the usual opam package doesn...

Ralf Jung's avatar
commented on merge request !40 "Added NixOS Docs" at Iris / lambda-rust

I've fixed coq-lint.sh in the repo.

Ralf Jung's avatar
pushed to branch master at Iris / lambda-rust
  • ee5b5960 · make coq-lint nix-compatible
HumamAlhusaini's avatar
closed merge request !40 "Added NixOS Docs" at Iris / lambda-rust
HumamAlhusaini's avatar
commented on merge request !40 "Added NixOS Docs" at Iris / lambda-rust

Well, it also instructs the user that the gmp should be installed as a shell rather than globally, though I do see your point. I'll close the MR.

Ralf Jung's avatar
commented on merge request !40 "Added NixOS Docs" at Iris / lambda-rust

With the changed shell file, NixOS.md seems to just say "use opam", which we recommend anyway. So I don't quite see the point.

HumamAlhusaini's avatar
commented on merge request !40 "Added NixOS Docs" at Iris / lambda-rust

It works

HumamAlhusaini's avatar
commented on merge request !40 "Added NixOS Docs" at Iris / lambda-rust

I'll test it out, but isn't the bigger issue the shell.nix? What I'm thinking is to just remove the shell.nix, replace /bin/bash with /bin/sh, and ...

Ralf Jung's avatar
commented on merge request !40 "Added NixOS Docs" at Iris / lambda-rust

Apparently /bin/sh exists on NixOS. I think we can change it to that.

Ralf Jung's avatar
commented on merge request !40 "Added NixOS Docs" at Iris / lambda-rust

You can have a branch in your fork, you don't need my permission for that.

HumamAlhusaini's avatar
commented on merge request !40 "Added NixOS Docs" at Iris / lambda-rust

@jung I understand. Rather than it being merged into main, can it be it's own branch?

Ralf Jung's avatar
commented on merge request !40 "Added NixOS Docs" at Iris / lambda-rust

Thanks for the MR! However, I don't think we want to have some odd shell.nix file here that nobody in the team understands -- this repo is portable...

HumamAlhusaini's avatar
opened merge request !40 "Added NixOS Docs" at Iris / lambda-rust
HumamAlhusaini's avatar
closed merge request !40 "Added NixOS Docs" at Iris / lambda-rust
HumamAlhusaini's avatar
opened merge request !40 "Added NixOS Docs" at Iris / lambda-rust
HumamAlhusaini's avatar
commented on issue #11 "Is forking lambda-rust allowed?" at Iris / lambda-rust

@jung Thanks

Ralf Jung's avatar
commented on issue #11 "Is forking lambda-rust allowed?" at Iris / lambda-rust

Forking repos requires manual approval of the account. We're being flooded by spambots so sadly this has become necessary....

Ralf Jung's avatar
closed issue #11 "Is forking lambda-rust allowed?" at Iris / lambda-rust
HumamAlhusaini's avatar
opened issue #11 "Is forking lambda-rust allowed?" at Iris / lambda-rust
HumamAlhusaini's avatar
commented on issue #10 "coqlint.sh is not found during make -j(CPU cores) command" at Iris / lambda-rust

Probably just nix shenanigans