Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-24 Thread Simone Tripodi
Just committed the updated version, I hope I didn't break anything... I'm following the ML to see Gump messages... Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Tue, Aug 24, 2010 at 6:16 PM, Rahul Akolkar wrote: > On Tue, Aug 24, 2010 at 8:40 AM, Simone Tripodi > wro

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-24 Thread Rahul Akolkar
On Tue, Aug 24, 2010 at 8:40 AM, Simone Tripodi wrote: > Hi Sebb, > thanks for your help! I'm not a commons committer and haven't checked > yet if my apache id is enabled to login into Gump (but I don't think > it does) Any Apache committer can edit Gump metadata, search here [1] for digester.

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-24 Thread Simone Tripodi
Hi Sebb, thanks for your help! I'm not a commons committer and haven't checked yet if my apache id is enabled to login into Gump (but I don't think it does) Have a nice day, Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Tue, Aug 24, 2010 at 1:03 PM, sebb wrote: > On 2

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-24 Thread sebb
On 24 August 2010 08:12, Simone Tripodi wrote: > I had a look at the ant build that Gump executes, the 'test' target > doesn't ignore abstract test case, adding the following lines: > >           >           > > In the 'test' taget fixes the broken build. Hope this helps! Since the non-Gump build

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-24 Thread Simone Tripodi
I had a look at the ant build that Gump executes, the 'test' target doesn't ignore abstract test case, adding the following lines: In the 'test' taget fixes the broken build. Hope this helps! Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Tue, Au

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-23 Thread Simone Tripodi
Hi Rahul, sorry for the silly question, but reading the following message: [...] org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.main(JUnitTestRunner.java:906) [junit] at org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.main(JUnitTestRunner.java:906) [junit] Ru

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-23 Thread Simone Tripodi
ouch!!! :) I'm looking into it NOW! Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Mon, Aug 23, 2010 at 10:53 PM, Rahul Akolkar wrote: > On Mon, Aug 23, 2010 at 1:38 AM, Simone Tripodi > wrote: >> Hi Rahul, >> just checked out the digester /trunk and run tests, everyt

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-23 Thread Rahul Akolkar
On Mon, Aug 23, 2010 at 1:38 AM, Simone Tripodi wrote: > Hi Rahul, > just checked out the digester /trunk and run tests, everything worked > fine. All new stuff are contained in the current development code. > I really appreciate you fixed the inglorious compiler problem, there > are no words to s

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-22 Thread Simone Tripodi
Hi Rahul, just checked out the digester /trunk and run tests, everything worked fine. All new stuff are contained in the current development code. I really appreciate you fixed the inglorious compiler problem, there are no words to say thank you for saving me a lot of work :P At that point I'd cons

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-22 Thread Rahul Akolkar
On Sun, Aug 22, 2010 at 11:26 AM, Simone Tripodi wrote: > Thanks for your help, I'll install a Win machine and fix it! I had a chance to look at the code. I've checked in a fix to humor my compiler, and merged the at-digester functionality into the digester trunk. Please take a look at the dige

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-22 Thread Simone Tripodi
Thanks for your help, I'll install a Win machine and fix it! Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Sun, Aug 22, 2010 at 4:18 PM, Rahul Akolkar wrote: > On Sun, Aug 22, 2010 at 8:17 AM, Simone Tripodi > wrote: >> Hi!!! >> really wired, we're using the same mav

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-22 Thread Rahul Akolkar
On Sun, Aug 22, 2010 at 8:17 AM, Simone Tripodi wrote: > Hi!!! > really wired, we're using the same maven version, the main differences > I notice are > - OS (win xp Vs mac os x) > - platform encoding > - Java version (1.5.0_17 Vs 1.5.0_24) > > I think the first 2 points could be ignored, I don't

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-22 Thread Simone Tripodi
Hi!!! really wired, we're using the same maven version, the main differences I notice are - OS (win xp Vs mac os x) - platform encoding - Java version (1.5.0_17 Vs 1.5.0_24) I think the first 2 points could be ignored, I don't know if the java version can imply something... I don't understand why

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-21 Thread Rahul Akolkar
On Sat, Aug 21, 2010 at 4:05 PM, Simone Tripodi wrote: > Just installed the Java5, ran 'mvn test' and worked... , follows below > my terminal output (omitted all the tests)... can you tell me please > what goes wrong? Ofcourse, see details below. -Rahul java -version java version "1.5.0_17"

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-21 Thread Simone Tripodi
Just installed the Java5, ran 'mvn test' and worked... , follows below my terminal output (omitted all the tests)... can you tell me please what goes wrong? I'm testing it with the JavaForMacOSX10.5Update7 package. Thanks in advance!!! Simo Simones-Computer-2:commons-digester simone$ echo $JAVA_HO

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-21 Thread Simone Tripodi
Hi Rahul, thanks for your feedbacks!!! Your understanding in a) can be confirmed, the original digester codebase has not been modified at all, just added the 'annotations' package. Apologizes for b) can you tell me please what's wrong? Unfortunately right now on my mac I only have the default Java6

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-21 Thread Rahul Akolkar
On Tue, Aug 17, 2010 at 1:10 PM, Simone Tripodi wrote: > Thanks a lot! :) Feel free to contact me whenever you need for clarifications. I just took a quick look and have two comments: (a) Being in sync with trunk - There are a large number of small differences between the trunk and the sandbox

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-17 Thread Simone Tripodi
Thanks a lot! :) Feel free to contact me whenever you need for clarifications. All the best, Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Tue, Aug 17, 2010 at 5:47 PM, Rahul Akolkar wrote: > On Tue, Aug 17, 2010 at 4:15 AM, Simone Tripodi > wrote: >> Hi Rahul, >> th

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-17 Thread Rahul Akolkar
On Tue, Aug 17, 2010 at 4:15 AM, Simone Tripodi wrote: > Hi Rahul, > this mail just as a reminder since you told me there was no problem at > all on ping you :) Can we plan in a short-therm a review on this? Yes, thats no problem at all, sometimes even necessary. I will try to get to this within

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-08-17 Thread Simone Tripodi
Hi Rahul, this mail just as a reminder since you told me there was no problem at all on ping you :) Can we plan in a short-therm a review on this? Thanks in advance, have a nice day, Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Sat, Jul 31, 2010 at 5:59 PM, Simone Tri

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-07-31 Thread Simone Tripodi
ok thanks, please don't forget me :P ;) have anice conference, Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Sat, Jul 31, 2010 at 5:52 PM, Rahul Akolkar wrote: > On Thu, Jul 29, 2010 at 10:49 AM, Simone Tripodi > wrote: >> Hi all guys, >> any news on this? > > > No

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-07-31 Thread Rahul Akolkar
On Thu, Jul 29, 2010 at 10:49 AM, Simone Tripodi wrote: > Hi all guys, > any news on this? None from me, plus I am at a conference next week. It is still on my list, however! -Rahul > Simo > > http://people.apache.org/~simonetripodi/ > http://www.99soft.org/ > > > > On Fri, Jul 23, 2010 at 3:

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-07-29 Thread Simone Tripodi
Hi all guys, any news on this? Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Fri, Jul 23, 2010 at 3:53 PM, Simone Tripodi wrote: > Hi Rahul, > thanks a lot, very appreciated :) > Have a nice WE, greetings, > Simo > > http://people.apache.org/~simonetripodi/ > http://w

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-07-23 Thread Simone Tripodi
Hi Rahul, thanks a lot, very appreciated :) Have a nice WE, greetings, Simo http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Fri, Jul 23, 2010 at 4:43 AM, Rahul Akolkar wrote: > On Wed, Jul 21, 2010 at 4:01 PM, Simone Tripodi > wrote: >> Hi all guys, >> as discussed and agre

Re: [Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-07-22 Thread Rahul Akolkar
On Wed, Jul 21, 2010 at 4:01 PM, Simone Tripodi wrote: > Hi all guys, > as discussed and agreed with Rahul[1] last week, I added the missing - > hopefully, detailed - package documentation on the "annotations" > package on the sandbox digester[2] and merged latest trunk > modifications, re-execute

[Digester] Sandbox "annotations" package ready to be revisioned and merged to /trunk

2010-07-21 Thread Simone Tripodi
Hi all guys, as discussed and agreed with Rahul[1] last week, I added the missing - hopefully, detailed - package documentation on the "annotations" package on the sandbox digester[2] and merged latest trunk modifications, re-executed tests that still pass. Now I'd ask your help/feedbacks/suggestio