go.uuid icon indicating copy to clipboard operation
go.uuid copied to clipboard

undefined: uuid.Must

Open leffen opened this issue 7 years ago • 6 comments

Hi,

I am struggelig with dep and a dependency with this library. Unfortunately it seems like renstrom/shortuuid uses uuid.Must. Is it possible to get a release with this included so Dep can be satisfied ?

leffen avatar Jan 10 '18 08:01 leffen

Did you update your local copy of satori/uuid ? The master branch from this repository provides uuid.Must (which may or may not be reverted: refer to #66 ).

ktnyt avatar Jan 10 '18 08:01 ktnyt

The problem for seems to be that the override to master in Gopkg.toml is overwritten by dep when doing the dep ensure -update and then its reverted back to the release. Probably due to another dependency.

leffen avatar Jan 10 '18 08:01 leffen

@leffen I've tagged v1.2.0 which is last commit before API has changed but contains some bug fixes and improvements including uuid.Must helper: https://github.com/satori/go.uuid/releases/tag/v1.2.0

I plan to make a few more changes over weekend / review some PRs and going to release v2.0.0 early next week.

satori avatar Jan 10 '18 19:01 satori

Cool, Thanks. Looking forward to the new release :-)

leffen avatar Jan 10 '18 21:01 leffen

Waiting the release, you can do this to force version for the transitive dependency :

[[override]] name = "github.com/satori/go.uuid" branch = "master"

More info on : https://github.com/golang/dep/blob/master/docs/FAQ.md#how-do-i-constrain-a-transitive-dependencys-version

marcaudefroy avatar Jan 13 '18 15:01 marcaudefroy

@satori No rush, any idea when the next release is going to get tagged?

welps avatar Feb 10 '18 00:02 welps