Skip to content

Add preliminary support for ghc-modi - #54

Open
int3 wants to merge 4 commits into
eagletmt:masterfrom
int3:ghc-modi
Open

Add preliminary support for ghc-modi#54
int3 wants to merge 4 commits into
eagletmt:masterfrom
int3:ghc-modi

Conversation

@int3

@int3 int3 commented Jul 24, 2014

Copy link
Copy Markdown

This enables the check and info functions to use ghc-modi if it is available. This addresses part of #48.

I had sketched an implementation for make as well, but it fails a lot of ghcmod-vim's tests due to DanielG/ghc-mod#275, so it's been left out for now.

Also, ghc-modi does not support spaces in filenames (DanielG/ghc-mod#295), so check and info inherit this issue.

The g:ghcmod_should_use_ghc_modi flag determines whether the plugin attempts to use ghc-modi.

int3 added 4 commits July 24, 2014 23:42
Right now only the 'type' and 'info' functions are supported. ghc-modi's
`make` is still somewhat buggy.
@darthdeus

Copy link
Copy Markdown

@RubenAstudillo

Copy link
Copy Markdown

I'm using this, I really like it.

@ibotty

ibotty commented Oct 6, 2014

Copy link
Copy Markdown

is anything holding it back?

@denibertovic

Copy link
Copy Markdown

What's the status on this?

@gilligan

gilligan commented Dec 2, 2014

Copy link
Copy Markdown
Contributor

@eagletmt can you merge this ? it seems to be working just fine and is a massive improvement in usability.

@rgrinberg

Copy link
Copy Markdown

+1

@denibertovic

Copy link
Copy Markdown

FYI, both issues mentioned by the @int3 in his first comment have been solved.

@ches

ches commented Feb 16, 2015

Copy link
Copy Markdown

It would be nice if :GhcModKillModi were mentioned in the doc file, and probably some general note about how the ghc-modi support works so that users have some troubleshooting hint to identify when an misbehaving process is to blame for things not working.

But otherwise 👍

@parsonsmatt

Copy link
Copy Markdown

I've been using this fork myself and it's a massive improvement. Anything we can do to bring it into master?

@liskin liskin mentioned this pull request Oct 7, 2016
@liskin

liskin commented Oct 7, 2016

Copy link
Copy Markdown

I rebased this on top of current master and changed a few things. Most notably I unified handling of newlines/boundary and added support for multiple ghc-modi instances (one per package). It works for me with a rather large multipackage haskell codebase that uses stack.

See #102 for my pull request.

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.

10 participants