-
Notifications
You must be signed in to change notification settings - Fork 14
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
how come the README example configures with --disable-documentation
?
#107
Comments
Hi Ben @bmillwood, Does this answer your question? |
In this run, it seems like cabal decided to build (at least many of) the docs for the dependencies at the haddock step anyway: https://github.com/bmillwood/bluesky-tools/actions/runs/12623008445/job/35171545170 But this is after the cache is saved, so the dependency docs don't get cached, and I imagine my haddock step would take 5 minutes every time, whereas if I built the dependency docs at the same time as building the dependencies, I think they would get cached. Am I better off enabling documentation, or did I miss something? |
Odd.
This it at least worth an experiment. Another experiment would be to pass If you do these experiments, please let me know, so I can update my advice accordingly! |
Thanks. I'll probably get around to doing some tests at some point but right now my thing works for me so it's unlikely to be high priority :) I forgot to mention it last time but I did already try passing It's a little hard to draw firm conclusions from this because of course even identically configured runs can differ in performance because of the changing state of the cache, but feel free to browse through the rest of my runs here if you'd find that useful or interesting: https://github.com/bmillwood/bluesky-tools/actions |
This issue could also be a regression in cabal-3.14
Actually, I think it is still this regression: The |
setup/README.md
Line 165 in f5975f0
It goes on to build documentation anyway. Is it something to do with the dependency cache?
(This is a documentation request, unless it's a mistake, in which case this is a bug report :) )
The text was updated successfully, but these errors were encountered: