Return-Path: Delivered-To: apmail-jakarta-commons-dev-archive@www.apache.org Received: (qmail 4237 invoked from network); 22 Mar 2006 20:46:15 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 22 Mar 2006 20:46:15 -0000 Received: (qmail 24122 invoked by uid 500); 22 Mar 2006 20:46:12 -0000 Delivered-To: apmail-jakarta-commons-dev-archive@jakarta.apache.org Received: (qmail 24022 invoked by uid 500); 22 Mar 2006 20:46:12 -0000 Mailing-List: contact commons-dev-help@jakarta.apache.org; run by ezmlm Precedence: bulk List-Unsubscribe: List-Help: List-Post: List-Id: "Jakarta Commons Developers List" Reply-To: "Jakarta Commons Developers List" Delivered-To: mailing list commons-dev@jakarta.apache.org Received: (qmail 24011 invoked by uid 99); 22 Mar 2006 20:46:12 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 22 Mar 2006 12:46:12 -0800 X-ASF-Spam-Status: No, hits=1.4 required=10.0 tests=DNS_FROM_RFC_POST,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (asf.osuosl.org: domain of robertburrelldonkin@blueyonder.co.uk designates 195.188.213.60 as permitted sender) Received: from [195.188.213.60] (HELO asmtp-out1.blueyonder.co.uk) (195.188.213.60) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 22 Mar 2006 12:46:11 -0800 Received: from [82.38.65.173] (helo=knossos.elmet) by asmtp-out1.blueyonder.co.uk with esmtpa (Exim 4.52) id 1FMADG-0007H2-40 for commons-dev@jakarta.apache.org; Wed, 22 Mar 2006 20:45:50 +0000 Subject: Re: [pool] Announcing Release Candidate 1 for Pool 1.3 From: robert burrell donkin To: Jakarta Commons Developers List In-Reply-To: <6bde122b0603212055w6639029ag96e93942cd0a72ea@mail.gmail.com> References: <6bde122b0603202250y469f381ao51345731ba420003@mail.gmail.com> <00a801c64d1d$407aae50$0300a8c0@DELL1800> <6bde122b0603212055w6639029ag96e93942cd0a72ea@mail.gmail.com> Content-Type: text/plain Date: Wed, 22 Mar 2006 20:51:43 +0000 Message-Id: <1143060703.5034.4.camel@knossos.elmet> Mime-Version: 1.0 X-Mailer: Evolution 2.0.1-1mdk Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N On Tue, 2006-03-21 at 23:55 -0500, Sandy McArthur wrote: > On 3/21/06, Niall Pemberton wrote: > > 4) The links on the changelog and file activity report are invalid. This is > > a bug in the maven plugins, which may be resolved in a later version. One > > way to get round this is to change to url in the repository element of > > project.xml to point to the root subversion directory, rather than commons > > pool (and since your using a custom "cvs-usage" report it shouldn't matter): > > > > > > http://svn.apache.org/viewcvs.cgi > > > > Setting the url like you suggested fixes the changelog links but > doesn't help the file activity report. Actually both reports are wrong > because they don't go back far enough. I think this is related to the > cvs to svn conversion. How about just removing those two reports until > the next release? there are doubts about the accuracy of these reports for dates before the date of the last import from CVS. so i'm happy to turn them off for now. > > 6) Theres a checkstyle config file in pool, but it doesn't appear to be used > > (and isn't included in the source distro). > That may be because when you include the checkstyle report it lists > about 5,000 style errors. Using the checkstyle.xml from IO brings that > number down to 510, almost all of which are JavaDoc related. When I > was working on the Pool 1.3 branch I mostly focused on bug fixes and > didn't try to touch more than was needed. The trunk for pool 2 has > more extensive clean ups. I suggest the checkstyle be left out until > the 2.0 release. +1 > > I ran the tests using maven with JDK 1.3.1_04 and JDK 1.4.2_05 on Windows > > XP and they passed OK. On W2K the tests passed with JDK 1.4, but failed for > > JDK 1.3 (TestGenericKeyedObjectPool failed on testEviction2). However my W2K > > m/c is a bit flakey so I would ignore this unless someone else verifies it. > This would be a result of the test expecting more determinist behavior > from the thread scheduler than the JVM guarantees. Odds are if you > re-run the tests they'll pass most of the time. I've made a good > effort to make the unit tests in the trunk all be deterministic while > I haven't put that effort into the 1.3 branch. a note to that effect might (in more diplomatic language) would make a good addition to the release notes. - robert --------------------------------------------------------------------- To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org For additional commands, e-mail: commons-dev-help@jakarta.apache.org