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

Clarify structure #2

Open
wants to merge 73 commits into
base: master
Choose a base branch
from
Open

Clarify structure #2

wants to merge 73 commits into from

Conversation

proux01
Copy link
Contributor

@proux01 proux01 commented Sep 29, 2024

This implements, together with coq/coq#19530 , the part of CEP#83 that reached a common agreement during the dedicated Coq call discussions (tl;dr: give stdlib its own repository, clarify its internal dependencies and ensure they are kept with a CI check, "coq" metapackage keeps depending on "coq-stdlib" and no distribution of packages for induvidual subcomponents)

  • split code in stdlib into subcomponents, according to the CEP
    added graph in doc/stdib/depends.png, included in doc/stdlib/index.html
  • add CI checks
    • job checking for no duplicate filenames
    • job stdlib-subcomponents checking subcomponents dependencies (easy thanks to Nix)
    • job checking that all files are in exactly one subcomponent

@proux01 proux01 force-pushed the master branch 2 times, most recently from 9304194 to 86539ec Compare October 2, 2024 11:25
@proux01 proux01 force-pushed the clarify-structure branch 2 times, most recently from 0342674 to 4cad32a Compare October 5, 2024 13:50
@proux01 proux01 force-pushed the master branch 7 times, most recently from 2b99949 to 721e8e8 Compare October 7, 2024 12:40
@proux01 proux01 force-pushed the master branch 3 times, most recently from d729338 to 0b34c66 Compare October 8, 2024 13:34
@herbelin herbelin requested a review from ppedrot October 8, 2024 13:49
@proux01 proux01 force-pushed the master branch 2 times, most recently from c674105 to a3439aa Compare October 10, 2024 15:16
Ensure that subcomponents build with the declared dependencies
Checking that Make.all and other Make.* remain in sync
Those would yield ambiguity when doing "From Stdlib Require File.".
There are already a few in the prelude, let's not add more.
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.

1 participant