Return-Path: Delivered-To: apmail-xml-cocoon-dev-archive@xml.apache.org Received: (qmail 64104 invoked by uid 500); 13 Mar 2003 12:46:22 -0000 Mailing-List: contact cocoon-dev-help@xml.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: list-post: Reply-To: cocoon-dev@xml.apache.org Delivered-To: mailing list cocoon-dev@xml.apache.org Received: (qmail 64089 invoked from network); 13 Mar 2003 12:46:22 -0000 Received: from www2.kc.aoindustries.com (209.15.201.84) by daedalus.apache.org with SMTP; 13 Mar 2003 12:46:22 -0000 Received: from dialup-188.147.220.203.acc01-aubu-gou.comindico.com.au (dialup-188.147.220.203.acc01-aubu-gou.comindico.com.au [203.220.147.188]) (authenticated) by www2.kc.aoindustries.com (8.11.6/8.11.6) with ESMTP id h2DCkLV18090 for ; Thu, 13 Mar 2003 06:46:22 -0600 Subject: Re: Finishing Deprecation Package From: David Crossley To: cocoon-dev@xml.apache.org In-Reply-To: References: Content-Type: text/plain Content-Transfer-Encoding: 7bit X-Mailer: Ximian Evolution 1.0.8 (1.0.8-10) Date: 13 Mar 2003 23:46:17 +1100 Message-Id: <1047559581.1065.2866.camel@ighp> Mime-Version: 1.0 X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N On Tue, 2003-03-11 Carsten Ziegeler wrote: > As I will not do it this week, here is a list of > things that have to be done to get the deprecation > package working. If someone wants to do it, I will > be very happy - otherwise I will perhaps do it > next week. > > The XScriptObject must be changed to inherit from the > excalibur source instead of the cocoon source. It might > be, that a logicsheet needs to be adapted as well. > > The Source interface from Cocoon and the > CocoonToAvalonSource class have to be moved from the > deprecated package into the core package. > > org.apache.cocoon.components.resolver.ResolverImpl has > to be changed to use the new Resolver from the excalibur > xmlutil package. Whoa, not so fast Carsten. I see that you have suddenly ripped out the comprehensive ResolverImpl.java and replaced it with a basic DefaultResolver.java ... Why? What is the reason for these drastic changes and why was there no discussion? This class was not marked as deprecated. Nicola Ken had already cleverly moved the deprecated Resolver.java into src/deprecated/ I note that 'build docs' is now busted because it is not using the entity resolver anymore. This is demonstrated with the document xdocs/catalog-test.xdoc during 'build docs' and via installing/tests.html with the webapp. And if we did not have the belt-and-braces thing of copying all of the DTDs under xdocs, as well as the originals in WEB-INF, then documentation would all be broken. Why is the entity resolver now busted? I do not know, but ... The ResolverImpl.java in Cocoon uses the new version of resolver.jar from xml-commons and configures it appropriately. Is that jar compatible with the new DefaultResolver.java ? We have also lost the ability to configure the entity resolver via cocoon.xconf which was one purpose of the ResolverImpl.java I would prefer that these changes were rolled back until we talk about it. --David