commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mike Stanley <>
Subject RE: [email] Please add new build to ibiblio
Date Mon, 04 Oct 2004 12:46:45 GMT

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 []
> >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:
> >For additional commands, e-mail:
> 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:
> For additional commands, e-mail:

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message