"adrian(a)jboss.org" wrote : The current way the WarClassLoaderDeployer works is
all over the place. :-)
|
| INCLUDE WEB-INF
|
| I thought the requirement was to include the root of the war
| since that is what the old UCL did?
| Anyway this should be in the structure deployer not the classloader deployer.
|
No, this is the workaround for webservices packaging expectations of being able to load
resources from the WEB-INF dir. Should be in WARStructure, but its isolated to the jbossas
codebase for now.
"adrian(a)jboss.org" wrote :
| COPYING TO TEMP
|
| Again this should be in the structure deployer. The VirtualFiles is adds
| should be the temp ones.
|
this should not be needed.
"adrian(a)jboss.org" wrote :
| ALT-DD
|
| This doesn't look like it is handled properly in the new deployers?
| Shouldn't this be handled in the metadata parsing?
|
Yes, we are not handling this. Its a bit awkward as you would first have to scan the base
descriptor for the alt path and then parse that file. Doesn't neatly fit into the
current metadata parsing abstractions.
These issues are spread across the various war deployers, not just the
WarClassLoaderDeployer. You wrote "is all of the place", but meant is all over
the place. The war deployers and tomcat embedding code still await a major overhaul.
View the original post :
http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4137874#...
Reply to the post :
http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&a...