D'oh.. Sorry, the pom I checked was dated and I didn't remember that the issues with Velocity 1.5 had been worked out.
No real complaints with this. Just slightly worried about long term maintenance issues. Should a enhancement request to make Texen have the needed hooks be put in as well? So eventually we can avoid any potential version change issues. And maybe a note in the javadocs about re-factoring after the Texen issue is resolved. > -----Original Message----- > From: Thomas Fischer [mailto:[EMAIL PROTECTED] > Sent: Tuesday, October 02, 2007 6:19 AM > To: Apache Torque Developers List > Subject: RE: Render correct whitespace in generator output > > The code stems from Texen 1.0. The migration already took place. > > Thomas > > "Greg Monroe" <[EMAIL PROTECTED]> schrieb am 02.10.2007 00:26:54: > > > How does this effect any future migration to Velocity 1.5 > and/or the > > new Texen subproject? > > > > > http://mail-archives.apache.org/mod_mbox/db-torque-dev/200705.mbox/%3C > > f1 > > [EMAIL PROTECTED] > > > > > -----Original Message----- > > > From: Thomas Fischer [mailto:[EMAIL PROTECTED] > > > Sent: Monday, October 01, 2007 5:07 PM > > > To: [email protected] > > > Subject: Render correct whitespace in generator output > > > > > > Hi all, > > > > > > I intend to commit the new patch I have attached to Thoralf > > > Rickert's suggestion to render correct whitespace in the > om target: > > > https://issues.apache.org/jira/browse/TORQUE-71 > > > > > > The only thing I am not quite confident about is that we need to > > > copy&paste large portions (approx. 180 lines) of > TexenTask's execute > > > method to the TorqueDataModelTask, because TexenTask does not > > > provide appropriate extension hooks to change velocity's > > > ResourceLoaders. However, I did not manage to think of a solution > > > where this is not needed. > > > > > > The lack of confidence is NOT in the functionality of the > patch, it > > > is a matter of style whether we want to copy&paste code > from another > > > project. > > > > > > If anyone is not comfortable with the patch or has a > better idea to > > > provide the functionality, please object in the next few days. > > > > > > Thomas > > > > > > > -------------------------------------------------------------------- > > > - To unsubscribe, e-mail: [EMAIL PROTECTED] > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > DukeCE Privacy Statement: > > Please be advised that this e-mail and any files > transmitted with it > > are confidential communication or may otherwise be privileged or > > confidential and are intended solely for the individual or > entity to > > whom they are addressed. If you are not the intended > recipient you may > > not rely on the contents of this email or any attachments, > and we ask > > that you please not read, copy or retransmit this > communication, but > > reply to the sender and destroy the email, its contents, and all > > copies thereof immediately. Any unauthorized dissemination, > > distribution or copying of this communication is strictly > prohibited. > > > > > --------------------------------------------------------------------- > > 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] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
