Skip to content

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