Mailing-List: contact commons-user-help@jakarta.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Jakarta Commons Users List" <commons-user@jakarta.apache.org>
Received-SPF: pass (asf.osuosl.org: domain of dion.gillard@gmail.com
 designates 66.249.82.207 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=W1JhQjyFOp85cLKYzVpvJ1yxcOGbkwU6rzZH/U2wPIABRkuPdlo5wiOm4B0P84epVRBEQKBUyMwmsHvlNpJQNVADD7+QMAMcDLAqQFUTOkh0Ps2n/KfQ+8Mo8vuN9bkaPPyP6CpWWqFnu+HaYGOaFVto89xdGqPBDdhj1P/pBFk=
Message-ID: <b22dff980602151356h523b67abo547f0b0a92765384@mail.gmail.com>
Date: Thu, 16 Feb 2006 08:56:12 +1100
From: Dion Gillard <dion.gillard@gmail.com>
To: Piero Ottuzzi <piero.ottuzzi@omnys.it>,
	Jakarta Commons Developers List <commons-dev@jakarta.apache.org>,
	Jakarta Commons Users List <commons-user@jakarta.apache.org>
Subject: Re: commons-email needs care
In-Reply-To: <ejpnn76y.yxudkioaszd0@a19yoxc6>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline
References: <ejpnn76y.yxudkioaszd0@a19yoxc6>

Tested and applied.

On 2/16/06, Piero Ottuzzi <piero.ottuzzi@omnys.it> wrote:
> Hi Dion, all,
>
> I'm still alive ;)
> Attached you can find my reworked patch to fix many problems you can have=
 if using commons-email with a MailSession mainly caused by typos in consta=
nt declaration or missing constant declaration.
> It fixes bugs 38538 and 38656 (just opened to hold the patch): please hav=
e a look and let me know.
> Probably tomorrow I will test what is in the repository.
>
> Thanks for your attention
> Bye
> Piero
>
>
> > Ping?
> >
> > On 2/12/06, Dion Gillard <dion.gillard@gmail.com> wrote:
> > > Hi Piero,
> > >
> > > some comments inline.
> > >
> > > On 2/9/06, Piero Ottuzzi <piero.ottuzzi@omnys.it> wrote:
> > > > Hi Hen,
> > > >
> > > > many thanks for your answer.
> > > > Here is a long mail on commons-email status :)
> > > >
> > > > SVN Repository
> > > > The activity on SVN repository (speaking of the src/java dir) is st=
alled about
> > > > 5 month ago. What you can find in SVN repository compiles fine and =
looks like
> > > > what has been released as 'Commons Email 1.0' on 2005-09-27.
> > > >
> > > > Bugs
> > > > As you can see in [1] there are 11 bug open
> > > > [37344] This bug provide a patch (in reality it provides a whole ne=
w class).
> > > > I'm using this provided class with my patched version of commons-em=
ail and it
> > > > is working fine without any issue so far.
> > >
> > > The issue here is the way the patch has been implemented.
> > > I'd be happier to see this implemented as a fix to HtmlEmail, rather
> > > than a new class
> > >
> > > > [37178] Can be resolved using the new class provided in previous bu=
g.
> > > See above.
> > >
> > > > [38538] This is fixed in my own attached patch.
> > >
> > > Unfortunately your patch does more than fix 38538. It adds new
> > > functionality as well, and this makes it hard to apply. e.g. there is
> > > code there to set the Mail Session from JNDI, which is a nice to have
> > > feature and means we now need to include the jndi dependency. It woul=
d
> > > be nicer to split these into separate enhancement requests as well as
> > > the bug fix.
> > >
> > > > [37902] Trivial ;)
> > > Sounds reasonable. Fixed.
> > >
> > > > [37783] Includes a patch. I'm using this provided patch with my pat=
ched
> > > > version of commons-email and it is working fine without any issue s=
o far.
> > > Looks reasonable. Will apply and test. Done.
> > >
> > > > [37782] Includes a patch which forces a new dependency on javax.nam=
ing.* I'm
> > > > using this provided patch with my patched version of commons-email =
and it is
> > > > working fine without any issue so far.
> > >
> > > it also includes the TLS authentication helpers. I'm committing this =
one as:
> > > a) The TLS helpers are useful
> > > b) jndi is in jdk1.3 and above
> > >
> > > > [36856] Includes patches I cannot test and I do not need
> > > Looks like it needs some testing.
> > >
> > > > [36844] Includes patches I did not test
> > > Introduces a dependency on commons-lang, something I'd rather not do.
> > >
> > > > [37363] Don't have a clue on this one.
> > > That one needs some more working out by the looks.
> > >
> > > > Patch
> > > > Attached you can find a SVN patch that fixes many problems you can =
see if
> > > > using a MailSession that needs authentication. It includes also the=
 patch
> > > > from bug [37782]. I'm using this provided patch with my patched ver=
sion of
> > > > commons-email and it is working fine without any issue so far.
> > >
> > > let me know if my recent commits are working for you.
> > >
> > > --
> > > http://www.multitask.com.au/people/dion/
> > > "If I close my eyes it doesn't seem so dark." - SpongeBob SquarePants
> > >
> >
> >
> > --
> > http://www.multitask.com.au/people/dion/
> > Chuck Norris sleeps with a night light. Not because Chuck Norris is
> > afraid of the dark, but because the dark is afraid of Chuck Norris
>
>
>
>
>


--
http://www.multitask.com.au/people/dion/
Chuck Norris sleeps with a night light. Not because Chuck Norris is
afraid of the dark, but because the dark is afraid of Chuck Norris

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-user-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-user-help@jakarta.apache.org