Skip to content

Conversation

peterbourgon
Copy link
Contributor

The dependency matttproud/golang_protobuf_extensions changed its package layout with this commit and thereby broke the Prometheus client_golang build. This fixes it.

The dependency matttproud/golang_protobuf_extensions changed its package
layout and thereby broke the Prometheus client_golang build. This fixes
it.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm... why is ext still referenced here? It should be gone for good, shouldn't it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it should be. I'm not sure how to remove it via the godep tool; I'll remove it manually.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ugh... godeps doesn't want us to edit that file manually, but apparently you have to... weirdness continues...

@discordianfish
Copy link
Member

LGTM!

@beorn7
Copy link
Member

beorn7 commented Apr 7, 2015

👍

beorn7 added a commit that referenced this pull request Apr 7, 2015
@beorn7 beorn7 merged commit 419b21c into prometheus:master Apr 7, 2015
@peterbourgon peterbourgon deleted the fix-ext-pbutil branch April 7, 2015 13:47
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.

3 participants