From: Robert Griesemer Date: Thu, 18 Jun 2015 00:02:40 +0000 (-0700) Subject: go/internal/gccgoimporter: enable tests on Plan9 X-Git-Tag: go1.5beta1~208 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=3cc7b27383c4f5814984a00c995182e632ca6de8;p=gostls13.git go/internal/gccgoimporter: enable tests on Plan9 Work-around issue #11265 and re-enable tests for Plan9. Change-Id: I3aabb674a149b8eb936f948dd4cda5fd81454646 Reviewed-on: https://go-review.googlesource.com/11194 Run-TryBot: Robert Griesemer Reviewed-by: Ian Lance Taylor --- diff --git a/src/go/internal/gccgoimporter/gccgoinstallation_test.go b/src/go/internal/gccgoimporter/gccgoinstallation_test.go index d9adcecab7..ef293edcbe 100644 --- a/src/go/internal/gccgoimporter/gccgoinstallation_test.go +++ b/src/go/internal/gccgoimporter/gccgoinstallation_test.go @@ -2,10 +2,6 @@ // Use of this source code is governed by a BSD-style // license that can be found in the LICENSE file. -// Exclude plan9 for now due to test failure with TestGoxImporter. -// TODO(gri) eliminate this build tag -// +build !plan9 - package gccgoimporter import ( diff --git a/src/go/internal/gccgoimporter/importer.go b/src/go/internal/gccgoimporter/importer.go index feb9849eef..aa0d01afdf 100644 --- a/src/go/internal/gccgoimporter/importer.go +++ b/src/go/internal/gccgoimporter/importer.go @@ -88,6 +88,12 @@ func openExportFile(fpath string) (reader io.ReadSeeker, closer io.Closer, err e if err != nil { return } + // reset to offset 0 - needed on Plan 9 (see issue #11265) + // TODO: remove once issue #11265 has been resolved. + _, err = f.Seek(0, 0) + if err != nil { + return + } var elfreader io.ReaderAt switch string(magic[:]) { diff --git a/src/go/internal/gccgoimporter/importer_test.go b/src/go/internal/gccgoimporter/importer_test.go index ce0dfb99ef..4fa01c762c 100644 --- a/src/go/internal/gccgoimporter/importer_test.go +++ b/src/go/internal/gccgoimporter/importer_test.go @@ -2,10 +2,6 @@ // Use of this source code is governed by a BSD-style // license that can be found in the LICENSE file. -// Exclude plan9 for now due to test failure with TestGoxImporter. -// TODO(gri) eliminate this build tag -// +build !plan9 - package gccgoimporter import (