> > I am all for adding this, the one problem I see is that I think torque
> > is supposed to be independent of turbine and fulcrum (I need to solve a
> > dependency on fulcrum.intake, maybe there are others). A simple way
> > around this is to not declare that the generated classes implement
> > interfaces in other packages, and force the application programmer to
> > declare it in the empty class if they wish to use the tool. This idea
> > sucks though, so maybe there is a better idea. Another (and this does
> > not seem ideal either) is to create some commons repo of interfaces.
> > The ApplicationTool (pull), Retrievable (intake), and Recyclable (pool),
> > etc. do not share anything in common that I can see though, so not sure
> > why they should be ripped out of their associated packages.
>
> Lets make a jakarta-turbine-common
Sounds good. I'm a bit confused about the whole 2.x vs 3.x issue. Would this
turbine-commons be used in 2.x
versions of turbine too ?
> That way, we can resolve dependencies across projects for Turbine specific
> interfaces that wouldn't really fit well into the Jakarta commons area.
>
> > If this can be solved, the code looks good. I would suggest storing the
> > ParameterParser in an attribute as it is the only thing in the RunData
> > that is used and this is made more clear and efficient by not
> > referencing the RunData. I also prefer the name getXXXFromParameters()
> > as I tend to think of Context as having to do with a TemplateContext and
> > you are pulling the info from request parameters.
+1 on all of these. I'll make the changes.
Do I just send my stuff to this list ? It's just three templates and some small
changes in build.properties
and build.xml (TDK application files). Again, I'm not sure whether I should diff
against
jakarta-turbine-torque or against jakarta-turbine-2 and jakarta-turbine-tdk...
Age
> +1
>
> -jon
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
