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]

Reply via email to