You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Nagesh Susarla (JIRA)" <ji...@codehaus.org> on 2009/06/12 23:03:42 UTC

[jira] Created: (MWAR-197) war:manifest does not add "manifestEntries" to generated manifest

war:manifest does not add "manifestEntries" to generated manifest
-----------------------------------------------------------------

                 Key: MWAR-197
                 URL: http://jira.codehaus.org/browse/MWAR-197
             Project: Maven 2.x WAR Plugin
          Issue Type: Bug
    Affects Versions: 2.1-beta-1, 2.1-alpha-2, 2.1-alpha-1, 2.0.2
            Reporter: Nagesh Susarla
         Attachments: unit_test_to_add.patch, WarManifestMojo.patch

Team: As stated in the summary, the manifest goal doesnt seem to add any of the manifestEntries to the generated manifest. Here is a testcase that I added to the project which exposes the issue

<project>
  <name>war-plugin-test</name>
  <build>
    <plugins>
      <plugin>
        <artifactId>maven-war-plugin</artifactId>
        <configuration>
          <project implementation="org.apache.maven.plugin.war.stub.MavenProjectBasicStub"/>
          <warSourceDirectory>
            ${basedir}/target/test-classes/unit/manifest/manifest-with-custom-attrs/src/main</warSourceDirectory>
          <archive>
            <manifest>
              <addExtensions>true</addExtensions>
              <addDefaultImplementationEntries>true</addDefaultImplementationEntries>
              <addDefaultSpecificationEntries>true</addDefaultSpecificationEntries>
            </manifest>
            <manifestEntries>
              <Custom-Version>2.0</Custom-Version>
	        </manifestEntries>
          </archive>
        </configuration>
      </plugin>
    </plugins>
  </build>
</project>


Expected: "Custom-Version" is expected in the manifest. But it not present in the generated manifest

Actual:

Manifest-Version: 1.0
Created-By: Apache Maven
Built-By: nageshs
Build-Jdk: 1.6.0_07
Specification-Title: Test Project 
Specification-Version: 0.0-Test
Specification-Vendor: Test Name
Implementation-Title: Test Project 
Implementation-Version: 0.0-Test
Implementation-Vendor-Id: org.apache.maven.plugin.test
Implementation-Vendor: Test Name


I'm also attaching a fix along with a unit test to test the presence of the custom manifest entry.

After the fix all tests pass and the "Custom-Version" is also included in the manifest.


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Closed: (MWAR-197) war:manifest does not add "manifestEntries" to generated manifest

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MWAR-197?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dennis Lundberg closed MWAR-197.
--------------------------------

       Resolution: Fixed
    Fix Version/s: 2.1-beta-2
         Assignee: Dennis Lundberg

