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

Drop support for GHC 7.10.3 #1649

Merged
merged 9 commits into from Jan 31, 2020
Merged

Drop support for GHC 7.10.3 #1649

merged 9 commits into from Jan 31, 2020

Conversation

Gabriella439
Copy link
Collaborator

The immediate motivation for doing this is so that we don't need to
introduce yet another GHC 7.10.3 workaround for
#1640

More generally, the main reason we were still supporting GHC 7.10.3
was for Eta compatibility, but Eta appears to no longer be actively
developed. Given that Eta seems unlikely to support GHC 8 and we
don't plan on supporting GHC 7.10.3 in perpetuity it's probably
time to drop support.

We could probably drop support for additional GHC versions, but this
is just the smallest change necessary to unblock #1640.

The immediate motivation for doing this is so that we don't need to
introduce yet another GHC 7.10.3 workaround for
#1640

More generally, the main reason we were still supporting GHC 7.10.3
was for Eta compatibility, but Eta appears to no longer be actively
developed.  Given that Eta seems unlikely to support GHC 8 and we
don't plan on supporting GHC 7.10.3 in perpetuity it's probably
time to drop support.

We could probably drop support for additional GHC versions, but this
is just the smallest change necessary to unblock #1640.
Copy link
Collaborator

@sjakobi sjakobi left a comment

Choose a reason for hiding this comment

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

We can get rid of some CPP conditionals for base, directory and template-haskell with this change. I can do that in a follow-up PR!

We should also raise the lower bound on base, but maybe that should happen in #1640 instead.

@jneira Do you plan on continuing to contribute if we stop supporting 7.10 and thereby Eta? Particularly your Windows expertise was very valuable!

dhall/src/Dhall/Tags.hs Show resolved Hide resolved
@jneira
Copy link
Collaborator

jneira commented Jan 30, 2020

I've been waiting to see a specific change that needed or was easier to implement droping support for ghc-7.10.3 to advise to do it so i totally agree on this (i missed #1640 was blocked on that).
Moreover, eta has the option of apply automatically patches to packages to make them buildables. Otoh it already has support for some ghc features of versions greater than 7.10.3.

@sjakobi well, i really like dhall and that was the reason to try keep it buildable with eta and use it as the main config format for etlas so my intent is to keep contributing. You are doing a great job so the project is going forward at a good pace. Otoh, i am lately more involved in the haskell ide ecosystem but i continue keeping an eye on dhall 😉

@jneira
Copy link
Collaborator

jneira commented Jan 30, 2020

Thinking twice there are one caveat: i used eta to make the java language binding. Keeping it in sync could take more effort cause we should patch dhall to make it buildable with eta via eta-hackage.
However i am not currently maintaining it and i am not sure if it is used at all (maybe @patrickmn?)

@patrickmn
Copy link
Sponsor Collaborator

@jneira I do use the dhall libraries with Eta, but agree with @Gabriel439 that it's not sustainable to directly support 7.10.3 forever. Somebody else maintaining Eta patches for later versions seems good! I'm too time-constrained right now, but I might take that on Later(TM).

... as suggested by @sjakobi
... which is GHC 7.10.3

We should probably add a `stack-lts-*.yml` for GHC 8, but I'll leave that to
another contributor since I do not use `stack`
@Gabriella439
Copy link
Collaborator Author

As a side note, in the course of this pull request I enabled Appveyor support for canceling old builds when new revisions come in for pull requests, which will improve Appveyor efficiency

@sjakobi: My plan is to change the bound on base in #1640

@Gabriella439 Gabriella439 merged commit ce7b4be into master Jan 31, 2020
@Gabriella439 Gabriella439 deleted the gabriel/drop_7_10_3 branch January 31, 2020 21:11
Gabriella439 added a commit that referenced this pull request Feb 15, 2020
Ever since #1642 the `dhall-try` package was not being built by CI, which led
to it being inadvertently broken in #1649.  This change fixes the build for
`dhall-try` and adds it back to CI.
Gabriella439 added a commit that referenced this pull request Feb 17, 2020
Ever since #1642 the `dhall-try` package was not being built by CI, which led
to it being inadvertently broken in #1649.  This change fixes the build for
`dhall-try` and adds it back to CI.
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.

None yet

4 participants