Re: svn commit: r230781...

2005-08-09 Thread Stefano Bagnara
> Actually please try to separate changes. > Commit groups of files in one commit with one message for one change. > It makes making sense of things (and reading diffs) s > much easier... As I said to Noel: I will do the next times! Unfortunately I've accumulated a lot of patches for James an

Re: svn commit: r230781 - in /james/server/trunk/src/java/org/apache/james: mailrepository/MBoxMailRepository.java transport/mailets/CommandListservManager.java transport/mailets/RemoteDelivery.java t

2005-08-09 Thread Danny Angus
Actually please try to separate changes. Commit groups of files in one commit with one message for one change. It makes making sense of things (and reading diffs) s much easier... On 08/08/05, Noel J. Bergman <[EMAIL PROTECTED]> wrote: > Stefano, > > Please be more careful with commits or at

Re: RTC vs CTR (Was: svn commit: r230781)

2005-08-08 Thread Stefano Bagnara
> > I haven't asked wether James use a RTC (review-then-commit) > or a CTR > > (commit-then-review) approach. > > That depends. :-) > > Are we about to do a release? How risky is the patch? > Should we create a branch first, apply the patches, > review/test and then merge back to trunk? >

RE: RTC vs CTR (Was: svn commit: r230781)

2005-08-08 Thread Noel J. Bergman
> I haven't asked wether James use a RTC (review-then-commit) > or a CTR (commit-then-review) approach. That depends. :-) Are we about to do a release? How risky is the patch? Should we create a branch first, apply the patches, review/test and then merge back to trunk? Those are the sorts of q

RTC vs CTR (Was: svn commit: r230781)

2005-08-08 Thread Stefano Bagnara
Sure, I will add better comments! I haven't asked wether James use a RTC (review-then-commit) or a CTR (commit-then-review) approach. I'm ready to apply a cumulative patch to: DSNBounce should be configured by default http://issues.apache.org/jira/browse/JAMES-357 Upgrade to dnsjava 2.0.0 http:/

RE: svn commit: r230781 - in /james/server/trunk/src/java/org/apache/james: mailrepository/MBoxMailRepository.java transport/mailets/CommandListservManager.java transport/mailets/RemoteDelivery.java t

2005-08-08 Thread Noel J. Bergman
Stefano, Please be more careful with commits or at least your commit messages. These commits: http://svn.apache.org/viewcvs/james/server/trunk/src/java/org/apache/james/mailrepository/MBoxMailRepository.java?rev=230781&r1=230780&r2=230781&view=diff http://svn.apache.org/viewcvs/james/se

svn commit: r230781 - in /james/server/trunk/src/java/org/apache/james: mailrepository/MBoxMailRepository.java transport/mailets/CommandListservManager.java transport/mailets/RemoteDelivery.java trans

2005-08-08 Thread bago
Author: bago Date: Mon Aug 8 02:59:57 2005 New Revision: 230781 URL: http://svn.apache.org/viewcvs?rev=230781&view=rev Log: added Locale.US to toUpperCase and toLowerCase (JAMES-323) Modified: james/server/trunk/src/java/org/apache/james/mailrepository/MBoxMailRepository.java james/ser