Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

multiple-value uuid.NewV4() in single-value context #72

Closed
davidrneves opened this issue Feb 21, 2018 · 3 comments
Closed

multiple-value uuid.NewV4() in single-value context #72

davidrneves opened this issue Feb 21, 2018 · 3 comments

Comments

@davidrneves
Copy link

davidrneves commented Feb 21, 2018

When I try to build caduceus master branch in a centOS 7 machine with go build, the go compiler gives the following error:

./outboundSender.go:495:27: multiple-value uuid.NewV4() in single-value context

This seems to be related with the following issue: hlandau/acmetool#293
This issue is due to an upstream API change in the github.com/satori/go.uuid package, specifically the change for satori/go.uuid#18. (https://godoc.org/github.com/satori/go.uuid#NewV4)

A simple fix would be:

if "" == tid {
randUuid, err: = uuid.NewV4()

if err != nil {
logging.Error(obs.logger).Log(logging.MessageKey(), "New Post request", "url", obs.listener.Config.URL,
logging.ErrorKey(), err)
// Handle error
} else {
tid = randUuid.String()
}
}

@schmidtw
Copy link
Member

To get the dependencies, are you using glide install or go get?

I've got caduceus pinned to f58768cc1a7a7e77a3bd49e98cdd21419399b6a3 of satori/go.uuid (latest release 1.2).

I'm not opposed to moving the revision we're building with, but noticed that my build failed due to it being pinned to the older release of the uuid.

@schmidtw
Copy link
Member

To build, I do the following:

cd ~/Projects/caduceus
export GOPATH=`pwd`
cd src
glide install
cd caduceus
go build

@davidrneves
Copy link
Author

I was using "go get" to install the project dependencies. With "glide install" the error is fixed. Thanks for your help.

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

No branches or pull requests

2 participants