Fixed in [r984603|http://svn.apache.org/viewvc?view=revision&revision=984603].
Thanks for the patches!

> war:manifest does not add "manifestEntries" to generated manifest
> -----------------------------------------------------------------
>
>                 Key: MWAR-197
>                 URL: http://jira.codehaus.org/browse/MWAR-197
>             Project: Maven 2.x WAR Plugin
>          Issue Type: Bug
>    Affects Versions: 2.0.2, 2.1-alpha-1, 2.1-alpha-2, 2.1-beta-1
>            Reporter: Nagesh Susarla
>            Assignee: Dennis Lundberg
>             Fix For: 2.1-beta-2
>
>         Attachments: unit_test_to_add.patch, WarManifestMojo.patch
>
>
> Team: As stated in the summary, the manifest goal doesnt seem to add any of the manifestEntries to the generated manifest. Here is a testcase that I added to the project which exposes the issue
> {code:xml}
> <project>
>   <name>war-plugin-test</name>
>   <build>
>     <plugins>
>       <plugin>
>         <artifactId>maven-war-plugin</artifactId>
>         <configuration>
>           <project implementation="org.apache.maven.plugin.war.stub.MavenProjectBasicStub"/>
>           <warSourceDirectory>
>             ${basedir}/target/test-classes/unit/manifest/manifest-with-custom-attrs/src/main</warSourceDirectory>
>           <archive>
>             <manifest>
>               <addExtensions>true</addExtensions>
>               <addDefaultImplementationEntries>true</addDefaultImplementationEntries>
>               <addDefaultSpecificationEntries>true</addDefaultSpecificationEntries>
>             </manifest>
>             <manifestEntries>
>               <Custom-Version>2.0</Custom-Version>
>             </manifestEntries>
>           </archive>
>         </configuration>
>       </plugin>
>     </plugins>
>   </build>
> </project>
> {code}
> Expected: "Custom-Version" is expected in the manifest. But it not present in the generated manifest
> Actual:
> Manifest-Version: 1.0
> Created-By: Apache Maven
> Built-By: nageshs
> Build-Jdk: 1.6.0_07
> Specification-Title: Test Project 
> Specification-Version: 0.0-Test
> Specification-Vendor: Test Name
> Implementation-Title: Test Project 
> Implementation-Version: 0.0-Test
> Implementation-Vendor-Id: org.apache.maven.plugin.test
> Implementation-Vendor: Test Name
> I'm also attaching a fix along with a unit test to test the presence of the custom manifest entry.
> After the fix all tests pass and the "Custom-Version" is also included in the manifest.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (MWAR-197) war:manifest does not add "manifestEntries" to generated manifest

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MWAR-197?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dennis Lundberg updated MWAR-197:
---------------------------------

    Description: 
Team: As stated in the summary, the manifest goal doesnt seem to add any of the manifestEntries to the generated manifest. Here is a testcase that I added to the project which exposes the issue
{code:xml}
<project>
  <name>war-plugin-test</name>
  <build>
    <plugins>
      <plugin>
        <artifactId>maven-war-plugin</artifactId>
        <configuration>
          <project implementation="org.apache.maven.plugin.war.stub.MavenProjectBasicStub"/>
          <warSourceDirectory>
            ${basedir}/target/test-classes/unit/manifest/manifest-with-custom-attrs/src/main</warSourceDirectory>
          <archive>
            <manifest>
              <addExtensions>true</addExtensions>
              <addDefaultImplementationEntries>true</addDefaultImplementationEntries>
              <addDefaultSpecificationEntries>true</addDefaultSpecificationEntries>
            </manifest>
            <manifestEntries>
              <Custom-Version>2.0</Custom-Version>
            </manifestEntries>
          </archive>
        </configuration>
      </plugin>
    </plugins>
  </build>
</project>
{code}

Expected: "Custom-Version" is expected in the manifest. But it not present in the generated manifest

Actual:

Manifest-Version: 1.0
Created-By: Apache Maven
Built-By: nageshs
Build-Jdk: 1.6.0_07
Specification-Title: Test Project 
Specification-Version: 0.0-Test
Specification-Vendor: Test Name
Implementation-Title: Test Project 
Implementation-Version: 0.0-Test
Implementation-Vendor-Id: org.apache.maven.plugin.test
Implementation-Vendor: Test Name


I'm also attaching a fix along with a unit test to test the presence of the custom manifest entry.

After the fix all tests pass and the "Custom-Version" is also included in the manifest.


  was:
Team: As stated in the summary, the manifest goal doesnt seem to add any of the manifestEntries to the generated manifest. Here is a testcase that I added to the project which exposes the issue

<project>
  <name>war-plugin-test</name>
  <build>
    <plugins>
      <plugin>
        <artifactId>maven-war-plugin</artifactId>
        <configuration>
          <project implementation="org.apache.maven.plugin.war.stub.MavenProjectBasicStub"/>
          <warSourceDirectory>
            ${basedir}/target/test-classes/unit/manifest/manifest-with-custom-attrs/src/main</warSourceDirectory>
          <archive>
            <manifest>
              <addExtensions>true</addExtensions>
              <addDefaultImplementationEntries>true</addDefaultImplementationEntries>
              <addDefaultSpecificationEntries>true</addDefaultSpecificationEntries>
            </manifest>
            <manifestEntries>
              <Custom-Version>2.0</Custom-Version>
	        </manifestEntries>
          </archive>
        </configuration>
      </plugin>
    </plugins>
  </build>
</project>


Expected: "Custom-Version" is expected in the manifest. But it not present in the generated manifest

Actual:

Manifest-Version: 1.0
Created-By: Apache Maven
Built-By: nageshs
Build-Jdk: 1.6.0_07
Specification-Title: Test Project 
Specification-Version: 0.0-Test
Specification-Vendor: Test Name
Implementation-Title: Test Project 
Implementation-Version: 0.0-Test
Implementation-Vendor-Id: org.apache.maven.plugin.test
Implementation-Vendor: Test Name


I'm also attaching a fix along with a unit test to test the presence of the custom manifest entry.

After the fix all tests pass and the "Custom-Version" is also included in the manifest.



> war:manifest does not add "manifestEntries" to generated manifest
> -----------------------------------------------------------------
>
>                 Key: MWAR-197
>                 URL: http://jira.codehaus.org/browse/MWAR-197
>             Project: Maven 2.x WAR Plugin
>          Issue Type: Bug
>    Affects Versions: 2.0.2, 2.1-alpha-1, 2.1-alpha-2, 2.1-beta-1
>            Reporter: Nagesh Susarla
>         Attachments: unit_test_to_add.patch, WarManifestMojo.patch
>
>
> Team: As stated in the summary, the manifest goal doesnt seem to add any of the manifestEntries to the generated manifest. Here is a testcase that I added to the project which exposes the issue
> {code:xml}
> <project>
>   <name>war-plugin-test</name>
>   <build>
>     <plugins>
>       <plugin>
>         <artifactId>maven-war-plugin</artifactId>
>         <configuration>
>           <project implementation="org.apache.maven.plugin.war.stub.MavenProjectBasicStub"/>
>           <warSourceDirectory>
>             ${basedir}/target/test-classes/unit/manifest/manifest-with-custom-attrs/src/main</warSourceDirectory>
>           <archive>
>             <manifest>
>               <addExtensions>true</addExtensions>
>               <addDefaultImplementationEntries>true</addDefaultImplementationEntries>
>               <addDefaultSpecificationEntries>true</addDefaultSpecificationEntries>
>             </manifest>
>             <manifestEntries>
>               <Custom-Version>2.0</Custom-Version>
>             </manifestEntries>
>           </archive>
>         </configuration>
>       </plugin>
>     </plugins>
>   </build>
> </project>
> {code}
> Expected: "Custom-Version" is expected in the manifest. But it not present in the generated manifest
> Actual:
> Manifest-Version: 1.0
> Created-By: Apache Maven
> Built-By: nageshs
> Build-Jdk: 1.6.0_07
> Specification-Title: Test Project 
> Specification-Version: 0.0-Test
> Specification-Vendor: Test Name
> Implementation-Title: Test Project 
> Implementation-Version: 0.0-Test
> Implementation-Vendor-Id: org.apache.maven.plugin.test
> Implementation-Vendor: Test Name
> I'm also attaching a fix along with a unit test to test the presence of the custom manifest entry.
> After the fix all tests pass and the "Custom-Version" is also included in the manifest.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira