-
Notifications
You must be signed in to change notification settings - Fork 64
Experiment: dunify proofs #1204
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
base: main
Are you sure you want to change the base?
Conversation
This removes the dependency of lang on compiler
using C locale
Thanks for exploring this. I really don’t like the following: “we have to use |
Two technical solutions to this:
Taking a step back, I think not knowing where files are located reveal that the file hierarchy is not perfect at the moment. So the best solution might be to agree on some changes to it. For me, apart from To some extent, I think having a more rigid build system might help for that, because it will be too painful to have something that does not make sense. |
This is an experiment to port the Coq part of Jasmin to dune. Instead of just one dune file (which is another valid option), I tried to use one dune file per directory, to split the sources into components, with clear dependencies between one another. This PR is more for starting discussions about whether we want to go in that direction.
Pros of porting to dune:
Cons of porting to dune:
theories
stanza in the dune file, but actually only sometimes, I don't understand the logicscoq.extraction
stanza must explicitly list the OCaml files created by extraction, including all the files coming from the Coq stdlib, this is uglydune coq top
somehow), meaning that we duplicate information between dune files and _CoqProject filesPros of having one dune file per directory:
Cons of having one dune file per directory:
From other_directory Require Import ...
to do the imports.pp_error
type is not known inarch
so cannot be used there, while there were a few occurrences inarch_extra.v
. I tried to put just a string there, it works rather okay, but I don't know if this was the best option.Current limitations:
compiler
depends onarch
, while only the lowest part ofcompiler
depends onit
. So we could splitcompiler
into an arch-dependent and an arch-independent parts.