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

[build] dune #567

Closed
wants to merge 16 commits into from
Closed

[build] dune #567

wants to merge 16 commits into from

Conversation

gares
Copy link
Contributor

@gares gares commented Dec 29, 2023

blocked by bug
unblocks using ppx_optcomp and have a single branch for multiple coq versions

@rlepigre
Copy link
Contributor

I think that this can be unblocked with current dune by creating "dummy" theories for the various elpi folders. This requires changing the logical paths a bit, but that should probably not be a problem.

The only slight issue is that the top-level elpi folder cannot be put under a logical path with the elpi. prefix (like elpi.elpi) due to something I don't completely understand. But maybe that can be worked around.

Here is my experiment: https://github.com/rlepigre/coq-elpi/tree/dune (based on a rebased version of this branch). It seems to work as expected, but it is hard to tell given the amount of output that tests give. 😅

@rlepigre rlepigre mentioned this pull request Mar 20, 2024
5 tasks
gares and others added 5 commits June 3, 2024 11:18
The [_CoqProject] is still useful for editors, it was partially
generated using:
```
grep "^ (name" apps/**/dune | \
  grep -v src | \
  sed 's/apps\/\([a-zA-Z]*\)\/\([a-zA-Z]*\)\/dune: (name \([^)]*\))/-Q apps\/\1\/\2 elpi.apps.\1.\2\n-Q _build\/default\/apps\/\1\/\2 elpi.apps.\1.\2/' | sed 's/\.theories//'
```

Co-Authored-By: Rodolphe Lepigre <[email protected]>
@gares gares closed this Jun 28, 2024
@gares gares deleted the dune branch June 28, 2024 09:43
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