forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Thorsten Scherler <thorsten.scherler....@juntadeandalucia.es>
Subject Re: Updating cocoon (was Re: [jira] Commented: (FOR-1083) switch to Cocoon 2.1.x)
Date Wed, 20 Aug 2008 07:12:35 GMT
On Wed, 2008-08-20 at 17:04 +1000, David Crossley wrote:
> Thorsten Scherler wrote:
> > David Crossley wrote:
> > > Thorsten Scherler wrote:
> > > > David Crossley (JIRA) wrote:
> > > > > 
> > > > > David Crossley commented on FOR-1083:
> > > > > -------------------------------------
> > > > > 
> > > > > Do the notes help that i made in FOR-955? It seems that i got further
than you have.
> > > > 
> > > > Hmm, not sure whether it is because of the cocoon you used in 955, but
> > > > with my current setup I actually always getting:
> > > > Unable to select source factory for 'sitemap.xmap'. No default factory
> > > > found.
> > > 
> > > Yesterday i started another attempt with the current
> > > Cocoon-2.1 branch. My setup is not ready yet. Probably
> > > won't have time today - other stuff to attend to.
> > > 
> > > In my previous attempt, i changed the XPathTransformer and
> > > IdGeneratorTransformer differently to you.
> > 
> > You write there:
> > 
> > Revert main/java/org/apache/forrest/util/XPathTransformer.java r111200
> > Revert main/java/org/apache/forrest/util/IdGeneratorTransformer.java
> > r111200
> > 
> > Do you mean 
> > svn up -r111200 main/java/org/apache/forrest/util/
> > 
> > which results in 
> > D    main/java/org/apache/forrest/util
> 
> No. I looked at the changes in r111200 for those two files
> and manually changed those lines to what it was prior to r111200.

http://svn.apache.org/viewvc/forrest/trunk/main/java/org/apache/forrest/util/IdGeneratorTransformer.java?r1=56266&r2=111200&diff_format=h

Index: java/org/apache/forrest/util/XPathTransformer.java
===================================================================
--- java/org/apache/forrest/util/XPathTransformer.java	(revision 687263)
+++ java/org/apache/forrest/util/XPathTransformer.java	(working copy)
@@ -21,6 +21,7 @@
 import java.util.Map;
 import java.util.Stack;
 
+import org.apache.avalon.framework.component.ComponentManager;
 import org.apache.avalon.framework.parameters.Parameters;
 import org.apache.avalon.framework.service.ServiceManager;
 import org.apache.avalon.framework.service.ServiceException;
@@ -104,8 +105,8 @@
         }
 
 
-    public void service(ServiceManager manager) throws ServiceException
{
-        super.service(manager);
+    public void compose(ComponentManager manager) {
+      super.compose(manager);
         try {


> 
> > > Also i edited cocoon.xconf to set the "Cocoon version" etc.
> > > That would probably explain your error.
> > 
> > -<cocoon version="2.2">
> > +<cocoon version="2.1">
> > 
> > > See my notes on FOR-955.
> > 
> > I am now a step further ahead and see another error (after adding the
> > webapp/WEB-INF/xconf/forrest-core.xconf directly into cocoon.xconf):
> > /home/thorsten/src/apache/forrest/trunk/main/webapp/symbols-core-v10.ent
> 
> Are you using a new 'forrest seed' initiated using that new
> FORREST_HOME etc. ?

I am testing directly in fresh-site.

> 
> > > > I will try to update our cocoon this week since I have some time this
> > > > week for the task.
> > > > 
> > > > I wonder whether it would be a good idea to create a branch for it so
it
> > > > is easier to debug in team.
> > > 
> > > I agree that we should do a branch so that a few of us
> > > can work on it. Didn't want to do that if it was just me,
> > > because the branch might be too long-lived - two years
> > > since my last attempt. Great to see renewed interest.
> > 
> > It would be good if you could add what you have into the branch this way
> > I can have a look.
> 
> When you have created the branch and added your Cocoon and
> other jar changes. I have a couple of hours remaining before
> going out to dinner and will have a look when i return.
 
Okay, but my setup is not working, that is why I think adding your
installation makes more sense.

Anyway will create a branch now.

salu2

> 
> -David
-- 
Thorsten Scherler                                 thorsten.at.apache.org
Open Source Java                      consulting, training and solutions


Mime
View raw message