The current way the WarClassLoaderDeployer works is all of 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. Same goes for some of the other flags, is "acceptNonWarDirs" still used? COPYING TO TEMP Again this should be in the structure deployer. The VirtualFiles is adds should be the temp ones. ALT-DD This doesn't look like it is handled properly in the new deployers? Shouldn't this be handled in the metadata parsing? PLAYING AROUND WITH METADATA The merging and altering of metadata shouldn't be left to the "RealDeployer" i.e. AbstractWarDeployer. Doing so means it cannot be overridden by some other policy. What's most annoying is that bad examples like this are being used by the other people as the correct way to do things :-) View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4137137#... Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&a...