public inbox for [email protected]  

help / color / mirror / Atom feed

From: Chapman Flack <[email protected]>
To: Kartik Ohri <[email protected]>
Cc: [email protected]
Subject: Re: Starting build-system work (Windows/Mac users please speak up)
Date: Thu, 21 May 2020 07:34:25 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAASLQ4N0waxE9JKXjcLD=oAYz8NdgVAjBQd7FWGkuCOghFy8ng@mail.gmail.com>
References: <[email protected]>
	<CAASLQ4N0waxE9JKXjcLD=oAYz8NdgVAjBQd7FWGkuCOghFy8ng@mail.gmail.com>

On 05/21/20 02:48, Kartik Ohri wrote:

> Should I add checks for a PostgreSQL version from each major version or
> just for one from the latest major version and the oldest version supported
> ? Like just test with 9.5 and 12.x .

Each major version in the range, as there are often changes from every
version to the next that require conditional compilation in PL/Java, and
it is important to make sure that no recent PL/Java change broke any of
those.

(It is exactly here that CI promises to make the release process for
PL/Java much less tedious and treacherous.)

A reasonable timeline might be to get both endpoints covered first,
and then fill in the middle.

> Same question with Java versions. Is
> testing with Java 9 and Java 14 (the latest java release) is enough ? Or
> each of java versions in between Java 9 and 14 should be tested as well.

Again, maybe endpoints first, and then middles. Without testing of
middles, the odd javac bug that appeared in 10 and 11 and was fixed
in 12 would have been a surprise. [1]

Regards,
-Chap


[1] https://github.com/tada/pljava/commit/c763cee

view thread (47+ 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]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Starting build-system work (Windows/Mac users please speak up)
  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