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

Fixing warnings #240

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

Fixing warnings #240

wants to merge 26 commits into from

Conversation

LinuxUser404
Copy link

@LinuxUser404 LinuxUser404 commented Aug 6, 2019

I fixed a bunch of warnings here and there(thrown at me by a recent ghc). I was going to move this whole thing to a more recent version of cabal, but eventually ended up reverting everything back, to keep things nice and clear. I hope I didn't break anything. I commented some unused things instead of deleting them because they looked like some test cases, which never get called for some reason.

@LinuxUser404
Copy link
Author

LinuxUser404 commented Aug 7, 2019

I have figured out why the build is failing btw. It is due too a missing dependency in libssh2:
portnov/libssh2-hs#54

@andreasabel
Copy link
Member

@deech Why hasn't this PR gotten any attention? (Looks useful.)

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

2 participants