Return-Path: Delivered-To: apmail-tomcat-dev-archive@www.apache.org Received: (qmail 93141 invoked from network); 3 Jul 2008 07:30:52 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 3 Jul 2008 07:30:52 -0000 Received: (qmail 17590 invoked by uid 500); 3 Jul 2008 07:30:46 -0000 Delivered-To: apmail-tomcat-dev-archive@tomcat.apache.org Received: (qmail 17527 invoked by uid 500); 3 Jul 2008 07:30:46 -0000 Mailing-List: contact dev-help@tomcat.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Tomcat Developers List" Delivered-To: mailing list dev@tomcat.apache.org Received: (qmail 17516 invoked by uid 99); 3 Jul 2008 07:30:46 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 03 Jul 2008 00:30:46 -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 henri.gomez@gmail.com designates 72.14.204.170 as permitted sender) Received: from [72.14.204.170] (HELO qb-out-1314.google.com) (72.14.204.170) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 03 Jul 2008 07:29:55 +0000 Received: by qb-out-1314.google.com with SMTP id e13so2265392qbe.12 for ; Thu, 03 Jul 2008 00:30:15 -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=W8APgb01iUmLHZoP6u6nPp7nsFbT7UWMdiEz/k1K71A=; b=EpJjypCsjK5cwgiVXrslqN4W0XzTazMKEVcL52+sjfVWMvS/WmtiMOvs97ptkSxfFD 1pl9crvR3WiTe9bDnCvt1wX3DfmF2YWS1qjoH5eNe3Y5qR4CXiClEd1A5CpGt4eAqQ4k RXyHYUJoIRPiWu95r+u6AsiUgZl0QO24aU+Hc= 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=D5BX6hgQt6p2jnfCez1tfOa4PIsb8XCSgpG/RugdFW0f0ZVyW5tVs/j6VItH/8Y7+u a/O/glD6j7DbwomwXRtgbXGu84WkTdCapf0k/hYH7UcQcrCR7oQ+YdtYuLyJr/g8BYFj mywazlnCe/bdjpBhth7807kM3mL8Lm7hyKtDY= Received: by 10.143.31.11 with SMTP id i11mr3435660wfj.328.1215070214297; Thu, 03 Jul 2008 00:30:14 -0700 (PDT) Received: by 10.142.246.14 with HTTP; Thu, 3 Jul 2008 00:30:14 -0700 (PDT) Message-ID: <6291fc850807030030r22445bc8j42446578b3ac411f@mail.gmail.com> Date: Thu, 3 Jul 2008 09:30:14 +0200 From: "Henri Gomez" To: "Tomcat Developers List" Subject: Re: [VOTE] Release tc-native 1.1.14 In-Reply-To: <486A9D1F.9030109@kippdata.de> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline References: <48622D5E.4010209@gmail.com> <486A9D1F.9030109@kippdata.de> X-Virus-Checked: Checked by ClamAV on apache.org Where could we found, when release, the 1.1.14 for windows ? Regards 2008/7/1 Rainer Jung : > Hi Jean-Frederic, > > jean-frederic clere schrieb: >> >> The candidates binaries are available here: >> http://people.apache.org/~jfclere/tcnative/v1.1.14/ >> >> According to the release process, the 1.1.14 tag is: >> [ ] Broken >> [ ] Alpha >> [ ] Beta >> [X] Stable > > Tested on Solaris 8. Library loads, threads show that it gets used, manag= er > status shows extended OS information. Nevertheless see some minor comment= s > below. I didn't test with OpenSSL or under high load though. > > On Windows I wasn't successful in building, because the build needs > apr_arch_misc.h, which is not included in the binary download, and buildi= ng > apr 1.3.2 was broken with errors in that file (apr_arch_misc.h). > > Thanks for doing the tcnative release. > > Detailed Comments > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > > Signature of Tarball is OK. > > Changelog looks a little strange, because it ends at 1_1.12 (or 1_1.13, b= ut > there's no headline for that version). > > README.txt: > > It says "This directory contains both the native and java-side code for > Tomcat Native Library.". But in fact there is only the jni directory > included. So also the other comments about ant and about test examples ar= e > not applicable. > > Instead of "To build the native part see native/BUILDING" maybe we should > use the path jni/native/BUILDING. > > File BUILDING: > > Before talking about buildconf, it might first list the usual procedure > (configure, make, make install). > > There is a block > > =3D=3D=3D > Build the jar containing the example by > cd .. > ant jar > Run the example: > ant example-basic > =3D=3D=3D > > which doesn't apply, because no build.xml and no examples and JavaFiles a= re > included in the tarball. > > The NOTE: "configure --without-ssl : Configure without ssl support." is > unclear, because there is also a --disable-openssl. > > configure: > > --enable-openssl and --disable-openssl both disable OpenSSl > > You should apply the following patch to configure.in: > > =3D=3D=3D > % diff -u configure.in.orig configure.in before the next release > --- configure.in.orig 2007-09-20 22:36:05.000000000 +0200 > +++ configure.in 2008-07-01 22:23:55.000000000 +0200 > @@ -141,10 +141,14 @@ > use_openssl=3Dtrue; > > AC_ARG_ENABLE(openssl, > -[ --disable-openssl avoid using OpenSSL toolkit], > +[AS_HELP_STRING([--disable-openssl],[avoid using OpenSSL toolkit])], > [ > - use_openssl=3Dfalse; > - AC_MSG_RESULT([Disabling SSL support...]) > + case "${enableval}" in > + no ) > + use_openssl=3Dfalse; > + AC_MSG_RESULT([Disabling SSL support...]) > + ;; > + esac > ]) > > if $use_openssl ; then > =3D=3D=3D > > The recreated configure then behaves as wanted (disables disables, enable > enables (default) and enable-openssl=3Dno disables). > > configure seems not to be in sync with configure.in. If I recreate it, on= e > message ch=E1nges: > > % diff configure.orig configure > 1257,1258c1257,1258 > < --with-apr=3DPATH prefix for installed APR, path to APR build > tree, > < or the full path to apr-config > --- >> --with-apr=3DPATH prefix for installed APR or the full path to >> apr-config > > > Compile warnings: > > src/poll.c: In function 'Java_org_apache_tomcat_jni_Poll_poll': > src/poll.c:284: warning: 'now' may be used uninitialized in this function > > src/ssl.c: In function 'ssl_rand_make': > src/ssl.c:364: warning: value computed is not used > > src/network.c: In function 'Java_org_apache_tomcat_jni_Socket_sendv': > src/network.c:667: warning: pointer targets in assignment differ in > signedness > src/network.c:673: warning: pointer targets in passing argument 3 of > '(*e)->ReleaseByteArrayElements' differ in signedness > src/network.c: In function 'Java_org_apache_tomcat_jni_Socket_sendfile': > src/network.c:1217: warning: pointer targets in assignment differ in > signedness > src/network.c:1222: warning: pointer targets in assignment differ in > signedness > src/network.c:1240: warning: pointer targets in passing argument 3 of > '(*e)->ReleaseByteArrayElements' differ in signedness > src/network.c:1244: warning: pointer targets in passing argument 3 of > '(*e)->ReleaseByteArrayElements' differ in signedness > > src/file.c: In function 'Java_org_apache_tomcat_jni_File_writev': > src/file.c:384: warning: pointer targets in assignment differ in signedne= ss > src/file.c:390: warning: pointer targets in passing argument 3 of > '(*e)->ReleaseByteArrayElements' differ in signedness > src/file.c: In function 'Java_org_apache_tomcat_jni_File_writevFull': > src/file.c:418: warning: pointer targets in assignment differ in signedne= ss > src/file.c:428: warning: pointer targets in passing argument 3 of > '(*e)->ReleaseByteArrayElements' differ in signedness > > Regards, > > Rainer > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org > For additional commands, e-mail: dev-help@tomcat.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org For additional commands, e-mail: dev-help@tomcat.apache.org