DavideD/shrinkwrap — comment on commit dbf14768086c50181adc4bc248e4292bb79ca8c9
File:  line 0
By ALRubinger:

It's a neat idea and it works, but I think it tangles up concerns.

From a base perspective, ClassLoader knows nothing special about WEB-INF/classes. Then the Servlet spec came along and defined this as a place to hold loadable assets, and also said that they'd take preference over stuff found in ClassLoaders higher in the hierarchy. So the Web ClassLoaders were introduced separate from ClassLoader, a layer atop.

I bet we'd be best served to employ a similar approach, or at least something a layer above to tell the existing ShrinkWrapClassloader to look in the right place (ie. under WEB-INF/classes for a WAR). What we'd really need for SHRINKWRAP-369 is an integration test case in the offending integration point (ie. Weld EE 1.1 Embedded).