On 3/8/08, Rory Winston <[EMAIL PROTECTED]> wrote: > I've taken into account the suggestions proposed by various people wrt > the TFTP server classes and accordingly made some changes. Thanks for > the feedback. >
So, does that mean you're cutting (or have already done so) a new release candidate for us to vote on? > > Rory > > > sebb wrote: > > On 07/03/2008, Henri Yandell <[EMAIL PROTECTED]> wrote: > > > >> On Fri, Mar 7, 2008 at 8:04 AM, James Carman <[EMAIL PROTECTED]> wrote: > >> > On 3/2/08, Rory Winston <[EMAIL PROTECTED]> wrote: > >> > > Hi > >> > > > >> > > This is a release vote for Commons Net 1.5. This release fixes a > few > >> > > issues with TFTP errors, threading hangs, and a leap year date > parsing bug. > >> > > > >> > > The RC deployment is here: > >> > > http://people.apache.org/~rwinston/commons-net-1.5.0/site/ > >> > > > >> > > Binaries: > >> > > > >> > > > http://people.apache.org/~rwinston/commons-net-1.5.0/commons-net-1.5.0-SNAPSHOT-bin.zip > >> > > > >> > > Source: > >> > > > >> > > > http://people.apache.org/~rwinston/commons-net-1.5.0/commons-net-1.5.0-SNAPSHOT-src.tar.gz > >> > > > >> > > Javadocs: > >> > > > >> > > > http://people.apache.org/~rwinston/commons-net-1.5.0/site/apidocs/index.html > >> > > > >> > > SVN tag: > >> > > > >> > > http://svn.apache.org/viewvc/commons/proper/net/tags/NET_1_5_0/ > >> > > > >> > > Here's my +1 in advance. > >> > > Cheers > >> > > Rory > >> > > > >> > > >> > I just wanted to post my official -1 vote. I feel that the > >> > org.apache.commons.net.tftp.TFTPServer class is outside the scope of > >> > Commons Net. I have no problem with it being in the test classes, as > >> > suggested, though. > >> > >> > >> Not a problem for me. +1 to including the server class. > >> > >> It's not big codewise, and is not that far from Net's scope or Commons > scope. > >> > > > > But it's not in scope (yet?) > > > > > >> Two proviso's: > >> > >> 1) The main method in TFTPServer should be removed. > >> > > > > +1 > > > > > >> 2) If we get lots of embedded server components, then we should > >> rethink this, but I can see that the 'simple embedded server space' is > >> pretty small. Might want to use a different package to allow it to > >> more easily be split out. org.apache.commons.net.server.* or > >> something. Then people can send in their dumb EchoServer > >> implementations :) > >> > > > > +1 to changing package name. > > > > I still think it should be in the test tree; AFAICS the purpose of the > > code contribution was to enable the TFTP Client to be tested. > > > > If Commons starts getting other simple server implementations then > > these ought to be part of a different project with a scope of > > Testing/Mocking or some such. Maybe not even in Commons. > > > > > >> Hen > >> > >> > >> --------------------------------------------------------------------- > >> 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] > > > > > > > > > > > > > --------------------------------------------------------------------- > 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]
