Thanks Niklas.
The backward compatibility question you mention was about the
suggestion to refactor the Commons Net "small protocol"
implementations into separate packages each:
> Basically there's no problem to deliver a
> commons-net-2.0-legacy.jar that contains something along
>
> package org.apache.commons.net;
> class Echo extends org.apache.commons.net.echo.Echo { ... };
>
> > I'm all in favor of code cleanup and refactoring when
> > I see clear advantages, but not at any price.
I was not in favor of such a costly, breaking change which
doesn't buy much, so I'd vote for going forward towards
a release. Can you put up a release candidate?
Cheers,
--
Martin Oberhuber, Senior Member of Technical Staff, Wind River
Target Management Project Lead, DSDP PMC Member
http://www.eclipse.org/dsdp/tm
> -----Original Message-----
> From: Niklas Gustavsson [mailto:[EMAIL PROTECTED]
> Sent: Friday, August 22, 2008 4:46 PM
> To: Commons Developers List
> Subject: Re: Commons Net 1.5 / 2.0 Releases
>
> On Tue, May 27, 2008 at 11:31 AM, Niklas Gustavsson
> <[EMAIL PROTECTED]> wrote:
> > On Thu, May 22, 2008 at 8:02 AM, Niklas Gustavsson
> <[EMAIL PROTECTED]> wrote:
> >> On Wed, May 21, 2008 at 11:27 PM, Rory Winston
> <[EMAIL PROTECTED]> wrote:
> >>> I meant to say "out of the project". It was moved to
> test/ after the
> >>> discussion.
> >>
> >> Great, so it seems we agree :-) However, the class is still in
> >> src/main, not in src/test:
> >>
> http://svn.apache.org/repos/asf/commons/proper/net/branches/NE
> T_2_0/src/main/java/org/apache/commons/net/tftp/TFTPServer.java
> >>
> >> Should I move it to the test directory?
> >
> > If no one objects I'll go ahead and move it to src/test.
>
> Alright, better later than never I guess. As no one objected, I'll go
> ahead and commit that change.
>
> Is there anything else that would be needed before a 2.0 release?
> Previously in this thread there was some discussions around backward
> compatibility, but no conclusion seems to been reached. Do we need to
> discuss this further?
>
> /niklas
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
