Skip to content
This repository has been archived by the owner on Apr 24, 2020. It is now read-only.

Throw on non-buffer, non-string messages #543

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

Conversation

platy
Copy link

@platy platy commented Jul 14, 2016

rather than converting them to strings implicitly.

Fixes #542

@platy
Copy link
Author

platy commented Jul 14, 2016

It looks like the build it failed connecting to the mirror to download zmq

@rgbkrk
Copy link

rgbkrk commented Aug 10, 2016

If you rebase or make a new commit to trigger the build we can see it again (since you and I can't retrigger the build).

@platy
Copy link
Author

platy commented Aug 19, 2016

Made a new commit but it's still failing to download packages

@ronkorving
Copy link
Collaborator

ronkorving commented Aug 22, 2016

It looks like the tarball URLs have changed on http://download.zeromq.org/
We should make a separate PR to fix that.

update: I started #553 for that

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants