Skip to content

/ go

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

Jump to bottom

net: TestCloseError failing on Plan 9 #20128

Closed
0intro opened this issue Apr 26, 2017 · 1 comment
Closed

net: TestCloseError failing on Plan 9 #20128

0intro opened this issue Apr 26, 2017 · 1 comment
Assignees
@0intro
Labels
FrozenDueToAge OS-Plan9
Milestone
Go1.9Early

Comments

@0intro
Copy link
Member

@0intro 0intro commented Apr 26, 2017

Since CL 39997, TestCloseError is failing on Plan 9.

--- FAIL: TestCloseError (0.01s)
	error_test.go:579: #1: error string "close tcp 127.0.0.1:52601: write /net/tcp/clone: file already closed" does not contain expected string "use of closed network connection"
	error_test.go:579: #2: error string "close tcp 127.0.0.1:52601: write /net/tcp/clone: file already closed" does not contain expected string "use of closed network connection"
FAIL
FAIL	net	23.932s

This error is returned by TCPListener.Close when writting the "hangup" string to the listener control file.

https://build.golang.org/log/728a4f3132c767a78fd84d38c47d30aa62b06ab6
@0intro 0intro added the OS-Plan9 label Apr 26, 2017
@0intro 0intro added this to the Go1.9Early milestone Apr 26, 2017
@0intro 0intro self-assigned this Apr 26, 2017
@gopherbot

This comment has been minimized.

Sign in to view
Copy link

@gopherbot gopherbot commented Apr 26, 2017

CL https://golang.org/cl/41850 mentions this issue.
@gopherbot gopherbot closed this in 8a4087a Apr 26, 2017
@golang golang locked and limited conversation to collaborators Apr 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@0intro 0intro

Labels
FrozenDueToAge OS-Plan9
Projects
None yet
Milestone
  Go1.9Early
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@0intro @gopherbot
You can’t perform that action at this time.