Merge pull request #635 from LK4D4/vet_only_needed

Fix some suspicious things in vendor
This commit is contained in:
Daniel, Dao Quang Minh 2015-06-17 11:18:41 +08:00
commit ca3ab4ba40
3 changed files with 3 additions and 5 deletions

View File

@ -9,7 +9,7 @@ validate() {
bash /go/src/github.com/docker/docker/hack/make/validate-dco bash /go/src/github.com/docker/docker/hack/make/validate-dco
bash /go/src/github.com/docker/docker/hack/make/validate-gofmt bash /go/src/github.com/docker/docker/hack/make/validate-gofmt
go get golang.org/x/tools/cmd/vet go get golang.org/x/tools/cmd/vet
go vet github.com/docker/libcontainer/... bash /go/src/github.com/docker/docker/hack/make/validate-vet
} }
# run validations # run validations

View File

@ -44,7 +44,7 @@ clone git github.com/codegangsta/cli 1.1.0
clone git github.com/coreos/go-systemd v2 clone git github.com/coreos/go-systemd v2
clone git github.com/godbus/dbus v2 clone git github.com/godbus/dbus v2
clone git github.com/Sirupsen/logrus v0.7.3 clone git github.com/Sirupsen/logrus v0.7.3
clone git github.com/syndtr/gocapability 8e4cdcb clone git github.com/syndtr/gocapability 66ef2aa
clone git github.com/golang/protobuf 655cdfa588ea clone git github.com/golang/protobuf 655cdfa588ea
# intentionally not vendoring Docker itself... that'd be a circle :) # intentionally not vendoring Docker itself... that'd be a circle :)

View File

@ -1281,9 +1281,7 @@ func TestEnum(t *testing.T) {
// We don't care what the value actually is, just as long as it doesn't crash. // We don't care what the value actually is, just as long as it doesn't crash.
func TestPrintingNilEnumFields(t *testing.T) { func TestPrintingNilEnumFields(t *testing.T) {
pb := new(GoEnum) pb := new(GoEnum)
if fmt.Sprintf("%+v", pb) == "" { fmt.Sprintf("%+v", pb)
t.Errorf("expected non-empty string")
}
} }
// Verify that absent required fields cause Marshal/Unmarshal to return errors. // Verify that absent required fields cause Marshal/Unmarshal to return errors.