public inbox for [email protected]  

help / color / mirror / Atom feed

Subject: [Pljava-dev] Compilation with both JDK 5 and JDK 6
Date: Wed, 16 Jun 2010 14:23:20 -0700
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<[email protected]>

On 6/16/2010 2:06 PM, Muhammad Mansoor wrote:
> On Wed, Jun 16, 2010 at 4:54 PM, Kris Jurka <books at ejurka.com
> <mailto:books at ejurka.com>> wrote:
>
>     It does not appear to do this.  A quick search through the patch
>     doesn't reveal any matches for JDBC4 or USE_JDK6.
>
>
> These changes are there in the start of patch.

Sorry, I was looking at the wrong patch.  I was looking at a version of 
Petr's original patch.  So my comments about whitespace changes are also 
incorrect.

>
> + output        := $(shell rm -rf
> "src/java/pljava/org/postgresql/pljava/jdbc")
> +
> + ifdef USE_JDK6
> +       output  := $(shell cp -rf
> "src/java/pljava/org/postgresql/pljava/jdbc4"
> "src/java/pljava/org/postgresql/pljava/jdbc")
> + else
> +       output  := $(shell cp -rf
> "src/java/pljava/org/postgresql/pljava/jdbc3"
> "src/java/pljava/org/postgresql/pljava/jdbc")
> + endif
> +
>
> I will make it more cleaner. Do you agree with the solution?
>

No, keeping two complete copies of slightly different code is not the 
way to go.  If you want to make pljava build against either JDK version 
you need to provide JDBC4 classes extending JDBC3 classes like the 
regular JDBC driver does so that only new JDBC4 code goes into the JDBC4 
class, not a complete copy of the JDBC3 code.  That's a significant 
amount of work to do though and based on other comments on this list, 
perhaps the best thing to do is to simply require JDK6 from here on out.

Kris Jurka

view thread (5+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Subject: Re: [Pljava-dev] Compilation with both JDK 5 and JDK 6
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox