-
Notifications
You must be signed in to change notification settings - Fork 670
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
Path cannot be guessed when only coq-core
is installed
#16510
Comments
Does that work if you use 8.16 (and the findlib loading syntax)? |
8.16 doesn't have the package split, so I'm not sure how I can test that? |
But I tried using findlib loading syntax on master and I get the same error:
|
Oh indeed, sorry @LasseBlaauwbroek , I meant in master, you are correct. The syntax should be The syntax you used precisely disabled the new lookup method (for some compat cases) |
Great, yes, that works if I use both |
Okay the next problem is with Dune and
It appears that Dune does not actually pass the correct flags to |
To answer my own question: |
The coq-core split hasn't really been finished yet. There are still tools (such as coqdep) that rely on the stdlib having been installed. For coqdep |
Well, Are there any existing tracking issues for the final pieces of the coq-core split? |
@ejgallego Can I ask you for an opinion on this? Is this a bug in Coqdep or Dune? Or no bug at all? Should I open an issue in Dune? |
@LasseBlaauwbroek This is still a coq bug. There are various code paths that require the presence of coqlib (unintentionally). coqdep is reusing these paying no mind to |
I'm not convinced that this is true. My output of I see two potential issues with dune
|
@LasseBlaauwbroek What version of Dune are you using? |
I'm on 3.4.1 |
Would you be able to try an older version like 3.2? There was some broken refactoring with the flags (Dune is passing) recently that we still haven't managed to fix. I am thinking this PR is the likely culprit: ocaml/dune#5866 But it is also quite possible that this never worked before that. We never really wrote down what |
I tried Dune 3.2.0 and the effect is the same as with 3.4.1. |
As you have noticed @LasseBlaauwbroek , Dune does not really support libraries that run before the standard library, except for the The medium term plan is stop having the Stdlib as a implicit library, but that will require some breaking changes. I'd suggest opening an issue in the Dune repository with your concrete use case, so we can have the Regarding your questions:
Yes, this is because
|
Closing in favour of ocaml/dune#6163 |
If I only have
coq-core
installed, givenMyCoqFile.v
Running
errors out with
Adding
-boot
results inError: Can't find file ltac_plugin.cmxs on loadpath.
.So I have to manually add
-coqlib
, which is rather tricky because this is system dependent. It seems to me that Coq should be able to detect the location of-coqlib
even ifcoq-stdlib
is not installed. Or alternatively, should support be added tocoq_makefile
and Dune to automatically add the correct argument for-coqlib
when the-noinit
flag is given?The text was updated successfully, but these errors were encountered: