Tim,
While working on the patch last night, I found that the current
tools has a built-in dependency on the 1.3 velocity jar. This
version apparently does NOT accept the newer key for the
velocity configuration file (I could only get it to read
the config file using the old name). You may want to leave
the example web.xml in the older form and make a note that
the key is changing.
I have the current cvs version of velocity, and when I try to use
that with the tools, I get a NullPointerException. I'm still
trying to work out if I caused that with my proposed patch,
or if it was already a problem.
Perhaps someone else could comment on the relation between Tools
and the current Velocity source tree?
--- Tim Colson <[EMAIL PROTECTED]> wrote:
> Hey there Gabe!
>
> I was getting worried that you'd left us for FreeMarker. <grin>
>
> > Finally I found some time to look at your documentation. It
> > looks great! Thanks a lot for your contribution!
> Thanks for the kind words. I'm happy to finally be able to bring
> something to the table, however small.
>
> > - You changed the name of the VelocityStruts subproject to
> StrutsView.
> > I actually prefer the original name. I feel Struts view is too
> generic
> > and the connection to Velocity is lost.
> Alrighty, I'll concede on this since both Nathan and you have the
> same
> opinion, and have also noted that the list majority agrees with that
> too.
>
> I'll fix it and change the header image too.
>
> > - The documentation of the VelocityLayoutServlet is very hidden at
> the
> > moment. Could this be made a little bit easier to find?
> Agreed. I'll pull this up into the top-level tools index.
>
> > - Regarding the section on Model 1 vs. Model 2 in the
> VelocityStruts
> > documentation: I would make this section part of the
> VelocityStruts
> > User Guide.
> Ok. The rationale you gave makes sense. I'll move it there.
>
>
> > aware of the fact that Velocity does not support model 1
> Heh heh..well, the VelocityViewServlet does make it possible, eh?
>
> Could build a Model 1 webapp using only *.vm files and a few tools
> added
> into the context via the toolbox. Example, a "SQL" tool, #set
> ($results
> = $SQL.select("foo, bar from tableFoo") ) )
>
>
> > his application was a mix of model 1 and model 2 design.
> I feel your pain. I'm now working on a similar project.
>
> > - Download: There is now a nightly snapshot available for
> > download. This should be mentioned in the doownload sections
> everywhere.
> Ok, where is that?
> I looked here without luck:
> http://jakarta.apache.org/builds/jakarta-velocity/
>
> > How do you want to proceed? I guess the easiest would be that you
> send
> > me all the documentation files (not diffs) and I check them
> > in from here.
> Hah. Nathan and Ted gave me grief for suggesting the same thing. :-)
> So either your being too nice, or that is honestly the way you'd
> prefer
> to do it.
>
> Either way, give me a chance to make the changes above first (and
> some
> fixes Tony and Nathan suggested.)
>
> Then I'll be happy to send in diffs, or a tarball. Also, the xdocs
> directories are published out on the crufty.happysearch.com server
> too.
>
>
> Whatever you prefer and is easiest for you - I'm flexible and happy
> to
> do whatever works.
>
> QUESTION: When we get the 'beta' docs up and available on
> jakarta.apache.org/velocity/xxxx so that we can point users at them
> to
> get more feedback? (both on the docs and presumably on the code,
> since
> the docs will I believe spur more developers to give it a try.)
>
> I don't want to put the cart before the horse, so if the answer to
> that
> question is, "Not until we release a 1.0" - I'd be ok with that.
>
> Cheers,
> Tim
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
__________________________________________________
Do you Yahoo!?
Yahoo! Shopping - Send Flowers for Valentine's Day
http://shopping.yahoo.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
