-
Notifications
You must be signed in to change notification settings - Fork 412
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 fails to build with dune cache enabled #4575
Comments
Could you check if |
@avsm did you find some time to test with |
unfortunately not -- do go ahead and close if there's not enough to repro or if its irrelevant in |
I'm closing as this is definitely fixed in main. Beacon files were removed along with the daemon mode. |
Indeed it's fixed in main (i.e. dune3), but it should definitely be fixed in 2 as well. It causes quite a few CI errors, mostly on macOS and Windows, which is pretty annoying. How do you feel about backporting them? (CC: @snowleopard @hhugo) |
@smorimoto Removing the cache daemon required major changes in Dune: see PRs #4443, #4465, #4484, #4498 and #4576 (plus I might have missed some smaller PRs). Backporting these changes will be a lot of work. I would be happy to review a PR that does that but at the moment I can't commit to doing this work myself. An alternative and more realistic approach would be to fix the daemon. This may turn out to be just a one-line change but will require some investigation. Again, I'm happy to help with this but I won't be able to lead, sorry. |
Hi folks people is hitting this bug "in the wild" , so I wonder if we should recommend disabling the cache in 2.9 + OSX ? Note the "Connect: operation not permitted", so it may be an easy fix the 2.9 series? |
Quick clarification: "Connect: operation not permitted" is a different error that you get with caching enabled and with the daemon manually started with |
I'm not so sure, since it involves the interaction between Dune and the cache daemon. One of the reasons we deleted the daemon in Dune 3.0 is to get rid of this logic. I have no access to a Mac machine for testing, so it would be very hard for me to investigate. I'm happy to review a PR if someone finds a way to fix this problem. |
I just tried to build an opam local switch, and got:
The text was updated successfully, but these errors were encountered: