gnodet commented on code in PR #611:
URL: https://github.com/apache/maven-jar-plugin/pull/611#discussion_r4108264755
##########
src/main/java/org/apache/maven/plugins/jar/AbstractJarMojo.java:
##########
@@ -122,6 +122,28 @@ public abstract class AbstractJarMojo implements
org.apache.maven.api.plugin.Moj
@Parameter
protected MavenArchiveConfiguration archive = new
MavenArchiveConfiguration();
+ /**
+ * Whether to add default implementation entries ({@code
Implementation-Title},
+ * {@code Implementation-Version}, {@code Implementation-Vendor}) to the
{@code MANIFEST.MF}.
+ * When set on the command line or via a system property, this overrides
the value configured
+ * inside {@code <archive><manifest><addDefaultImplementationEntries>}.
+ *
+ * @since 3.4.3
+ */
+ @Parameter(property = "maven.jar.manifest.addDefaultImplementationEntries")
Review Comment:
Fixed in 52186d0.
##########
src/main/java/org/apache/maven/plugins/jar/AbstractJarMojo.java:
##########
@@ -122,6 +122,28 @@ public abstract class AbstractJarMojo implements
org.apache.maven.api.plugin.Moj
@Parameter
protected MavenArchiveConfiguration archive = new
MavenArchiveConfiguration();
+ /**
+ * Whether to add default implementation entries ({@code
Implementation-Title},
+ * {@code Implementation-Version}, {@code Implementation-Vendor}) to the
{@code MANIFEST.MF}.
+ * When set on the command line or via a system property, this overrides
the value configured
+ * inside {@code <archive><manifest><addDefaultImplementationEntries>}.
+ *
+ * @since 3.4.3
+ */
+ @Parameter(property = "maven.jar.manifest.addDefaultImplementationEntries")
Review Comment:
Fixed in 52186d0.
##########
src/main/java/org/apache/maven/plugins/jar/AbstractJarMojo.java:
##########
@@ -122,6 +122,28 @@ public abstract class AbstractJarMojo implements
org.apache.maven.api.plugin.Moj
@Parameter
protected MavenArchiveConfiguration archive = new
MavenArchiveConfiguration();
+ /**
+ * Whether to add default implementation entries ({@code
Implementation-Title},
+ * {@code Implementation-Version}, {@code Implementation-Vendor}) to the
{@code MANIFEST.MF}.
+ * When set on the command line or via a system property, this overrides
the value configured
+ * inside {@code <archive><manifest><addDefaultImplementationEntries>}.
+ *
+ * @since 3.4.3
+ */
+ @Parameter(property = "maven.jar.manifest.addDefaultImplementationEntries")
+ private Boolean addDefaultImplementationEntries;
+
+ /**
+ * Whether to add default specification entries ({@code
Specification-Title},
+ * {@code Specification-Version}, {@code Specification-Vendor}) to the
{@code MANIFEST.MF}.
+ * When set on the command line or via a system property, this overrides
the value configured
+ * inside {@code <archive><manifest><addDefaultSpecificationEntries>}.
+ *
+ * @since 3.4.3
+ */
+ @Parameter(property = "maven.jar.manifest.addDefaultSpecificationEntries")
+ private Boolean addDefaultSpecificationEntries;
Review Comment:
Fixed in 52186d0.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]