warSourceExcludes not working properly

I checked the code for the WAR plugin and found that this is already fixed.

However, when I tested it, I found a bug in plexus.getDefaultExcludes bec it was returning a fixed-sized List so adding objects into it is causing an exception.

The plexus patch would fix the plexus bug.
The war-plugin patch is a work-around for the plexus bug.

If the plexus patch is applied, the second patch is not needed anymore.

Well, I'm not sure this bug is fixed... I've just tested the following configuration (2.0-beta-2 has just been released, and which seems to have your patch):

<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-war-plugin</artifactId>
<version>2.0-beta-2</version>
<configuration>
<warSourceExcludes>*/lib/.jar</warSourceExcludes>
</configuration>
</plugin>

and all the JARs are still excluded from the packaging.

Looking at the code, #getExcludes (which uses the 'warSourceExcludes' property) is indeed called in AbstractWarMojo#copyResources, but it is also called in WarMojo#performPackaging, which results in excluding all the JARs from the WAR - even the JARs that come from the pom dependencies.

See my last comment - #getExcludes is used both in #copyResources and #performPackaging.

Fabrice,

I tested if the bug exists first before I go and look at the source. And when I did test the bug (after the submitted fix), warSourceExcludes is working properly for the */lib/.* value. That's when I said the bug seemed fixed already.

Anyway, looking at the source, i see that you are correct that #getExcludes is inside copyResources. But please note that #copyResources is done only on the webapp directory... and lib is not in it. This is the reason why during my tests does not fail.

Also, looking at the m1 source, m1 is also applying the war.src.excludes both to the war file and the resources. So the port from m1 to m2 is still correct.

Btw, I tested again the configuration on 2.0-beta-2 and its working... the archive does not have anything inside lib but the WEB-INF/lib directory outside the archive has them all.

Hi Edwin,

I'm getting confused for it seems that we don't understand each other: the actual problem is that the final War archive has no JARs in its WEB-INF/lib folder, which is definitely not correct because a WAR must contain the libs it depends on (if not, the web application can't be run, right?).

My use case is the following:
1- I check out a project from CVS: this project has JARs in its WEB-INF/lib folder because developers put them in the SCM. But some of those JARs may be unused (even if not removed from CVS). Only the POM can tell which JARs the project really depends on.
2- I want to build the WAR with "m2 package". In the final WAR archive, I don't want to have the unused JARs, but only the ones referenced in the POM.

• With m1, when I build my project, the "war.src.excludes" property excludes the JARs that where checked out from CVS, and does not exclude the JARs that are copied from the local repo to be included in the War. And that's exactly what I expect this property to do!
• But in m2, because #getExcludes is used in #performPackaging, the JARs previously copied from the local repo to be put in the WAR are also excluded from the War! Which I don't want, of course (a WAR archive needs its libs to be able to work properly!). So 'warSourceExcludes' should normally be used only in #copyResources (because it is meant to exclude files from the WAR source), but not in #performPackaging (because at that stage, the MOJO has copied from local repo the JARs corresponding to the dependencies defined in the POM, and I don't want the MOJO to exclude such files which are not sources).

Ok. I get your point and I totally misunderstood you there.

The attached patch will remove the includes/excludes in #performPackaging.

It seems that the patch hasn't been applied yet... When will the war plugin be released with these bug fixes? It is still in beta version while most other plugins have been finalised with Maven 2.0.

I read the documentation on the war plugin (http://maven.apache.org/plugins/maven-war-plugin/war-mojo.html) and it should be possible to send in a comma separated list of excludes. I looked into the source but couldn't find anything that can make this work, so I think this is bug. The code below depends on the commons StringUtils helper class.

protected String[] getExcludes()
{
List excludeList = new ArrayList( FileUtils.getDefaultExcludesAsList() );
if ( warSourceExcludes != null && !"".equals( warSourceExcludes ) )
{
String[] excludes = StringUtils.split(warSourceExcludes, ",");
for (int i = 0; i < excludes.length; i++)

}

// if webXML is specified, omit the one in the source directory
if ( getWebXml() != null && !"".equals( getWebXml() ) )

return (String[]) excludeList.toArray( EMPTY_STRING_ARRAY );
}

I haven't written a testcase or even tested the code, but I want to verify if I've understood the function right.

Patch applied.

Niklas Ekman: more info about the plugin can be found here: http://maven.apache.org/plugins/maven-war-plugin/war-mojo.html

Has this been released yet? Which version?

Unfortunately, not yet. There are efforts in testing the war plugin prior to release. And I think the release will be soon.

I see that there seem to be some changes in the trunk that could be related to this issue, is it planned for the next release of the plugin? It seems that the 2.1 alpha 1 doesn't have this fix yet, but since it's in the trunk, the next alpha or beta might. Is there any timeframe for that release?