-
Notifications
You must be signed in to change notification settings - Fork 114
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
Maven will autogenerate RiakMessageCodes #82
Conversation
Uses a template and generates com.basho.riak.protobuf.RiakMessageCodes which will now be packaged along with the protobufs
|
||
} | ||
|
||
result.put("packageName", "com.basho.riak.protobuf"); |
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.
do you think it's worth adding a "This file was generated etc" comment in the generated source?
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.
It can't hurt.
+1, although, although I'm sure @mgodave should review too.
|
Yay! 💃 |
I can't get this to build. Looking into it further... [ERROR] Failed to execute goal com.webguys:string-template-maven-plugin:1.1:render (default) on project riak-pb: Execution default of goal com.webguys:string-template-maven-plugin:1.1:render failed: An API incompatibility was encountered while executing com.webguys:string-template-maven-plugin:1.1:render: java.lang.NoSuchMethodError: org.apache.maven.execution.MavenSession.getRepositorySession()Lorg/sonatype/aether/RepositorySystemSession; |
It's a maven incompatibility issue |
There is a conflict between the the plugin and maven 3.1 |
verified that I get the same error w/ |
So, we're good with "We need to use maven 3.0.x right now, but we'll submit a patch to the plug-in repo" ? |
Yeah, logged #83 to track it. |
👍 |
Maven will autogenerate RiakMessageCodes
Uses a template and generates com.basho.riak.protobuf.RiakMessageCodes
which will now be packaged along with the protobufs
To verify, run:
mvn clean install
then inspecttarget/generated-sources/protobuf/com/basho/riak/protobuf/RiakMessageCodes.java
as well as check the jar:jar tvf target/riak-pb-2.0.0.12-SNAPSHOT.jar |grep MessageCod