public inbox for [email protected]
help / color / mirror / Atom feedSubject: [Pljava-dev] Compilation with both JDK 5 and JDK 6
Date: Wed, 16 Jun 2010 13:54:45 -0700
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
On 6/16/2010 1:35 PM, Muhammad Mansoor wrote:
> Hi,
>
> The attached patch takes care of the following issues.
>
> 1. It adds PostgreSQL 9.0 support.
The code in CVS already supports postgresql 9.0. You need to create
your patch from CVS, not from the 1.4.0 source release.
> 2. It has the JDK support patch ( taken from /Petr Mich?lek original
> patch/ )
Your patch has all the problems his does, tons of gratuitous whitespace
changes. You should not be reformatting/re-indenting unrelated code.
> 3. It enables PLjava to be compiled with both JDK 5 and JDK 6.
It does not appear to do this. A quick search through the patch doesn't
reveal any matches for JDBC4 or USE_JDK6.
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