Skip to content

Checkout specific gnmi_ext version to fix build error#44

Merged
lguohan merged 1 commit intosonic-net:masterfrom
project-arlo:gnmi_ext_fix
Mar 25, 2020
Merged

Checkout specific gnmi_ext version to fix build error#44
lguohan merged 1 commit intosonic-net:masterfrom
project-arlo:gnmi_ext_fix

Conversation

@seiferteric
Copy link
Copy Markdown
Contributor

This is to fix build error:

../../../gopkgs/src/github.com/openconfig/gnmi/proto/gnmi_ext/gnmi_ext.pb.go:47:11: undefined: "github.com/golang/protobuf/proto".ProtoPackageIsVersion4

@renukamanavalan
Copy link
Copy Markdown

Looks similar to the last break. Will there be more to handle for correct commit-id?

@lguohan lguohan merged commit f789b29 into sonic-net:master Mar 25, 2020
lguohan pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Mar 25, 2020
@jeff-yin
Copy link
Copy Markdown

@renukamanavalan -- I think so. Eric checked and remarked:

These are the imports (direct and indirect) that I see that are not yet locked to a commit id:

golang.org/x/crypto
golang.org/x/sys
golang.org/x/text
golang.org/x/net
gopkg.in/go-playground/validator.v9
google.golang.org/grpc
google.golang.org/genproto
google.golang.org/protobuf
github.com/golang/protobuf
github.com/golang/groupcache
github.com/go-playground/locales
github.com/go-playground/universal-translator
github.com/leodido/go-urn
github.com/kylelemons/godebug
github.com/google/go-cmp
github.com/antchfx/xpath
github.com/pborman/getopt

@renukamanavalan
Copy link
Copy Markdown

Thanks Jeff! The obvious next question, 'What is the plan for locking to the commit-id?'.

abdosi pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Mar 26, 2020
tiantianlv pushed a commit to SONIC-DEV/sonic-buildimage that referenced this pull request Apr 24, 2020
@jeff-yin jeff-yin deleted the gnmi_ext_fix branch September 13, 2023 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants