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

Backport critical fixes to network-2.3 #195

Merged
merged 2 commits into from
Jun 5, 2016

Conversation

hvr
Copy link
Member

@hvr hvr commented Jun 5, 2016

These are critical fixes to the 2.3-branch which fix issues we're seeing on http://matrix.hackage.haskell.org/ for several old packages which depend on network < 2.4 directly or indirectly and therefore pick the currently broken network-2.3.2.0 release.

I've tested this patch from GHC 7.0 through GHC 8.0.

Please upload a network-2.3.2.1 release to Hackage soon so that install-plans affected are restored

/cc @phadej @bergmark

@hvr hvr added the bug fix label Jun 5, 2016
@eborden
Copy link
Collaborator

eborden commented Jun 5, 2016

The changes look fine. I'm not sure why travis fell on its face on this one.

@hvr
Copy link
Member Author

hvr commented Jun 5, 2016

@eborden because in the 2.3 branch there was no travis script yet!

@eborden
Copy link
Collaborator

eborden commented Jun 5, 2016

That'll do it :)

@eborden eborden merged commit 5af3a23 into haskell:2.3 Jun 5, 2016
@hvr
Copy link
Member Author

hvr commented Jun 5, 2016

@eborden ...you were optimistic to merge this before the travis script finished :-)

@hvr hvr deleted the pr/network-2.3-maintainance branch June 5, 2016 20:09
@eborden
Copy link
Collaborator

eborden commented Jun 5, 2016

ick, my brain is not screwed on right today. I'll likely have to back this out and re-open this PR.

@eborden
Copy link
Collaborator

eborden commented Jun 5, 2016

Ah, @hvr I didn't notice you snuck that travis script in.

@hvr
Copy link
Member Author

hvr commented Jun 5, 2016

np, I'll fix it up; gimme a sec

@eborden
Copy link
Collaborator

eborden commented Jun 5, 2016

@hvr This is the same failure I saw over here #194. Seems like there is an issue in the auto gen.

@eborden
Copy link
Collaborator

eborden commented Jun 5, 2016

Fix is being handled here #197

@hvr hvr mentioned this pull request Jun 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants