-
Notifications
You must be signed in to change notification settings - Fork 3.9k
COMPILING.md: update protoc version requirement #7064
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
Conversation
older version can't compile new proto3 field presence changes (#7058) ``` > Task :grpc-compiler:compileJava_pluginExecutableJava_pluginCpp FAILED Deprecated Gradle features were used in this build, making it incompatible with Gradle 6.0. Use '--warning-mode all' to show the individual deprecation warnings. See https://docs.gradle.org/5.6.2/userguide/command_line_interface.html#sec:command_line_warnings 67 actionable tasks: 21 executed, 46 up-to-date <-------------> 0% WAITING > Resolve files of :grpc-android-interop-testing:debugCompileClasspath > Transf > Transforming artifact annotations.jar (com.google.android:annotations:4.1.1.4 > Resolve files of :grpc-android-interop-testing:debugCompileClasspath > Transf > :grpc-android:verifyReleaseResources /home/jihuncho/git/grpc-java/compiler/src/java_plugin/cpp/java_plugin.cpp:46:12: error: ‘uint64_t JavaGrpcGenerator::GetSupportedFeatures() const’ marked ‘override’, but does not override 46 | uint64_t GetSupportedFeatures() const override { | ^~~~~~~~~~~~~~~~~~~~ /home/jihuncho/git/grpc-java/compiler/src/java_plugin/cpp/java_plugin.cpp: In member function ‘uint64_t JavaGrpcGenerator::GetSupportedFeatures() const’: /home/jihuncho/git/grpc-java/compiler/src/java_plugin/cpp/java_plugin.cpp:47:12: error: ‘FEATURE_PROTO3_OPTIONAL’ was not declared in this scope 47 | return FEATURE_PROTO3_OPTIONAL; | ^~~~~~~~~~~~~~~~~~~~~~~ ```
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
compiler/README.md
also referred to 3.0.0
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is changed in this line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
git added LF before EOF.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but that is probably already broken for many release if using protobuf 3.0.0
possibly yes. I was using |
older version can't compile new proto3 field presence changes (#7058)