Skip to content

Commit

Permalink
Fix gRPC Maven configuration
Browse files Browse the repository at this point in the history
Adding multiple pluginParameter tags doesn't work,
the options need to be separated by comma.

Closes gh-1686
  • Loading branch information
mhalbritter committed Dec 20, 2024
1 parent 1f305a0 commit 7f436b3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -79,10 +79,8 @@ private void addProtobufPlugin(MavenPluginContainer plugins, VersionProperty pro
});
plugin.execution("compile", (execution) -> {
execution.goal("compile").goal("compile-custom");
execution.configuration((configuration) -> {
configuration.add("pluginParameter", "jakarta_omit");
configuration.add("pluginParameter", "@generated=omit");
});
execution.configuration(
(configuration) -> configuration.add("pluginParameter", "jakarta_omit,@generated=omit"));
});
});
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -160,8 +160,7 @@ void shouldAddProtobufPluginForMaven() {
<goal>compile-custom</goal>
</goals>
<configuration>
<pluginParameter>jakarta_omit</pluginParameter>
<pluginParameter>@generated=omit</pluginParameter>
<pluginParameter>jakarta_omit,@generated=omit</pluginParameter>
</configuration>
</execution>
</executions>
Expand Down

0 comments on commit 7f436b3

Please sign in to comment.