crowdint / gopher-spree-api

2 stars 1 forks source link

Update README, let's forget about gvp #88

Closed joiggama closed 9 years ago

joiggama commented 9 years ago

So, it turned out that the usage of gvp + gpm-local has been a PITA, especially for this project that has subpackages. From time to time, I've seen errors caused by vendored subpackages being checked out at a previous state, differing from their actual source code.

Note that the packages I mention are our project's subpackages, using gvp involves vendoring our own subpackages..., mindblowing, right?

From now on, we can rely on bare gpm without the need of gvp and gpm-local, the reason is that if we already have some packages installed, gpm only makes sure to check them out in the version we require.

isay-sosa commented 9 years ago

LGTM

cone commented 9 years ago

LGTM after a rebase to master