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

[[deprecated]] attribute additions break compilation of protobuf-generated files #19568

Open
pitrou opened this issue Dec 9, 2024 · 3 comments
Assignees

Comments

@pitrou
Copy link
Contributor

pitrou commented Dec 9, 2024

What version of protobuf and what language are you using?
Version: 29.0.0
Language: C++

What operating system (Linux, Windows, ...) and version?
All.

What runtime / compiler are you using (e.g., python version or gcc version)
Multiple (gcc, clang, MSVC... depending on platform)

What did you do?
Compiling Apache Arrow.
See apache/arrow#44954

What did you expect to see
I expect protobuf-generated files to compile fine.

What did you see instead?
Compiling protobuf-generated files with -Werror -Wdeprecated fails because protobuf's own generated C++ code is not safe to compile with these options.

This is because of #18036 . I commented there, but the author/submitter is a bot (are contributions to protobuf anonymous now?).

@pitrou pitrou added the untriaged auto added to all issues by default when created. label Dec 9, 2024
@pitrou
Copy link
Contributor Author

pitrou commented Dec 9, 2024

@emkornfield FYI :)

@zhangskz zhangskz added c++ 29.x and removed untriaged auto added to all issues by default when created. labels Dec 9, 2024
@tonyliaoss
Copy link
Member

This seems to be related to #18205

@zhangskz
Copy link
Member

We're rolling this back in f05e51c (and cherry-picking this for a patch to v29.2)

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

5 participants