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

ghcide: Add flags to toggle building each executable #2212

Merged
merged 2 commits into from
Sep 19, 2021

Conversation

hololeap
Copy link
Contributor

@hololeap hololeap commented Sep 19, 2021

This package is a transient dependency of haskell-language-server.
However, only the library is needed. These three flags give the option for
those who do not need the executables to disable them.

executable:

  • Toggles the ghcide executable
  • enabled by default.

test-exe:

  • Toggles the ghcide-test-preprocessor executable
  • disabled by default

bench-exe:

  • Toggles the ghcide-bench executable
  • disabled by default

@hololeap hololeap changed the title Add flags to toggle building each executable ghcide: Add flags to toggle building each executable Sep 19, 2021
@pepeiborra
Copy link
Collaborator

pepeiborra commented Sep 19, 2021

I'm not supportive of this change as it is:

  • The test-exe flag default value breaks cabal test ghcide
  • The bench-exe flag default value breaks cabal bench ghcide

I understand that you are coming at this from "why do I have to build all these executables if all I want is HLS", however I would only be supportive of a change that solves that problem if it has no negative impact on the ghcide development workflow.

This package is a transient dependency of `haskell-language-server`.
However, only the library is needed. These three flags give the option for
those who do not need the executables to disable them. All three are
enabled by default.

`executable`:
* Toggles the `ghcide` executable

`test-exe`:
* Toggles the `ghcide-test-preprocessor` executable

`bench-exe`:
* Toggles the `ghcide-bench` executable
@hololeap
Copy link
Contributor Author

I changed it so that all three flags have default: True in hopes this will not break the workflow. This seems like a good compromise to me since building the exe's can still be disabled from the command line.

Copy link
Collaborator

@pepeiborra pepeiborra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@michaelpj
Copy link
Collaborator

I'm slightly missing the motivation. HLS doesn't depend on these, so if you're just building HLS then cabal won't build them. Is it so that you can run cabal build all and not have to build them?

@jneira
Copy link
Member

jneira commented Sep 19, 2021

I'm slightly missing the motivation. HLS doesn't depend on these, so if you're just building HLS then cabal won't build them. Is it so that you can run cabal build all and not have to build them?

hmm cabal builds executable components by default i think

@jneira jneira added the merge me Label to trigger pull request merge label Sep 19, 2021
@mergify mergify bot merged commit 6f9c448 into haskell:master Sep 19, 2021
@hololeap
Copy link
Contributor Author

@michaelpj @jneira Yes, cabal builds all executable components unless you create a mechanism like this to turn them off. I run into this when packaging for Gentoo, which strives to give the end user as much control over their packages as possible (and currently uses cabal under the hood). Without this patch I would have to either:

  1. Give the user no control over whether the executables are installed, even if they don't want them
  2. Maintain a patch for the ghcide.cabal file for every new release.

(Just so you know what my personal motivations are.)

@michaelpj
Copy link
Collaborator

Yes, cabal builds all executable components unless you create a mechanism like this to turn them off.

Well, perhaps the question becomes: how are you invoking cabal?

If you build the executable component for haskell-language-server it doesn't build the ghcide executables.

> cabal build exe:haskell-language-server
...
> vim ghcide/exe/Main.hs # make trivial edit
> cabal build exe:haskell-language-server
Up to date

@hololeap
Copy link
Contributor Author

@michaelpj The way we have been doing it is to convert each .cabal package into a Gentoo package, where the dependency graph is handled solely by our package manager. We then compile Setup.hs and invoke setup for the configure, build, test, etc. phases on each package individually, and install them to a global location. So each package is built in relative isolation and there is no current integration of the v2 cabal features.

cdsmith pushed a commit to cdsmith/haskell-language-server that referenced this pull request Sep 21, 2021
This package is a transient dependency of `haskell-language-server`.
However, only the library is needed. These three flags give the option for
those who do not need the executables to disable them. All three are
enabled by default.

`executable`:
* Toggles the `ghcide` executable

`test-exe`:
* Toggles the `ghcide-test-preprocessor` executable

`bench-exe`:
* Toggles the `ghcide-bench` executable

Co-authored-by: Javier Neira <atreyu.bbb@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge me Label to trigger pull request merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants