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

Nix toolbox update #727

Merged
merged 1 commit into from
Dec 13, 2024
Merged

Nix toolbox update #727

merged 1 commit into from
Dec 13, 2024

Conversation

proux01
Copy link
Contributor

@proux01 proux01 commented Dec 3, 2024

No description provided.

@proux01 proux01 force-pushed the ci-update branch 2 times, most recently from 4110b50 to b964e29 Compare December 3, 2024 12:53
@gares
Copy link
Contributor

gares commented Dec 3, 2024

amazing

@gares
Copy link
Contributor

gares commented Dec 3, 2024

it is still half broken #726 (review) but it would be nice if the manual edit I do on the job was propagated to the toolbox, since it has a killer feature: it prints the errors.

@proux01
Copy link
Contributor Author

proux01 commented Dec 3, 2024

There is something wrong happening with the Nix cache @CohenCyril would you have a clue?
We get CI errors like

Compiled library mathcomp.algebra.ssrnum (in file /nix/store/2zi311i5fx8z96r7a5f6akfb9s32zyd5-coqdev-mathcomp-algebra-dev/lib/coq/dev/user-contrib/mathcomp/algebra/ssrnum.vo)
makes inconsistent assumptions over library mathcomp.fingroup.fingroup

@proux01 proux01 force-pushed the ci-update branch 3 times, most recently from 820eb93 to 2f6e8be Compare December 4, 2024 09:54
@CohenCyril
Copy link
Collaborator

There is something wrong happening with the Nix cache @CohenCyril would you have a clue? We get CI errors like

Compiled library mathcomp.algebra.ssrnum (in file /nix/store/2zi311i5fx8z96r7a5f6akfb9s32zyd5-coqdev-mathcomp-algebra-dev/lib/coq/dev/user-contrib/mathcomp/algebra/ssrnum.vo)
makes inconsistent assumptions over library mathcomp.fingroup.fingroup

I've been banging my head on it for days now...

@proux01
Copy link
Contributor Author

proux01 commented Dec 13, 2024

Now testing elpi 2.0.6 with LPCIC/elpi#303

@proux01 proux01 force-pushed the ci-update branch 2 times, most recently from f00dad7 to bd31409 Compare December 13, 2024 16:04
@gares
Copy link
Contributor

gares commented Dec 13, 2024

seems to work!

@gares gares marked this pull request as ready for review December 13, 2024 17:36
@proux01 proux01 changed the title [do not merge] Test Nix toolbox update Nix toolbox update Dec 13, 2024
@proux01
Copy link
Contributor Author

proux01 commented Dec 13, 2024

Yes, I guess our best course of action is to merge this now and come back to coq-nix-toolbox master in a few days once coq-community/coq-nix-toolbox#302 will be merged.

@gares gares merged commit 31ebf2b into LPCIC:master Dec 13, 2024
142 of 144 checks passed
@proux01 proux01 deleted the ci-update branch December 13, 2024 19:12
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 this pull request may close these issues.

3 participants