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

v2 module incorrectly named #4271

Closed
kagyselinck-asml opened this issue Jan 20, 2025 · 4 comments
Closed

v2 module incorrectly named #4271

kagyselinck-asml opened this issue Jan 20, 2025 · 4 comments

Comments

@kagyselinck-asml
Copy link

kagyselinck-asml commented Jan 20, 2025

Good afternoon,

I noticed your v2 branch doesn't have github.com/cockroachdb/pebble/v2 as it's module name.
Is this intentional?

I'm having some issues importing your v2 branch:

command
go get github.com/cockroachdb/pebble@v2.0.0

output
go: github.com/cockroachdb/pebble@v2.0.0: invalid version: module contains a go.mod file, so module path must match major version ("github.com/cockroachdb/pebble/v2")

I could be wrong but I think this is because your go.mod file shows github.com/cockroachdb/pebble as the module name instead of github.com/cockroachdb/pebble/v2

Jira issue: PEBBLE-326

@RaduBerinde RaduBerinde self-assigned this Jan 21, 2025
RaduBerinde added a commit to RaduBerinde/pebble that referenced this issue Jan 21, 2025
A go module with version `v2.0` must have a path that ends in `/v2`;
otherwise `go get` gives you a "module path must match major version"
error.

This commit enacts this change for the pebble-release-2.0. In the
future we will consider making this change on master as well so we
don't have inconsistency between the branches.

Fixes cockroachdb#4271
@RaduBerinde
Copy link
Member

Sorry for that, will fix right away.

RaduBerinde added a commit to RaduBerinde/pebble that referenced this issue Jan 22, 2025
A go module with version `v2.0` must have a path that ends in `/v2`;
otherwise `go get` gives you a "module path must match major version"
error.

This commit enacts this change for the `pebble-release-2.0` branch. In
the future we will consider making this change on master as well so we
don't have inconsistency between the branches.

Fixes cockroachdb#4271
RaduBerinde added a commit that referenced this issue Jan 22, 2025
A go module with version `v2.0` must have a path that ends in `/v2`;
otherwise `go get` gives you a "module path must match major version"
error.

This commit enacts this change for the `pebble-release-2.0` branch. In
the future we will consider making this change on master as well so we
don't have inconsistency between the branches.

Fixes #4271
@RaduBerinde
Copy link
Member

I tagged a v2.0.1, it should work now.

@kruskall
Copy link

@RaduBerinde v2.0.1 is still using the wrong module path: /~https://github.com/cockroachdb/pebble/blob/v2.0.1/go.mod

it doesn't work:

$ go get github.com/cockroachdb/pebble/v2@v2.0.1
go: github.com/cockroachdb/pebble@v2.0.1: invalid version: module contains a go.mod file, so module path must match major version ("github.com/cockroachdb/pebble/v2")

@RaduBerinde
Copy link
Member

Oops, sorry, I tagged the wrong branch. Try now with v2.0.2 (I removed v2.0.1).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants