Skip to content

Commit

Permalink
cmd/godoc: exclude TestTypeAnalysis on Plan 9 for now
Browse files Browse the repository at this point in the history
Attempt to make build work on Plan 9.

For golang/go#11811.

Change-Id: I37a5eaef441262342994a8f77c86aa5e120deea7
Reviewed-on: https://go-review.googlesource.com/13033
Reviewed-by: Ian Lance Taylor <iant@golang.org>
  • Loading branch information
griesemer committed Jul 31, 2015
1 parent 50fdc6b commit 32dcbb4
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion cmd/godoc/godoc_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -303,6 +303,10 @@ func testWeb(t *testing.T, withIndex bool) {

// Basic integration test for godoc -analysis=type (via HTTP interface).
func TestTypeAnalysis(t *testing.T) {
if runtime.GOOS == "plan9" {
t.Skip("skipping test on plan9 (issue #11974)") // see comment re: Plan 9 below
}

// Write a fake GOROOT/GOPATH.
tmpdir, err := ioutil.TempDir("", "godoc-analysis")
if err != nil {
Expand Down Expand Up @@ -360,7 +364,7 @@ func main() { print(lib.V) }
// Wait for type analysis to complete.
reader := bufio.NewReader(stderr)
for {
s, err := reader.ReadString('\n')
s, err := reader.ReadString('\n') // on Plan 9 this fails
if err != nil {
t.Fatal(err)
}
Expand Down

0 comments on commit 32dcbb4

Please sign in to comment.