DeploymentLoader.java Revision 1416096 TOMEE-613: Extracted EARs do not deploy without META-INF/application.xml file

[AndyG <an...@orprovision.com>]

Pretty obvious what's going on here due to changes in DeploymentLoader.java
on revision 1416096:

UnknownModuleTypeException: Unable to determine module type for jar:
file:/[path]/apps/*.svn*

I have added some code to filter directory names, but not sure it's the best
way to filter. Shouldn't we just check for the existence of an 'META-INF'
sub-directory?

Input would be welcome.

Andy.




--
View this message in context: http://openejb.979440.n4.nabble.com/DeploymentLoader-java-Revision-1416096-TOMEE-613-Extracted-EARs-do-not-deploy-without-META-INF-applie-tp4659113.html
Sent from the OpenEJB Dev mailing list archive at Nabble.com.

Re: DeploymentLoader.java Revision 1416096 TOMEE-613: Extracted EARs do not deploy without META-INF/application.xml file

[Romain Manni-Bucau <rm...@gmail.com>]

Hi,

your fix is fine IMO

wonder about .git, .hg too (i have to admit i never had such dir in a binary)

Romain Manni-Bucau
Twitter: @rmannibucau
Blog: http://rmannibucau.wordpress.com/
LinkedIn: http://fr.linkedin.com/in/rmannibucau
Github: https://github.com/rmannibucau



2012/12/5 AndyG <an...@orprovision.com>:
> Pretty obvious what's going on here due to changes in DeploymentLoader.java
> on revision 1416096:
>
> UnknownModuleTypeException: Unable to determine module type for jar:
> file:/[path]/apps/*.svn*
>
> I have added some code to filter directory names, but not sure it's the best
> way to filter. Shouldn't we just check for the existence of an 'META-INF'
> sub-directory?
>
> Input would be welcome.
>
> Andy.
>
>
>
>
> --
> View this message in context: http://openejb.979440.n4.nabble.com/DeploymentLoader-java-Revision-1416096-TOMEE-613-Extracted-EARs-do-not-deploy-without-META-INF-applie-tp4659113.html
> Sent from the OpenEJB Dev mailing list archive at Nabble.com.