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

No stdlib #681

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Conversation

patrick-nicodemus
Copy link

This makes some changes throughout the build script to replace the automatic importing of the Prelude everywhere with an explicit, qualified importation, as part of an attempt to track down why Require Elpi has the effect of Require Prelude.

After changing the dune files it seems clear that we cannot remove the dependence on the Prelude without removing the dependence on the String module, so I will hold off on doing anything further until someone can suggest a workaround to using String in the build script.

@gares
Copy link
Contributor

gares commented Aug 30, 2024

The string thing is a hack to work around a limitation in dune. I'll keep the pr open waiting for that to be fixed, there is a pr that will eventually be merged.

@proux01
Copy link
Contributor

proux01 commented Sep 11, 2024

Note that this may be related to coq/coq#19530
I also made an attempt in that direction https://github.com/proux01/coq-elpi/tree/without_stdlib

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