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

net: ListenMulticastUDP doesn't work on Solaris #20898

Closed
iamacarpet opened this issue Jul 4, 2017 · 2 comments

Comments

Projects
None yet
4 participants
@iamacarpet
Copy link

commented Jul 4, 2017

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

1.8.3

What operating system and processor architecture are you using (go env)?

linux/amd64 and darwin/amd64 building for solaris/amd64

What did you do?

Trying to set up a multicast UDP listen using net.ListenMulticastUDP

On Linux, this runs great, but on Solaris it returns

listen udp 239.255.255.250:1900: option not supported by protocol

This is an example of the method that doesn't work: https://gist.github.com/iamacarpet/d0ea5c28f1753c7b624b79b9f5d6504d#file-listen_default-go

And this is an example that does work (using golang.org/x/net/ipv4): https://gist.github.com/iamacarpet/d0ea5c28f1753c7b624b79b9f5d6504d#file-listen_ipv4-go

I wasn't sure if this related to #7399, but then I'd expect it to not work at all, rather than having one method that works and one that is broken.

@mikioh mikioh changed the title Multicast UDP listen using net.ListenMulticastUDP fails (Solaris) net: ListenMulticastUDP fails on Solaris Jul 5, 2017

@mikioh mikioh added the OS-Solaris label Jul 5, 2017

@mikioh

This comment has been minimized.

Copy link
Contributor

commented Jul 5, 2017

Basically #7399 is unrelated even though the functionality is required for testing. The reason why ListenMulticatsUDP doesn't work (and the external networking library does work well for IP multicasting) is because the solaris port maintainers didn't enable the function when they made the solaris port available.

Simply tweaking build constraint lines in the net and internal/poll packages of the standard library could enable the function on Solaris.

@mikioh mikioh changed the title net: ListenMulticastUDP fails on Solaris net: ListenMulticastUDP doesn't work on Solaris Jul 5, 2017

@gopherbot

This comment has been minimized.

Copy link

commented Jul 5, 2017

CL https://golang.org/cl/47450 mentions this issue.

@bradfitz bradfitz added the NeedsFix label Jul 5, 2017

@bradfitz bradfitz added this to the Go1.10 milestone Jul 5, 2017

@gopherbot gopherbot closed this in 6362fea Aug 9, 2017

@golang golang locked and limited conversation to collaborators Aug 9, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.