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

Dependency version warning #99

Open
Ben131-Go opened this issue Jan 29, 2023 · 0 comments
Open

Dependency version warning #99

Ben131-Go opened this issue Jan 29, 2023 · 0 comments

Comments

@Ben131-Go
Copy link

Dependency line:

github.com/observatorium/thanos-receive-controller --> github.com/thanos-io/thanos --> github.com/bradfitz/gomemcache

github.com/thanos-io/thanos v0.25.2 --> github.com/bradfitz/gomemcache 24332e2

https://github.com/thanos-io/thanos/blob/v0.25.2/go.mod#L210

Background

Repo github.com/thanos-io/thanos at version v0.25.2 uses replace directive to pin dependency github.com/bradfitz/gomemcache to version 24332e2.

According to Go Modules wikis, replace directives in modules other than the main module are ignored when building the main module.
It means such replace usage in dependency's go.mod cannot be inherited when building main module. And it turns out that observatorium/thanos-receive-controller indirectly relies on bradfitz/gomemcache@a41fca8, which is different from the pinned version thanos-io/thanos needed.

https://github.com/observatorium/thanos-receive-controller/blob/master/go.mod(Line 26)

github.com/bradfitz/gomemcache v0.0.0-20190913173617-a41fca850d0b // indirect

https://github.com/thanos-io/thanos/blob/v0.25.2/go.mod(line 18&210)

github.com/bradfitz/gomemcache v0.0.0-20190913173617-a41fca850d0b
github.com/bradfitz/gomemcache => github.com/themihai/gomemcache v0.0.0-20180902122335-24332e2d58ab

So this is just a reminder in the hope that you can notice such an inconsistency.

Solution

1. Bump the version of dependency github.com/thanos-io/thanos

You may try upgrading dependency github.com/thanos-io/thanos to a newer version, which may have eliminated the use of this directive.

2. Add the same replace rule to your go.mod

replace github.com/bradfitz/gomemcache => github.com/themihai/gomemcache v0.0.0-20180902122335-24332e2d58ab
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

1 participant