You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have src/encoding/gob/blorp.go (dated from Apr 17) in my source tree with the following contents for some reason (probably just an artifact related to investigating a bug a while back):
// +build ignore
package main
import (
"bytes"
"encoding/gob"
"encoding/hex"
"fmt"
"log"
)
func main() {
x := make([]int, 1000)
fmt.Println(len(x))
var b bytes.Buffer
if err := gob.NewEncoder(&b).Encode(x); err != nil {
log.Fatal(err)
}
fmt.Println(hex.EncodeToString(b.Bytes())[:52] + "...")
}
However, updating to a34b8cb, deps_test.go is now failing for me unless I remove it:
$ go test go/build
--- FAIL: TestDependencies (0.07s)
deps_test.go:472: unexpected dependency: encoding/gob imports [encoding/gob encoding/hex]
FAIL
FAIL go/build 0.088s
If this is working as intended, then I'm happy to just remove the culprit file. Just reporting in case the change in test behavior was not intentional.
The text was updated successfully, but these errors were encountered:
Looks like deps_test.go expects a newline character before the "// +build ignore" command, presumably because proper source files in the tree should all start with license headers: 54a966e#diff-46b0925ee213530c712e3199549cae21R479
If I had a blank line to the top of blorp.go, then deps_test.go passes again. I can live with that.
I have
src/encoding/gob/blorp.go
(dated from Apr 17) in my source tree with the following contents for some reason (probably just an artifact related to investigating a bug a while back):However, updating to a34b8cb, deps_test.go is now failing for me unless I remove it:
If this is working as intended, then I'm happy to just remove the culprit file. Just reporting in case the change in test behavior was not intentional.
The text was updated successfully, but these errors were encountered: