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

Dune build error #92

Open
lukaszcz opened this issue Oct 25, 2020 · 2 comments
Open

Dune build error #92

lukaszcz opened this issue Oct 25, 2020 · 2 comments

Comments

@lukaszcz
Copy link
Owner

I run "dune build" and get the following error:

  coqdep theories/Plugin/Hammer.v.d

*** Warning: in file Hammer.v, library Hammer.Tactics.Tactics is required and has not been found in the loadpath!
File "./theories/Plugin/Hammer.v", line 1, characters 27-42:
Error: Unable to locate library Tactics.Tactics with prefix Hammer.

I notice that uncommenting the line

;(theories Hammer.Tactics)

in the file theories/Plugin/dune solves the problem. But I don't know what the purpose of commenting the line out was and whether it won't affect something else.

@palmskog
Copy link
Contributor

palmskog commented Oct 25, 2020

This is a current limitation of Dune support for Coq. We have to make a choice between:

  1. supporting the case where Dune is used to build and install coq-hammer-tactics and coq-hammer in turn via dune build -p <package> && dune install, or
  2. supporting builds of the whole thing with dune build && dune install.

I chose the first option, which is why (theories Hammer.Tactics) is commented out. We can't support both cases simultaneously until Dune 2.8 or so.

@lukaszcz
Copy link
Owner Author

lukaszcz commented Oct 25, 2020

OK, so let's leave it as it is for now

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

No branches or pull requests

2 participants