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 1.7.0 expects files where they aren't (exhibited by lwt.3.2.1) #1841

Closed
pirbo opened this issue Feb 13, 2019 · 3 comments
Closed

dune 1.7.0 expects files where they aren't (exhibited by lwt.3.2.1) #1841

pirbo opened this issue Feb 13, 2019 · 3 comments

Comments

@pirbo
Copy link

pirbo commented Feb 13, 2019

Note that I'm not using cutting edge lwt.4.1.0 but an old lwt.3.2.1 (because my real use case is in the context of cross compilation for windows where we don't manage to get lwt.4.1.0 but the issue is unrelated to cross compilation. If I downgrade lwt to 3.2.1 on any of my machine, the problem arises.)

After that (travis-ci) updated to dune 1.7.0, I started to see some

dune build --only-packages kappa-library,kappa-binaries
File "agents/dune", line 9, characters 8-14:
9 |   (name KaStor)
            ^^^^^^
Error: File unavailable: /home/pboutill/.opam/for_nightly/lib/lwt/log/lwt_log.a
File "agents/dune", line 9, characters 8-14:
9 |   (name KaStor)
            ^^^^^^
Error: File unavailable: /home/pboutill/.opam/for_nightly/lib/lwt/log/lwt_log.cmxa

[...]

and indeed there is no /home/pboutill/.opam/for_nightly/lib/lwt/log/ and /home/pboutill/.opam/for_nightly/lib/lwt/unix/. All the files "dune" looks for are directly in /home/pboutill/.opam/for_nightly/lib/lwt/.

Is this a problem on my side or a general problem of dune?

@rgrinberg
Copy link
Member

rgrinberg commented Feb 13, 2019

As far as I understand, the issue is that Lwt 3.2.1 has some hacks where it post processes the META file. Since dune >= 1.7.0 uses its own format for external packages (dune-package) all of those hacks are ignored. So essentially, Lwt 3.2.1 will not work with dune 1.7 and above.

@hcarty
Copy link
Member

hcarty commented Feb 13, 2019

Lwt 3.2.1 and above will not work with dune 1.7 and above

@rgrinberg Do you mean "Lwt 3.2.1 and below will not work with dune 1.7 and above" here? A quick local test seems to show Lwt 4.1.0 and several dependent packages building with dune 1.7.0.

@pirbo
Copy link
Author

pirbo commented Feb 13, 2019

I see and indeed newer lwt work fine.
If you think it's worth having a mention of the incompatibility here, reopen the issue. Else let's document the incompatibility somewhere (probably the opam file) and close here...

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

3 participants