Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1741)

Issue 4174056: net: fix multicast tests (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 1 month ago by dfc
Modified:
13 years, 1 month ago
Reviewers:
CC:
rsc, mikio, rsc1, dho, golang-dev
Visibility:
Public.

Description

net: fix multicast tests

Patch Set 1 #

Patch Set 2 : diff -r f814f61fdbab https://go.googlecode.com/hg/ #

Total comments: 4

Patch Set 3 : diff -r afc3d9f30d78 https://go.googlecode.com/hg/ #

Patch Set 4 : diff -r afc3d9f30d78 https://go.googlecode.com/hg/ #

Patch Set 5 : diff -r 8e157f1abc87 https://go.googlecode.com/hg/ #

Patch Set 6 : diff -r 8e157f1abc87 https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M src/pkg/net/multicast_test.go View 1 2 3 4 5 3 chunks +3 lines, -1 line 0 comments Download

Messages

Total messages: 17
mikio
http://codereview.appspot.com/4174056/diff/2001/src/pkg/net/multicast_test.go File src/pkg/net/multicast_test.go (right): http://codereview.appspot.com/4174056/diff/2001/src/pkg/net/multicast_test.go#newcode28 src/pkg/net/multicast_test.go:28: mcast := IPv4(224, 7, 7, 7) 224.0.1/24 is internet ...
13 years, 1 month ago (2011-02-17 13:16:39 UTC) #1
rsc1
I don't know anything about multicast addresses. If you guys can come to an agreement ...
13 years, 1 month ago (2011-02-17 14:43:12 UTC) #2
mikio
On Thu, Feb 17, 2011 at 9:43 AM, <rsc@google.com> wrote: > If you guys can ...
13 years, 1 month ago (2011-02-17 15:08:05 UTC) #3
rsc1
So is the correct change 224.0.0.251 -> 224.0.0.254 ? Was there a reason the original ...
13 years, 1 month ago (2011-02-17 15:33:27 UTC) #4
mikio
On Thu, Feb 17, 2011 at 10:33 AM, Russ Cox <rsc@google.com> wrote: > So is ...
13 years, 1 month ago (2011-02-17 15:58:33 UTC) #5
dfc
I can't reproduce it on a VM, and the freebsd/386 build passes with the same ...
13 years, 1 month ago (2011-02-17 22:50:10 UTC) #6
dfc
Mikioh, RSC, please take another look. I've asked Devon to try this patch directly to ...
13 years, 1 month ago (2011-02-17 22:58:29 UTC) #7
dho
Hi guys, I'll be able to test this momentarily when I get home... --dho 2011/2/17 ...
13 years, 1 month ago (2011-02-17 23:19:20 UTC) #8
dho
So, I applied the requested patch, but it seems to only fix Makefiles. I tried ...
13 years, 1 month ago (2011-02-18 00:44:11 UTC) #9
dfc
Sorry, that may have been my mistake, I may have given you the wrong cl ...
13 years, 1 month ago (2011-02-18 00:50:27 UTC) #10
dfc
According to dho the last patch did not fix the issue. I have changed the ...
13 years, 1 month ago (2011-02-18 01:58:02 UTC) #11
dho
On 2011/02/18 01:58:02, dfc wrote: > According to dho the last patch did not fix ...
13 years, 1 month ago (2011-02-18 04:05:00 UTC) #12
rsc1
Please re-enable the test and delete the flag, since we fixed FreeBSD.
13 years, 1 month ago (2011-02-18 07:06:28 UTC) #13
dfc
Please take another look. This final patchset updates the mcast address, but leaves the test ...
13 years, 1 month ago (2011-02-18 07:12:40 UTC) #14
rsc1
LGTM Thanks.
13 years, 1 month ago (2011-02-18 07:17:59 UTC) #15
mikio
LGTM
13 years, 1 month ago (2011-02-18 16:41:18 UTC) #16
rsc
13 years, 1 month ago (2011-02-18 18:48:15 UTC) #17
*** Submitted as http://code.google.com/p/go/source/detail?r=3ccbde90814f ***

net: fix multicast tests

R=rsc, mikioh, rsc1, dho
CC=golang-dev
http://codereview.appspot.com/4174056

Committer: Russ Cox <rsc@golang.org>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b