james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Burrell Donkin" <robertburrelldon...@gmail.com>
Subject Re: [mailet base] Move org.apache.james.util.mail.**.* to base? [WAS Re: [jsieve] Movin Sieve Mailet to JSieve]
Date Wed, 12 Nov 2008 20:17:28 GMT
On Wed, Nov 12, 2008 at 5:27 PM, Robert Burrell Donkin
<robertburrelldonkin@gmail.com> wrote:
> On Wed, Nov 12, 2008 at 9:23 AM, Stefano Bagnara <apache@bago.org> wrote:
>> Robert Burrell Donkin ha scritto:
>>> On Sun, Nov 9, 2008 at 9:53 PM, Robert Burrell Donkin
>>> <robertburrelldonkin@gmail.com> wrote:
>>>> On Sun, Nov 9, 2008 at 5:32 PM, Robert Burrell Donkin
>>>> <robertburrelldonkin@gmail.com> wrote:
>>>>> On Sun, Nov 9, 2008 at 4:09 PM, Stefano Bagnara <apache@bago.org>
wrote:
>>>>>> Robert Burrell Donkin ha scritto:
>>>> <snip>
>>>>
>>>>>>> i think that it should be possible to move the sieve mailet (and
the
>>>>>>> dependent mdn micro-library) from server into the jsieve project.
>>>>>>> AFAICT samples contains only a mailet example so i think this
can be
>>>>>>> removed. unless anyone can see anything wrong with this approach,
i'll
>>>>>>> probably just dive in sometime soonish...
>>>>>> Sounds good!
>>>>> done
>>>> not quite: i missed the dependency between the
>>>> org.apache.james.util.mail.mdn and
>>>> org.apache.james.util.mail.MimeMultipartReport. this is used by other
>>>> mailets so it's not really possible to move it out into sieve.
>>>>
>>>> i suppose that one possibility might be to move some or all of the
>>>> stuff in javamail-util into mailet-base
>>>
>>> the stuff currently in javamail utils is packaged
>>> org.apache.james.util but is mostly used by mailets and is (mostly)
>>> not strongly coupled to the rest of the server. moving most of the
>>> contents to mailet-base (and the rest to mailet-function module in the
>>> server) would allow this module to eliminated and allow more mailets
>>> (including sieve) to be factored out of the server code base.
>>>
>>> any objections?
>>
>> Most code can be moved but SpamAssassinInvoker.java seems a bit off
>> topic in mailet-base.
>
> yes : i think that belongs with the spam assassin mailet

the class is shared with the spam assassin fail fast plugin so it'll
have to stay where it is for now

- robert

---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Mime
View raw message