commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Simone Tripodi <simonetrip...@apache.org>
Subject Re: [DIGESTER][SANDBOX] back proposing the Digester3 merge to trunk
Date Wed, 01 Jun 2011 14:25:24 GMT
Hi Rahul!
thanks for your kind feedback :) I'm going to plug the clirr plugin
and redeploy the site so we can have a better overview about what
changed.
The last point is indeed very important for users, the first attempt
broke that kind of compatibility that maybe was too rude :P
I'll let you know as soon as the new site is online!
Thanks!
Simo

http://people.apache.org/~simonetripodi/
http://www.99soft.org/



On Wed, Jun 1, 2011 at 4:15 PM, Rahul Akolkar <rahul.akolkar@gmail.com> wrote:
> On Wed, Jun 1, 2011 at 1:46 AM, Simone Tripodi <simonetripodi@apache.org> wrote:
>> Hi Rahul :)
>> thanks for following up!
>>
>> I can confirm they're NOT binary compatible even without running
>> clirr, because of the following reasons:
>>
> <snip/>
>
> We'll need to analyze with clirr anyway, minimally to produce good
> release notes and 2.x -> 3.0 migration notes.
>
> Not all breaks are equal, so in below:
>
>>  - deprecated APIs don't exist anymore;
> <snap/>
>
> Permissible in major release.
>
>>  - internals of annotations/xml modules have been rewritten as EDSL extensions;
>>  - some Digester methods changed signature, like Digester.pushParams(
>> Object object ) versus the new Digester.pushParams( Object... object )
> <snip/>
>
> Above may be OK.
>
>>  - due to all these changes, I took advantage to repackage classes to
>> org.apache.commons.digester3
>>
> <snap/>
>
> +1 to repackage.
>
>> Anyway I didn't break the pure Digester use as I did in the previous
>> attempt, users are still able to bin rules using the usual pattern
>>
>>    Digester d = new Digester();
>>    d.addObjectCreate("foo", "mypackage.Foo");
>>    ...
>>
> <snip/>
>
> This is useful IMO.
>
> -Rahul
>
>
>> I would really appreciate if you have spare time to have a look at the
>> code, I immagine you are quiet busy but there's no rush ;)
>> Have a nice day, all the best!!!
>> Simo
>>
>> http://people.apache.org/~simonetripodi/
>> http://www.99soft.org/
>>
>>
>>
>> On Wed, Jun 1, 2011 at 12:08 AM, Rahul Akolkar <rahul.akolkar@gmail.com> wrote:
>>> On Tue, May 31, 2011 at 5:46 PM, Simone Tripodi
>>> <simonetripodi@apache.org> wrote:
>>>> New site is online, does someone have some spare time to check[1] and
>>>> provide feedbacks about the merge proposal?
>>> <snip/>
>>>
>>> Don't have much time, but if its now compatible with 2.x thats a good
>>> thing :-) You may want to confirm using a clirr or equivalent report.
>>>
>>> -Rahul
>>>
>>>
>>>> Thanks in advance, have a nice day!!!
>>>> Simo
>>>>
>>>> [1] http://commons.apache.org/sandbox/digester3/
>>>>
>>>> http://people.apache.org/~simonetripodi/
>>>> http://www.99soft.org/
>>>>
>>>>
>>>>
>>>> On Tue, May 31, 2011 at 6:41 PM, Simone Tripodi
>>>> <simonetripodi@apache.org> wrote:
>>>>> Hi all guys,
>>>>> after some day of work on Sandbox, I'm back to propose once again the
>>>>> merge of my work in proper /trunk.
>>>>> Failing my first attempt was good, because I had the opportunity to
>>>>> learn a lot and this time the proposal is much better :P
>>>>>
>>>>> What I did:
>>>>>
>>>>>  - moved the current sandbox in a separate branch;
>>>>>  - copied the current trunk in sandbox;
>>>>>  - polished APIs (few trivial checkstyle violations yet), removed
>>>>> @Deprecated methods, added more power with generics;
>>>>>  - re-introduced - simplifying! - the Digester EDSL;
>>>>>  - removed old custom DigesterLoader for annotations/xml package.
>>>>>
>>>>> So, at the end of the day, this time users are still able to create
>>>>> Digester instances using the old-fashioned APIs - even if they're
>>>>> encouraged to use the more expressive fluent APIs - except when using
>>>>> annotations/XML extensions.
>>>>>
>>>>> I still have to publish the site - it's the site on /trunk + EDSL
>>>>> documentation - I'll do it as soon as I'll get home (I'm leaving the
>>>>> office now) so you can see how APIs/Doc look
>>>>>
>>>>> Anyway interested people can start having a look at the component code
on SVN[1]
>>>>>
>>>>> Please send huge feedbacks!!!
>>>>> Have a nice day!
>>>>> Simo
>>>>>
>>>>> [1] https://svn.apache.org/repos/asf/commons/sandbox/digester3/trunk
>>>>>
>>>>> http://people.apache.org/~simonetripodi/
>>>>> http://www.99soft.org/
>>>>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>

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


Mime
View raw message