Skip to content
This repository was archived by the owner on Oct 7, 2020. It is now read-only.

Only generate hoogle database once #1151

Merged
merged 2 commits into from
Mar 28, 2019

Conversation

fendor
Copy link
Collaborator

@fendor fendor commented Mar 26, 2019

Closes #1147
Afaik, hoogle databases are located in ~/.hoogle.
Thus, if we invoke hoogle generate multiple times, we override the old database.
In the shake commands cabal-build-all, build-all we generate it multiple times.
As I understood it, we override all databases except the last one.
This change makes sure, shake installs hoogle and generates docs only once per command.

Also, shake now searches for the executable before installing it. If the executable is found, it is used to generate the hoogle database. Otherwise it is installed by either stack or cabal and then the database is generated. Executable is always installed into .local/bin or the respective path for Macos and Windows.

@fendor fendor mentioned this pull request Mar 26, 2019
@fendor fendor requested a review from alanz March 26, 2019 18:58
@fendor fendor requested a review from Anrock March 27, 2019 10:29
@fendor fendor merged commit b875e8d into haskell:master Mar 28, 2019
@fendor fendor deleted the shake-remove-redundant-hoogle branch April 13, 2019 16:47
@alanz alanz added this to the 2019-04 milestone May 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hoogle already exists during shake installation
3 participants