not sure if they want to make Java 7 minimum requirement

----- 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

Reply via email to