incubator-graffito-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Christophe Lombart <christophe.lomb...@gmail.com>
Subject Re: [PATCH] graffito-J2-2.0-dev svn HEAD deployment patches
Date Sun, 20 Nov 2005 22:52:50 GMT
Thank you very much Randy - That's great !
It runs with the J2 Head source and with mysql.

A couple of comments before commit your changes :
* Some unit tests failed in the subproject "components" :
AllFileTestRbmsPolicy and TestRdbmsPolivy. I will try to fix thoses
unit tests tomorrow.
* The subproject "engine" doesn't compile because the constructor for
SpringComponentManager in J2 head has been changed. This is not use
with J2. It can be use only when Graffito is running outside J2.
* The maven goal "allDeploy" deploy in the wrong folder. Maybe it is a
invalid config on my machine.
* I think we have to change the jetspeed version reference (in
etc/artifact-versions.ent). I set "2.0-dev" instead of "2.0-M3". Can
you confirm which J2 version to use (2.0-dev or 2.0-M4) ?

Nevertheless, it works if i skip the unit tests and deploy manually
:-) I'm wondering if you got those issues. Tommorow I will try to fix
them and commit your changes.

>From now, I will made a new J2 build every week in order to have a
stable integration for the Apache Conference.

Thanks,
Christophe

On 11/20/05, Randy Watler <watler@wispertel.net> wrote:
> All,
>
> Attached are my revised patches for graffito/J2 deployment against the
> current J2 svn HEAD. Here are a few notes:
>
> - as J2 commits move forward, watch the J2 src/webapp/WEB-INF/assembly
> directory... the graffito J2 deployment builds are very sensitive to
> changes there.
> - jetspeed2-deploy/src/assembly/jetspeed-spring.xml has been removed.
> -
> applications/browser/src/java/org/apache/portals/graffito/portlets/browser/actions/DocumentAction.java
has been patched to fix NPEs that occured on document insertion/save; I am not sure if that
is a new problem in either J2 or graffito.
>
> These patches have been tested against these versions:
>
> graffito:  345725
> J2:        345720
>
> HTH, YMMV :-),
>
> Randy
>
>
>
>

Mime
View raw message