Skip to content

Conversation

@rlepigre
Copy link
Contributor

I have no idea what I am doing (not a nix user), so help would be appreciated.

@Alizter
Copy link
Collaborator

Alizter commented Nov 16, 2025

Looks like rocq isn't a valid package name in nixpkgs. Likely they haven't committed to switching to the new name just yet. I think coq would work here instead IIUC.

@rlepigre
Copy link
Contributor Author

Let's try!

@Alizter
Copy link
Collaborator

Alizter commented Nov 16, 2025

Looks like the nix part is working. However as you can see, the test suite will need to be updated which will probably entail a lot of work.

I don't mind bumping the coq/rocq version to 9.1 and stopping the testing of 8.16. There is little point of us testing an older version anyway if most potential users are using something else.

This PR will likely have to be drafted until the fixes are done.

@Alizter Alizter added the rocq label Nov 16, 2025
Signed-off-by: Rodolphe Lepigre <[email protected]>
@rlepigre
Copy link
Contributor Author

Yeah, it indeed looks non-trivial... On the other hand, it'd be nice if we could have have the test suite work for both versions for a while, to notice regressions when we move to using the rocq command instead of coq on newer versions for example.

I'll experiment a bit more to see what it would take. If that's too hard, I guess we can just bump to using Rocq 9.

@rlepigre rlepigre marked this pull request as draft November 17, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants