hi,

On Fri, 2004-10-01 at 08:24, Shapira, Yoav wrote:

> Hi,
> Hmm.  Email is a simple component.  If we try to make it more
> complicated, we'd end up with something like [configuration] should have
> had a simple, clean, fast, efficient 1.0 release ages ago but has been
> adding features, refactoring, etc.


I couldn't agree more


> 
> I understand the use case for combining velocity with email, and it's a
> good one, sending all kinds of fancy and flexible templated email
> messages.  But it it's done, it should be a plugin or contrib. module or
> whatever technique you want to apply such that the simple use case
> without velocity is supported without the user having to have velocity
> on the classpath.


I don't know about this.  The Velocity Email (which I have yet to submit
;-), is a single class that extends HtmlEmail.  

I will look at what chains did to only build it if velocity is on the
classpath, but I really don't like this idea.  I think it results in
fractioned builds and will cause more confusion in the long run.  

Still, adding this class to the email.jar doesn't affect the end user at
all.  They can still use the email classes and jar without having
velocity on the classpath.  However, if they choose to use the
VelocityEmail class they will obviously, need velocity.  


> 
> Yoav Shapira
> Millennium Research Informatics
> 
> 
> >-----Original Message-----
> >From: Martin van den Bemt [mailto:[EMAIL PROTECTED]
> >Sent: Friday, October 01, 2004 8:22 AM
> >To: Jakarta Commons Developers List
> >Subject: Re: [email] Please add new build to ibiblio
> >
> >Maybe this has a better home at velocity ?
> >
> >Mvgr,
> >Martin
> >
> >On Fri, 2004-10-01 at 01:58, Joe Germuska wrote:
> >> At 12:54 PM -0700 9/30/04, Martin Cooper wrote:
> >> >  > Hmmm..  Personally, I don't think putting it in a different
> package
> >is
> >> >>  overkill.  Adding velocity requires the addition of Velocity.jar
> to
> >> >>  build, but not to run (unless of course, you use the class ;-).
> >
> >
> >
> >---------------------------------------------------------------------
> >To unsubscribe, e-mail: [EMAIL PROTECTED]
> >For additional commands, e-mail: [EMAIL PROTECTED]
> 
> 
> 
> 
> This e-mail, including any attachments, is a confidential business communication, 
> and may contain information that is confidential, proprietary and/or privileged.  
> This e-mail is intended only for the individual(s) to whom it is addressed, and may 
> not be saved, copied, printed, disclosed or used by anyone else.  If you are not 
> the(an) intended recipient, please immediately delete this e-mail from your computer 
> system and notify the sender.  Thank you.
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to