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

MonadFail proposal #1108

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

MonadFail proposal #1108

wants to merge 5 commits into from

Conversation

xaverdh
Copy link

@xaverdh xaverdh commented Feb 17, 2019

This does the necessary changes to make yi-core compile with ghc 8.6, implementing the monad fail proposal.

@xaverdh
Copy link
Author

xaverdh commented Feb 17, 2019

Maybe someone with more knowledge of the code can tell if the marks in 6f0b1c0 are expected to be there (in which case this should be doing the right thing; erroring out), or not.
Something similar appears to be going on in Yi.Search here for example, but I understand the assumptions there even less.

@xaverdh
Copy link
Author

xaverdh commented Feb 17, 2019

The Hoogle part appears to be unused and in a bad shape in general. As it is, it would crash on you more or less immediately. I just did the bare minimum necessary to make it compile, but it should probably be reworked / dropped?

@xaverdh xaverdh changed the title WIP: MonadFail proposal MonadFail proposal Jun 15, 2019
@na4zagin3 na4zagin3 mentioned this pull request Oct 2, 2019
@jhance
Copy link
Member

jhance commented Feb 1, 2020

Since this has been waiting forever and not received any review, I'm going to play with it a bit, if it needs any changes I'll do them myself.

@xnox
Copy link

xnox commented Jul 7, 2020

Is this related?

src/Parser/Incremental.hs:185:5-8: error:
    ‘fail’ is not a (visible) method of class ‘Monad’
    |
185 |     fail _message = Empt
    |     ^^^^
Haddock coverage:
Haddock failed (no modules?), refusing to create empty documentation package.

@xaverdh
Copy link
Author

xaverdh commented Jul 7, 2020

Is this related?

src/Parser/Incremental.hs:185:5-8: error:
    ‘fail’ is not a (visible) method of class ‘Monad’
    |
185 |     fail _message = Empt
    |     ^^^^
Haddock coverage:
Haddock failed (no modules?), refusing to create empty documentation package.

Yes since ghc 8.8the Monad class no longer has fail a method (my pr was written when ghc 8.6 was the newest release).
Will need to be fixed on top of this pr.

@xnox
Copy link

xnox commented Jul 7, 2020

Ubuntu has removed yi from the Ubuntu Archive and will no longer ship it, as it fails to build from source with current 8.8.3 ghc + upgraded dependencies.

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