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

Fixes to compile on ghc-8.10.1 #572

Merged
merged 2 commits into from
Apr 15, 2020
Merged

Fixes to compile on ghc-8.10.1 #572

merged 2 commits into from
Apr 15, 2020

Conversation

newhoggy
Copy link
Contributor

@newhoggy newhoggy commented Apr 10, 2020

  • Bump upper bound of haskell-src-exts.
  • Fix MonadUnlifIO compiler errors on ghc-8.10.1

@@ -249,8 +249,7 @@ instance MonadBaseControl b m => MonadBaseControl b (AWST' r m) where
restoreM = defaultRestoreM

instance MonadUnliftIO m => MonadUnliftIO (AWST' r m) where
askUnliftIO = AWST' $ (\(UnliftIO f) -> UnliftIO $ f . unAWST)
<$> askUnliftIO
withRunInIO inner = (AWST' $ (\(UnliftIO f) -> UnliftIO $ f . unAWST) <$> askUnliftIO) >>= \u -> liftIO (inner (unliftIO u))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Basically inlined the old definition of withRunInIO: https://hackage.haskell.org/package/unliftio-core-0.1.2.0/docs/src/Control.Monad.IO.Unlift.html#withRunInIO

There may be a better way to write this.

@newhoggy
Copy link
Contributor Author

If this works, I'd like a new release to support ghc-8.10.1

@newhoggy newhoggy changed the title Bump upper bound of haskell-src-exts Fixes to compile on ghc-8.10.1 Apr 11, 2020
@brendanhay
Copy link
Owner

👍

@brendanhay brendanhay merged commit 43ddd87 into brendanhay:develop Apr 15, 2020
@newhoggy newhoggy deleted the bump-upper-bound-of-haskell-src-exts branch April 15, 2020 15:59
@TomMD
Copy link

TomMD commented Aug 18, 2020

Could we get a release to hackage with this merge?

@ocharles
Copy link

Another request for a release 🙏

@chris-martin
Copy link

unlift-io-0.2 is still unsupported on Hackage - Is there anything we can do to help make this happen? I think you've got a lot of people willing to help here.

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.

5 participants