Christopher Oliver wrote:

This seems to have broken the build:

compile-core:
Copying 40 files to C:\cocoon5\build\cocoon-2.1.4-dev\classes
Copied 70 empty directories to 38 empty directories under C:\cocoon5\build\cocoo
n-2.1.4-dev\classes
Created dir: C:\cocoon5\build\cocoon-2.1.4-dev\mocks
Compiling 1 source file to C:\cocoon5\build\cocoon-2.1.4-dev\mocks
Compiling 559 source files to C:\cocoon5\build\cocoon-2.1.4-dev\classes
C:\cocoon5\src\java\org\apache\cocoon\environment\ForwardRedirector.java:138: ca
nnot resolve symbol
symbol : variable COCOON_REDIRECT_ATTR
location: class org.apache.cocoon.components.treeprocessor.TreeProcessor
this.env.setAttribute(TreeProcessor.COCOON_REDIRECT_ATTR, uri);
^


Oops, fixed. Sorry.


Also wouldn't it make more sense to have ForwardRedirector define this constant rather than TreeProcessor so that ForwardRedirector need not have a dependency on TreeProcessor?


Actually, this constant is now totally useless. I removed it and changed ForwardRedirector to an abstract class, and TreeProcessor provides a concrete implementation.

Sylvain

--
Sylvain Wallez                                  Anyware Technologies
http://www.apache.org/~sylvain           http://www.anyware-tech.com
{ XML, Java, Cocoon, OpenSource }*{ Training, Consulting, Projects }
Orixo, the opensource XML business alliance  -  http://www.orixo.com




Reply via email to