Berin Loritsch wrote:
I have improved the framework build some, so that the announcements are
built again. The changes are also back in. I want to provide ample
time for anyone to bring up issues with the build. PLEASE, find any and
all issues and bring them up now--no matter how minor.
One major issue, one minor issue: 1. wrapper classes issue needs to be resolved 2. documentation update needed (ComponentManager to ServiceManager) Wrapper classes (major) ----------------------- The wrapper classes that that exist in framework are broken. The WrapperComponentManager takes a ServiceManager as a constructor argument and when lookup invocations are made against the wrapper, the wrapper returns Component instance "if and only if the object returned from a lookup on the service manager is an instance of Component". The wrapper makes no attempt to proxy an Object as a Component. The above problem resulted in failure when testing against the James platform. This problem was resolved by putting in place a working version under the assembly package that automatically generated a Component using the JDK 1.3 Proxy class. Refer: org.apache.avalon.assembly.lifecyycle.context.WrapperComponentManager I'm presuming that the Proxy from Object to Component was not included in the framework due to the JDK version dependency. I recommend that we move the wrapper classes out of framework and into an Excalibur utility package such as Excalibur/legacy. Documetation Updates (minor) ---------------------------- The framework documentation concerning COP uses several examples of how a container supplies resources to a component. These examples all use the deprecated CM and CS classes. I recommend we get that cleaned up by simply upgrading the documentation to the corresponding service package class and interface names. Cheers, Steve. -- Stephen J. McConnell mailto:[EMAIL PROTECTED] http://www.osm.net --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
