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

3.12.3 might be wrongly tagged - created from master branch? (C#-release package lose IBufferMessage.cs) #7637

Closed
IllusionCui opened this issue Jun 18, 2020 · 7 comments
Assignees
Labels

Comments

@IllusionCui
Copy link

3.12.3-C#-release package lose IBufferMessage.cs
3.12.3-source-release package contains IBufferMessage.cs

@anandolee
Copy link
Contributor

Jan, can you help to take a look?

@jtattermusch
Copy link
Contributor

The 3.12.3 tag indeed contains
https://github.com/protocolbuffers/protobuf/commits/v3.12.3/csharp/src/Google.Protobuf/IBufferMessage.cs
from this PR
#7351

That is odd because to my knowledge #7351 should only be present in the master branch and it should not be available in any of the releases yet.

@haberman was the 3.12.3 tag was created wrong (maybe it was tagged off of master branch, not off of 3.12.x branch?)

See history of the release branch vs the history of the tag - they look suspiciously different?
https://github.com/protocolbuffers/protobuf/commits/3.12.x
https://github.com/protocolbuffers/protobuf/commits/v3.12.3

@haberman can you please take a look?

@jtattermusch jtattermusch changed the title 3.12.3-C#-release package lose IBufferMessage.cs 3.12.3 might be is wrongly tagged? (C#-release package lose IBufferMessage.cs) Jun 24, 2020
@jtattermusch
Copy link
Contributor

Btw, the 3.12.3 release subtitle says "· 25 commits to master since this release", which also seems to indicate the tag was created off of master branch (other releases say "N commits to 3.x.x since this release"). If the protobuf tags are created manually (same case as with grpc), this is easy to miss because there's just a small dropdown to select the source branch when creating a release.

https://github.com/protocolbuffers/protobuf/releases/tag/v3.12.3

@jtattermusch jtattermusch changed the title 3.12.3 might be is wrongly tagged? (C#-release package lose IBufferMessage.cs) 3.12.3 might be wrongly tagged? (C#-release package lose IBufferMessage.cs) Jun 24, 2020
@jtattermusch jtattermusch changed the title 3.12.3 might be wrongly tagged? (C#-release package lose IBufferMessage.cs) 3.12.3 might be wrongly tagged - created from master branch? (C#-release package lose IBufferMessage.cs) Jun 24, 2020
@kevinburke1
Copy link

Hi @anandolee, @jtattermusch, do you have an update by any chance? The 3.12.3 release was tagged incorrectly. This is causing major pain for users of the library; see the number of commits tagged to this issue and #7632 that describe reverting and/or pinning to 3.12.2. This situation has persisted for over a month now.

There is a fairly simple solution: create a 3.12.4 Git tag, and upload a new release, even if the contents are identical to 3.12.3.

Is there a chance you could make time to perform that release this week?

@ohroy
Copy link

ohroy commented Jul 28, 2020

+1
major pain

@jtattermusch
Copy link
Contributor

@dlj-NaN

@jtattermusch jtattermusch assigned dlj-NaN and unassigned haberman and jtattermusch Jul 28, 2020
@acozzette
Copy link
Member

The 3.12.4 release is now done.

xkszltl added a commit to xkszltl/Roaster that referenced this issue Jul 31, 2020
Cannot revert to 3.11.4 because grpc relies on it.
protocolbuffers/protobuf#7637
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

8 participants