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

scope(coq): make Coq_lib.DB.t field lazy #7132

Conversation

Alizter
Copy link
Collaborator

@Alizter Alizter commented Feb 20, 2023

This is to delay the evaluation as much as possible, so that non-coq projects do not end up scanning for theories. At the moment, this is not the case, but will be when we compose with installed libs.

Signed-off-by: Ali Caglayan alizter@gmail.com

@Alizter Alizter added the coq label Feb 20, 2023
src/dune_rules/scope.mli Outdated Show resolved Hide resolved
src/dune_rules/scope.ml Outdated Show resolved Hide resolved
This is to delay the evaluation as much as possible, so that non-coq
projects do not end up scanning for theories. At the moment, this is
not the case, but will be when we compose with installed libs.

Signed-off-by: Ali Caglayan <alizter@gmail.com>

<!-- ps-id: ae628d6f-8b9c-4b2f-97b4-106cd60b70ed -->
@Alizter Alizter force-pushed the ps/rr/scope_coq___make_coq_lib_db_t_field_lazy branch from e5f1d07 to 96c4885 Compare February 20, 2023 20:00
@ejgallego
Copy link
Collaborator

Replaced by #7133

@ejgallego ejgallego closed this Feb 20, 2023
@Alizter Alizter deleted the ps/rr/scope_coq___make_coq_lib_db_t_field_lazy branch February 20, 2023 22:45
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