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

Track of possible improvements in the install script #2092

Closed
3 tasks
jneira opened this issue Aug 13, 2021 · 3 comments
Closed
3 tasks

Track of possible improvements in the install script #2092

jneira opened this issue Aug 13, 2021 · 3 comments
Labels
old_type: distribution old_type: meta Planing and organizing other issues type: enhancement New feature or request

Comments

@jneira
Copy link
Member

jneira commented Aug 13, 2021

//cc @hasufell @fendor

@jneira jneira added type: enhancement New feature or request old_type: meta Planing and organizing other issues component: install old_type: distribution labels Aug 13, 2021
@hasufell
Copy link
Member

hasufell commented Aug 15, 2021

Here are a few things:

  1. The build system should be stable (whether it's directly buildable via cabal or we use a wrapper script/makefile/shell script doesn't matter, as long as it's guaranteed to work for future versions). This isn't a hard requirement, but if it isn't met, any build system change may require a new full ghcup release.
  2. We need to be able to control the install destination and be able to pass arbitrary cabal flags to the invocations for maximum user control if needed. Currently, HLS build system uses cabal-install-parsers to read the global config. I'm not sure that's the best way. The build also needs to be sandboxed, so it must be possible to set --store-dir. This may be possible by overriding CABAL_DIR and adjusting all the variables in the config, but it seems rather clunky.
  3. findInstalledGhcs relies on stack-<ghcver>.yaml to decide which GHC version is supported, even for cabal. That means you currently cannot build hls-8.10.6, although that should be perfectly buildable. Users will especially want this compile feature to use HLS with newly released GHC versions. This needs to be supported.
  4. Install seems to be symlinking, instead of copying.

@jneira
Copy link
Member Author

jneira commented Sep 23, 2021

Ok, so @hasufell gently added the feature of compiling hls from source in ghcup directly so the original motivation has lost senses.
Will keep it as general issue to track improvments in the install script

@jneira jneira changed the title Improve install script to use it in ghcup's install hls from source method Track of possible improvements in the install script Sep 23, 2021
@jneira
Copy link
Member Author

jneira commented Jan 31, 2022

The install script is deprecated (#2491) and ghcup implemented the compilation without it

@jneira jneira closed this as completed Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
old_type: distribution old_type: meta Planing and organizing other issues type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants