Skip to content

Test NixOS/nixpkgs#541797. - #492

Draft
Zimmi48 wants to merge 9 commits into
masterfrom
test-merge-derivations
Draft

Test NixOS/nixpkgs#541797.#492
Zimmi48 wants to merge 9 commits into
masterfrom
test-merge-derivations

Conversation

@Zimmi48

@Zimmi48 Zimmi48 commented Jul 23, 2026

Copy link
Copy Markdown
Member

@Zimmi48
Zimmi48 force-pushed the test-merge-derivations branch 3 times, most recently from e1e6862 to 980f6be Compare July 24, 2026 12:19
@Zimmi48

Zimmi48 commented Jul 24, 2026

Copy link
Copy Markdown
Member Author

I was getting confused with some of the errors that I get in CI, but actually, I just realized now that I should simply ignore all the pull_request_target jobs.

What I'm still having trouble understanding is why I need to keep the coqPackages references in the bundles for Rocq >= 9 (otherwise the CI files do not build, with a very cryptic error).

@Zimmi48

Zimmi48 commented Jul 24, 2026

Copy link
Copy Markdown
Member Author

Actually, it appears that there is an issue with correctly propagating the overridden versions from rocq-core to other packages. At least that's what causes the issue with math-classes in the 8.20 bundle. The derivation reveals that the bignums dependency is wrong: it is /nix/store/xwn8r9nlccns1g4scs26ngh1lhrlk83n-rocq9.1-bignums-9.0.0+rocq9.1.drv instead of a version for Coq 8.20. Note that this doesn't happen when building coqPackages_8_20.math-classes from the tested branch, so it is specifically an issue with the override mechanism.

@proux01

proux01 commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

I'll have a look

@proux01
proux01 force-pushed the test-merge-derivations branch 5 times, most recently from 3f746ba to 2aed22b Compare July 28, 2026 07:49
proux01 added a commit to proux01/mathcomp that referenced this pull request Jul 28, 2026
@proux01

proux01 commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

So, I rebased the nixpkgs branch and did some cleanup of the toolbox. CI here is green which is good. However, I tested it on mathcomp CI (to test other overrides than just Rocq itself): math-comp/math-comp#1634 There we can observe the following remaining issues:

  • some disappearing dependencies in the action scripts (likely a toolbox issues, can be fixed later)
  • some unexpected coq (instead of rocq-core) dependencies, for instance mathcomp-finmap shouldn't depend on coq, which I don't understand because the package is similar to mathcomp-bigenough which is fine (that may be an issue in nixpkgs)
  • various CI failures (for instance mathcomp-single) that I should investigate further

proux01 added a commit to proux01/mathcomp that referenced this pull request Jul 29, 2026
@proux01
proux01 force-pushed the test-merge-derivations branch 2 times, most recently from b5eaa74 to afb6a40 Compare July 29, 2026 12:34
proux01 added a commit to proux01/mathcomp that referenced this pull request Jul 29, 2026
@proux01
proux01 force-pushed the test-merge-derivations branch from afb6a40 to 80f1b09 Compare July 29, 2026 13:08
proux01 added a commit to proux01/mathcomp that referenced this pull request Jul 29, 2026
@proux01
proux01 force-pushed the test-merge-derivations branch 2 times, most recently from d7a3696 to b12fb2b Compare July 29, 2026 14:02
@proux01

proux01 commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Doing some more cleanup in the toolbox and a few fixes in nixpkgs, CI is finally happy both here and in math-comp/math-comp#1634
I had to give up support of Coq <= 8.20 in the toolbox but I guess it's fine (I doubt anyone is still using that).
So if you can retrieve my last few fixes from https://github.com/proux01/nixpkgs/tree/merge-bignums-derivations I think this can now be merged.

@Zimmi48

Zimmi48 commented Jul 30, 2026

Copy link
Copy Markdown
Member Author

Thanks! After this change, it might be finally time to rename the Coq Nix Toolbox to the Rocq Nix Toolbox 🎉

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.

2 participants