On 15 July 2012 20:54, Mark Thomas <ma...@apache.org> wrote: > On 15/07/2012 20:49, Filip Hanik Mailing Lists wrote: >> not sure if they want to make Java 7 minimum requirement
I'd forgotten that DBCP stands for the Don't Be Compatible Project ... > > That will need a discussion on the commons dev list. If no-one brings it > up sooner, I'll bring it up once Pool2 is sorted as work will naturally > move to dbcp2 at that point. > > Mark > >> >> ----- Original Message ----- >>> From: "sebb" <seb...@gmail.com> >>> To: "Tomcat Developers List" <dev@tomcat.apache.org> >>> Sent: Saturday, July 14, 2012 8:04:42 PM >>> Subject: Re: svn commit: r1360729 - in /tomcat/trunk: ./ >>> modules/jdbc-pool/src/test/java/org/apache/tomcat/jdbc/test/driver/ >>> res/dbcp/ >>> >>> Do these patches need to be fed back to Commons DBCP? >>> >>> Or do they only apply to the version embedded by Tomcat? >>> >>> On 14 July 2012 21:47, Rainer Jung <rainer.j...@kippdata.de> wrote: >>>> On 14.07.2012 22:25, Filip Hanik Mailing Lists wrote: >>>>> >>>>> I know, it's the same patch for DBCP as for DBCP2. >>>> >>>> >>>> Roger that. >>>> >>>> >>>>> we can fix it, not urgent though >>>> >>>> >>>> No hurry, maybe we'll be on DBCP2 until the first TC 8 release. >>>> >>>> Rainer >>>> >>>> >>>>> ----- Original Message ----- >>>>>> >>>>>> From: "Rainer Jung" <rainer.j...@kippdata.de> >>>>>> To: dev@tomcat.apache.org >>>>>> Sent: Friday, July 13, 2012 3:47:27 AM >>>>>> Subject: Re: svn commit: r1360729 - in /tomcat/trunk: ./ >>>>>> modules/jdbc-pool/src/test/java/org/apache/tomcat/jdbc/test/driver/ >>>>>> res/dbcp/ >>>>>> >>>>>> On 12.07.2012 17:38, fha...@apache.org wrote: >>>>>>> >>>>>>> Author: fhanik >>>>>>> Date: Thu Jul 12 15:38:28 2012 >>>>>>> New Revision: 1360729 >>>>>>> >>>>>>> URL: http://svn.apache.org/viewvc?rev=1360729&view=rev >>>>>>> Log: >>>>>>> Configure Tomcat trunk to build with Java 7. >>>>>>> This includes adding a patch to the Commons-DBCP code from >>>>>>> res/dbcp >>>>>>> >>>>>>> >>>>>>> Added: >>>>>>> tomcat/trunk/res/dbcp/ >>>>>>> tomcat/trunk/res/dbcp/dbcp-java-7.patch (with props) >>>>>> >>>>>> >>>>>> Just an info: when compiling TC trunk with Java 7 and ant 1.8.2 a >>>>>> few >>>>>> minutes ago, many of the offsets when applying the patch were >>>>>> quite >>>>>> big: >>>>>> >>>>>> [copy] Copying 68 files to >>>>>> /shared/build/dev/tomcat/deps/tomcat8-deps/tomcat8-deps/dbcp >>>>>> [patch] patching file >>>>>> src/java/org/apache/commons/dbcp/DelegatingCallableStatement.java >>>>>> [patch] Hunk #1 succeeded at 660 (offset -114 lines). >>>>>> [patch] patching file >>>>>> src/java/org/apache/commons/dbcp/cpdsadapter/DriverAdapterCPDS.java >>>>>> [patch] patching file >>>>>> src/java/org/apache/commons/dbcp/DelegatingResultSet.java >>>>>> [patch] Hunk #1 succeeded at 1078 (offset -196 lines). >>>>>> [patch] patching file >>>>>> src/java/org/apache/commons/dbcp/PoolingDataSource.java >>>>>> [patch] Hunk #1 succeeded at 437 (offset -52 lines). >>>>>> [patch] patching file >>>>>> src/java/org/apache/commons/dbcp/DelegatingConnection.java >>>>>> [patch] Hunk #1 succeeded at 678 (offset -126 lines). >>>>>> [patch] patching file >>>>>> src/java/org/apache/commons/dbcp/PoolingDriver.java >>>>>> [patch] Hunk #1 succeeded at 496 (offset -5 lines). >>>>>> [patch] patching file >>>>>> src/java/org/apache/commons/dbcp/DelegatingStatement.java >>>>>> [patch] Hunk #1 succeeded at 385 (offset -144 lines). >>>>>> [patch] patching file >>>>>> src/java/org/apache/commons/dbcp/DelegatingDatabaseMetaData.java >>>>>> [patch] Hunk #1 succeeded at 1206 (offset -171 lines). >>>>>> [patch] patching file >>>>>> src/java/org/apache/commons/dbcp/BasicDataSource.java >>>>>> [patch] Hunk #1 succeeded at 28 with fuzz 1. >>>>>> [patch] Hunk #2 succeeded at 1580 (offset -221 lines). >>>>>> [patch] patching file >>>>>> src/java/org/apache/commons/dbcp/datasources/InstanceKeyDataSource.java >>>>>> [patch] Hunk #1 succeeded at 887 (offset -1 lines). >>>>>> >>>>>> Compilation for everything including DBCP works though. >>>>>> >>>>>> Regards, >>>>>> >>>>>> Rainer >>>> >>>> >>>> --------------------------------------------------------------------- >>>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org >>>> For additional commands, e-mail: dev-h...@tomcat.apache.org >>>> >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org >>> For additional commands, e-mail: dev-h...@tomcat.apache.org >>> >>> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org >> For additional commands, e-mail: dev-h...@tomcat.apache.org >> > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org > For additional commands, e-mail: dev-h...@tomcat.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org