Thanks a lot Seb, much more than appreciated :) Going to try it now and re-uploading the report! All the best, Simo
http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Thu, Jun 2, 2011 at 1:25 PM, sebb <seb...@gmail.com> wrote: > On 2 June 2011 08:13, Simone Tripodi <simonetrip...@apache.org> wrote: >> Hi Rahul, >> I forgot to notify you the clirr report[1] is now online, >> unfortunately is not really useful due to repackaging :( > > See > > https://issues.apache.org/jira/browse/VFS-344 > > for one way to allow comparison across packages > >> I'll wait for more feedbacks from you before calling a vote for the >> merge - in the meanwhile I'll continue developing on Sandbox with the >> aim to prepare the 3.0 release. >> Have a nice day, all the best!!! >> Simo >> >> [1] http://commons.apache.org/sandbox/digester3/clirr-report.html >> >> http://people.apache.org/~simonetripodi/ >> http://www.99soft.org/ >> >> >> >> On Wed, Jun 1, 2011 at 4:25 PM, Simone Tripodi <simonetrip...@apache.org> >> wrote: >>> 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.akol...@gmail.com> >>> wrote: >>>> On Wed, Jun 1, 2011 at 1:46 AM, Simone Tripodi <simonetrip...@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.akol...@gmail.com> >>>>> wrote: >>>>>> On Tue, May 31, 2011 at 5:46 PM, Simone Tripodi >>>>>> <simonetrip...@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 >>>>>>> <simonetrip...@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-unsubscr...@commons.apache.org >>>> For additional commands, e-mail: dev-h...@commons.apache.org >>>> >>>> >>> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org >> For additional commands, e-mail: dev-h...@commons.apache.org >> >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
