My bad, the branches/NET_2_0 actually gives the following test failures:

Tests in error:
  testInitial(org.apache.commons.net.telnet.TelnetClientTest)
  testOptionNegotiation(org.apache.commons.net.telnet.TelnetClientTest)
  testOptionRenegotiation(org.apache.commons.net.telnet.TelnetClientTest)
  testNotification(org.apache.commons.net.telnet.TelnetClientTest)
  testDeleteOptionHandler(org.apache.commons.net.telnet.TelnetClientTest)
  testAYT(org.apache.commons.net.telnet.TelnetClientTest)
  testSpy(org.apache.commons.net.telnet.TelnetClientTest)
  testSetReaderThread(org.apache.commons.net.telnet.TelnetClientTest)
  testReadOnly(org.apache.commons.net.tftp.TFTPServerPathTest)
  testWriteOnly(org.apache.commons.net.tftp.TFTPServerPathTest)
  testWriteOutsideHome(org.apache.commons.net.tftp.TFTPServerPathTest)

The following patch fixes the TelnetClientTest failures:

Index: src/main/java/org/apache/commons/net/io/FromNetASCIIInputStream.java
===================================================================
--- src/main/java/org/apache/commons/net/io/FromNetASCIIInputStream.java        
(revision
645441)
+++ src/main/java/org/apache/commons/net/io/FromNetASCIIInputStream.java        
(working
copy)
@@ -42,6 +42,7 @@

     static {
         _lineSeparator = System.getProperty("line.separator");
+        _lineSeparatorBytes= _lineSeparator.getBytes();
         _noConversionRequired = _lineSeparator.equals("\r\n");
     }

Downloading the RC3 build that Rory published, I get the following
test failure, probably fixable by this patch:

  testInitial(org.apache.commons.net.telnet.TelnetClientTest)
  testOptionNegotiation(org.apache.commons.net.telnet.TelnetClientTest)
  testOptionRenegotiation(org.apache.commons.net.telnet.TelnetClientTest)
  testNotification(org.apache.commons.net.telnet.TelnetClientTest)
  testDeleteOptionHandler(org.apache.commons.net.telnet.TelnetClientTest)
  testAYT(org.apache.commons.net.telnet.TelnetClientTest)
  testSpy(org.apache.commons.net.telnet.TelnetClientTest)
  testSetReaderThread(org.apache.commons.net.telnet.TelnetClientTest)

I would love to have a 2.0 release out, but it should build without
any test failures.

/niklas


On Mon, Apr 7, 2008 at 11:58 PM, sebb <[EMAIL PROTECTED]> wrote:
> trunk (Java 1.3+)
>
>  and
>
>  branches/NET_2_0 (Java 1.5)
>
>  have the most recent fixes.
>
>
>
>  On 07/04/2008, Niklas Gustavsson <[EMAIL PROTECTED]> wrote:
>  > Hi
>  >
>  >  From where is this code built? I've tried building from some different
>  >  tags and branches, but always get test failures. Here's the summaries
>  >  and the relevant surefire reports are attached.
>  >
>  >  http://svn.apache.org/repos/asf/commons/proper/net/tags/NET_2_0_RC3/
>  >
>  >
>  >  Tests in error:
>  >   testInitial(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testOptionNegotiation(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testOptionRenegotiation(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testNotification(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testDeleteOptionHandler(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testAYT(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testSpy(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testSetReaderThread(org.apache.commons.net.telnet.TelnetClientTest)
>  >
>  >
>  > http://svn.apache.org/repos/asf/commons/proper/net/tags/NET_2_0_RC_3/
>  >
>  >
>  >  Tests in error:
>  >   testInitial(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testOptionNegotiation(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testOptionRenegotiation(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testNotification(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testDeleteOptionHandler(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testAYT(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testSpy(org.apache.commons.net.telnet.TelnetClientTest)
>  >   testSetReaderThread(org.apache.commons.net.telnet.TelnetClientTest)
>  >
>  >   testWriteOnly(org.apache.commons.net.tftp.TFTPServerPathTest)
>  >   testWriteOutsideHome(org.apache.commons.net.tftp.TFTPServerPathTest)
>  >
>  >
>  >  http://svn.apache.org/repos/asf/commons/proper/net/branches/NET_2_0/
>  >
>  >  Tests in error:
>  >   testWriteOnly(org.apache.commons.net.tftp.TFTPServerPathTest)
>  >   testWriteOutsideHome(org.apache.commons.net.tftp.TFTPServerPathTest)
>  >
>  >
>  >  /niklas
>  >
>  >
>  >
>  >  On Mon, Apr 7, 2008 at 1:12 AM, Rory Winston <[EMAIL PROTECTED]> wrote:
>  >  > Hi
>  >  >
>  >  >   This is a release vote for Commons Net 2.0 RC3. This is a
>  >  >  maintenance release. Thanks to everyone who contributed.
>  >  >
>  >  >   The changes are here:
>  >  >
>  >  >  
> http://people.apache.org/~rwinston/commons-net-2.0/site/changes-report.html
>  >  >
>  >  >   The RC deployment is here:
>  >  >   http://people.apache.org/~rwinston/commons-net-2.0/
>  >  >
>  >  >   Javadocs:
>  >  >  
> http://people.apache.org/~rwinston/commons-net-2.0/site/apidocs/index.html
>  >  >
>  >  >   have at it!
>  >  >   Cheers
>  >  >   Rory
>  >  >
>  >  >
>  >  >  ---------------------------------------------------------------------
>  >  >  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]
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to