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

WIP: Update deps for lts 18.10 -- Issue #205 #206

Open
wants to merge 1 commit into
base: postgres
Choose a base branch
from

Conversation

lowderdev
Copy link

Issue #205

My plan of attack was as follows:

  • run yesod-scaffold build
  • check error message like:
    persistent version 2.13.1.2 found
        - PROJECTNAME requires >=2.9 && <2.11
  • bump dep to next minor version -- error says 2.13.1.2 found, so bumped <2.11 to <2.14

However I also ran into this error:

    classy-prelude-yesod not found
        - PROJECTNAME requires ==1.5.*

I guess classy-prelude-yesod doesn't ship with lts 18.10, and I don't see a stack.yaml at the root level of the project that I could add extra-deps to.

Sorry I couldn't get further than this! Any guidance on how to get passed this dependence on classy-prelude-yesod?
My guess is that possible paths forward are:

  • try bumping to a lower lts
  • update classy-prelude-yesod to work with lts 18.10
  • figure out a way to add extra-deps to the scaffold

Thoughts? Thanks in advance

@snoyberg
Copy link
Member

I didn't remember that classy-prelude-yesod isn't in LTS. I think either adding it in, or modifying the scaffold to not use classy-prelude-yesod, are both good approaches here.

@malteneuss
Copy link

malteneuss commented Mar 15, 2022

@lowderdev
You probably need to add

extra-deps:
  - classy-prelude-yesod-1.5.0

to stack.yaml

edit: i've just seen that there's no such file in the postgres branch.
@snoyberg Is there some other way to add this setting? The alternative of removing classy-prelude-yesod seems to be stuck in #194 for a long time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants