Add configuration files so the project can be built with cabal #1667
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
janmasrovira
approved these changes
Dec 14, 2022
Cabal requires the files to be specified here because it uses an isolated build work directory.
janmasrovira
force-pushed
the
cabal-alt
branch
from
d1f756d
to
3824907
Compare
janmasrovira
pushed a commit
that referenced
this pull request
Dec 15, 2022
* Add files to embed in the exe to extra-source-files Cabal requires the files to be specified here because it uses an isolated build work directory. * Add cabal configuration files * Document how to use `cabal` with the project
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR makes it possible to build the project using
cabalas an alternative to usingstack.stackto build and test the project.stackshould see no change in their developer experienceThe reason why you might want to use
cabalinstead ofstackis that the LSP integration seems to work better (I get fewer LSP crashes, and I can switch between the test, lib and app targets without problems).The
import: https://www.stackage.org/lts-20.2/cabal.configline in thecabal.projectmeans thatstackandcabalwill resolve the same dependencies. We need to keep this link in sync with thestackresolver version.To use cabal with LSP you need to manually copy
cabal.hie.yamltohie.yaml. Unfortunately there's no way to configure thehie.yamlpath in his or hie-bios configuration options.