[#MOJO-377] Strip META-INF/**.xml files - jira.codehaus.org

Mojo

Strip META-INF/**.xml files

Created: 21/Apr/06 12:40 PM Updated: 28/Mar/08 02:22 PM Resolved: 14/Mar/08 04:08 AM

Return to search

Component/s:

jboss-packaging-maven-plugin

Affects Version/s:

None

Fix Version/s:

None

Time Tracking:

Not Specified

File Attachments:

AbstractSarMojo.patch (3 kB)
2.

AbstractSarMojo.patch (3 kB)
3.

jorgs.patch (3 kB)

Complexity:

Intermediate

Description

« Hide

The JBoss JARDeployer rejects all jars containing META-INF/**.xml files. Since jars created using maven by default contain a Maven and contain a META-INF/maven/<groupId>/<artifactId>/pom.xml file, all those JARs are rejected during deployment and thus excluded from the classpath.

The attached patch fixes this problem by filtering/skipping all META-INF/**.xml files during SAR assembly.

Sort Order:

[ Permalink | « Hide ]

Alex Karasulu added a comment - 21/Apr/06 02:29 PM

Trying to apply your patch but I keep getting this:

[akarasulu@newton sar]$ patch -p0 < AbstractSarMojo.patch
(Stripping trailing CRs from patch.)
patch: **** File . is not a regular file – can't patch

Did you generate this patch with svn diff?

[ Permalink | « Hide ]

Jörg Henne added a comment - 24/Apr/06 06:13 AM

I created it using the subversive eclipse plugin. Here is one that hopefully works better. Sorry.

Jörg Henne made changes - 24/Apr/06 06:13 AM

Field	Original Value	New Value
Attachment		AbstractSarMojo.patch [ 20232 ]

[ Permalink | « Hide ]

Alex Karasulu added a comment - 17/Aug/06 11:02 PM

Jorg I took your patch and hand edited the file by reading your patch. I could never get it to work. Hopefully this is good enough for someone to apply it. I used to have access to this repo but I lost me ssh key on another machine to beaver. I'll ask the codehaus guys to hook me up again.

Alex Karasulu made changes - 17/Aug/06 11:02 PM

Attachment

jorgs.patch [ 22313 ]

[ Permalink | « Hide ]

Alex Karasulu added a comment - 17/Aug/06 11:23 PM

I applied your patch. Found how to reset my codehaus password .

[ Permalink | « Hide ]

Paul Gier added a comment - 13/Mar/08 05:46 PM

Since the patch was applied, can this issue be closed now?

Jörg Henne made changes - 14/Mar/08 04:08 AM

Resolution		Fixed [ 1 ]
Status	Open [ 1 ]	Closed [ 6 ]