-
Notifications
You must be signed in to change notification settings - Fork 454
odd "no packages can be updated" #164
Comments
I think I am seeing the same issue. I have a repo that contains a go package as a sub-directory in a repository where the top level is not a go package. It saves fine but won't update. I assume this is a side-affect of not wanted to allow for updating of subpackages. But in this case it isn't a subpackage, but a top level package just contained in a subdirectory of the repo. |
Just encountered the same issue. Glad to see I'm not alone. |
Same issue here. We vendor
The exact revision at which this is happening in Prometheus is this: prometheus/prometheus@d4ef509 I wonder if this is connected to us manually including some files in |
This seems to be caused by this line here: https://github.com/tools/godep/blob/master/update.go#L205 If packages A and B are under the same root, and I try to only update B, the root will be marked for skipping update because A isn't being updated. I'm not sure what the motivation for this feature is, it seems that developers should be able to selectively update sub packages as they desire. For what it's worth, I fixed my problem by globbing from the root in my |
@mpuncel thanks so much saves a lot of time! Is this on the radar to be fixed? |
@mpuncel How would |
You're totally right. A better error message would be more than sufficient, this was a very confusing error to encounter as a user. |
@mpuncel agreed. |
@mpuncel Thank you for providing the information. That is very useful and saved my time :) |
This PR updates the `github.com/prometheus/client_golang` dependency to the v0.8.0 release tag. For future note, `godep update github.com/prometheus/client_golang/prometheus/promhttp` doesn't work correctly for this package due to the nesting structure (See tools/godep#164). This can be worked around by using `godep update github.com/prometheus/client_golang/...` instead ¯\\\_(ツ)\_/¯ Per `CONTRIBUTING.md` I verified the unit tests pass: ``` daniel@XXXXXX:~/go/src/github.com/prometheus/client_golang$ git show -s commit c5b7fccd204277076155f10851dad72b76a49317 Merge: a4d14b3 1b26087 Author: Björn Rabenstein <bjoern@rabenste.in> Date: Wed Aug 17 17:48:24 2016 +0200 Merge pull request #226 from prometheus/beorn7/alloc Bring back zero-alloc label-value access for metric vecs daniel@XXXXXX:~/go/src/github.com/prometheus/client_golang$ go test ./... ok github.com/prometheus/client_golang/api/prometheus 0.003s ? github.com/prometheus/client_golang/examples/random [no test files] ? github.com/prometheus/client_golang/examples/simple [no test files] ok github.com/prometheus/client_golang/prometheus 28.661s ok github.com/prometheus/client_golang/prometheus/promhttp 0.013s ok github.com/prometheus/client_golang/prometheus/push 0.007s ```
After a fresh git pull -u of kubernetes. The main goal here is to update the pkg/util/iptables dependency in order to leverage the iptables-restore locking support from kubernetes/kubernetes#44895. I'm not very familiar with godep, and ran into an tools/godep#164. Comments on that issue imply godep update -v repo/... are a decent workaround.
I've an old version of
go.crypto/{sha3,blowfish}
(two packaged from 1 repo) that i want to update. in manifest:i try:
:(
I ended up blowing out the dependencies manually, and re-adding it and it worked. :)
The text was updated successfully, but these errors were encountered: