Trustin Lee wrote:
Hi guys,
On 9/3/06, Alex Karasulu <[EMAIL PROTECTED]> wrote:
Emmanuel Lecharny wrote:
> Alex Karasulu a écrit :
>
>> Hi all,
>>
>> As you know, we have dependency issues with ApacheDS and MINA which
>> need to be resolved. There are several ways in which we can resolve
>> the problem. However before I propose some solutions let me restate
>> the problem again. Here's some dependencies which exist today:
>>
>> ApacheDS
>> -> MINA filter-codec-asn1 -> shared-asn1
>> -> shared-asn1
>>
>> ApacheDS depends on shared-asn1 through two paths as can be seen
>> above. One is a direct dependency and the other is via the
>> filter-codec-asn1 module in MINA. This makes it so the version of
>> shared-asn1 that ApacheDS depends on directly must be the same as the
>> version of shared-asn1 that is depended upon by the filter-codec-asn1
>> module.
>>
>> There are several ways in which we can solve this problem but the key
>> is to be able to release this filter-codec-asn1 separately whenever
>> changes are made to the shared ASN.1 module. We need to figure out
>> how we can release parts of MINA separately to prevent having to push
>> a full MINA release every time.
>
> In my mind, the filter-ASN.1 should not be a part of MINA. MINA has
> nothing to do with ASN.1, as it doesn't know how to handle PDUs.
>
> We should implement a MINA filter into ADS, in order to get rid of this
> dependency.
Exactly! I fully agree.
+1 here, too. I think filter-codec-asn1 should move as a subproject of
shared-asn1 as an integration layer between shared-asn1 and MINA. I
initially thought it will evolve into full-featured ASN.1 codec, but it's
still just an LDAP BER codec for now.
What do you think about just moving filter-codec-asn1 right now and release
ApacheDS before the vote for MINA 0.9.5 ends? I think it's an easier way
and should be done as soon as possible.
Ooops we just deployed the shared stuff. We will have to do this for
the next immediate release after RC4 which will be 1.0. I think we can
do this in a couple weeks. I'll move it over immediately though after
this release.
That cool?
Thanks,
Alex
