Return-Path: Delivered-To: apmail-commons-dev-archive@www.apache.org Received: (qmail 75638 invoked from network); 19 May 2008 15:29:44 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 19 May 2008 15:29:44 -0000 Received: (qmail 9600 invoked by uid 500); 19 May 2008 15:29:44 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 9533 invoked by uid 500); 19 May 2008 15:29:44 -0000 Mailing-List: contact dev-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Commons Developers List" Delivered-To: mailing list dev@commons.apache.org Received: (qmail 9522 invoked by uid 99); 19 May 2008 15:29:44 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 19 May 2008 08:29:44 -0700 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of sebbaz@gmail.com designates 74.125.46.153 as permitted sender) Received: from [74.125.46.153] (HELO yw-out-1718.google.com) (74.125.46.153) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 19 May 2008 15:28:58 +0000 Received: by yw-out-1718.google.com with SMTP id 9so1232024ywk.60 for ; Mon, 19 May 2008 08:29:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; bh=x0EwKrs9W8uWznHWC2llvOeUMZBblG4MQyAaap6muU8=; b=aSDEgJqUlyJXtfMM5pU0xGJTzRIYa1lvmN+1FGc6rGml71Jm3FEiFEVEuGI6LgBiaqxH4jPMgBvlMw0TSfACNVOJoMlj2DqIbjq02NcpgObOTg+KKY8x5fLOqnoG9nrnJqRMrtSvhrqyJbTXBUk/4PinmgaycQOpjg09xx/aPfc= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=PBehmyka1LuC2BPNDkO50++Aby4ZPznerkDkpP+3E4fqCkyVrl25q/LH1DP1N1U1BnMATFb5wlQ8MwUUXowlgbFmWRzxW8u+RTOb5ONQCsisELGfn4qb6aWCjuDhO2ATGWE8o1PBeR+L4dcxrQizl2LqHgskumEJ9xJK/Y4igLw= Received: by 10.150.12.3 with SMTP id 3mr6449115ybl.18.1211210951695; Mon, 19 May 2008 08:29:11 -0700 (PDT) Received: by 10.150.91.10 with HTTP; Mon, 19 May 2008 08:29:11 -0700 (PDT) Message-ID: <25aac9fc0805190829p38c47f28gf1c5609a4836ca3c@mail.gmail.com> Date: Mon, 19 May 2008 16:29:11 +0100 From: sebb To: "Commons Developers List" Subject: Re: Commons Net 1.5 / 2.0 Releases In-Reply-To: <8833dc800805190718p64a84db7g41b5961dca2ea034@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <460801A4097E3D4CA04CC64EE648584805B5157A@ism-mail03.corp.ad.wrs.com> <482F5555.6090109@eircom.net> <8833dc800805190718p64a84db7g41b5961dca2ea034@mail.gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org On 19/05/2008, Niklas Gustavsson wrote: > On Sat, May 17, 2008 at 11:59 PM, Rory Winston wrote: > > Are we ready to cut another RC? I can do so if you guys are happy with where > > we're at. > > > I still got this issue: > https://issues.apache.org/jira/browse/NET-217 > > I haven't look into any depth at why it's happening, but changing the > test to using unique ports for each of the tests (6900, 6901, 6902...) > make JUnit all happy. Should I commit this fix? With it, I would be > happy to see 2.0 released. I think I saw the same error, but only once, so I assumed it was something else happening on my system. If it's occurring regularly, then there may be a problem with releasing the port from previous tests in the same class. However, I have just seen a problem in the TFTPServer code - the constructors call launch() which creates a new thread and starts it. This is not thread-safe, as the class is not guaranteed to be fully constructed until after the constructor has completed. There are some other thread-safety problems too; I'll raise a JIRA. I don't know if it relates to the test failure or not - I suppose the unsafe constructor could mean that shutdown does not happen reliably. > > > /niklas > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > For additional commands, e-mail: dev-help@commons.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org For additional commands, e-mail: dev-help@commons.apache.org