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

add mod path suffix to release as v5.x.x #76

Merged
merged 1 commit into from
Jun 13, 2020

Conversation

koron
Copy link
Collaborator

@koron koron commented Jun 13, 2020

v5.8.2 以降 module enabled で go get すると v5.8.1 が振ってくるのを直せるはずの修正です。

無理矢理 v5.8.5 (最新) を module enabled で持ってこようとすると
こんな感じで怒られます。

$ GO111MODULE=on go get github.com/mattn/jvgrep@v5.8.5
go get github.com/mattn/jvgrep@v5.8.5: github.com/mattn/jvgrep@v5.8.5: invalid version: module contains a go.mod file, so major version must be compatible: should be v0 or v1, not v5

なんでかなって調べてみたら v2 以降をリリースする場合は
module path に /v2 もしくは .v2 の suffix が要るっぽいです。
ただココに置くのが正しいのかは実際にやってみないとちょっとわかりません。

@koron koron requested a review from mattn June 13, 2020 14:00
@koron
Copy link
Collaborator Author

koron commented Jun 13, 2020

これマージした後に v5.8.6 あたりをリリースすることでやっととれるはずです。
もしくは v6.0.0 にするなら /v6 に書き換えてください。

@mattn mattn merged commit 0137e3a into mattn:master Jun 13, 2020
@mattn
Copy link
Owner

mattn commented Jun 13, 2020

あざます。これ go-sqlite3 にも言える話ですね...

@koron koron deleted the release-as-v5.x.x branch June 13, 2020 14:11
johejo added a commit to johejo/jvgrep that referenced this pull request Jun 14, 2020
@johejo johejo mentioned this pull request Jun 14, 2020
johejo added a commit to johejo/jvgrep that referenced this pull request Jun 14, 2020
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.

2 participants