Backport proposal of the Marlin renderer in OpenJDK8

[Andrew John Hughes]

On 08/08/2019 08:18, Laurent Bourgès wrote:

snip...

>>
> This class change must stay specific to jdk8u, as a different patch.
> 
> Finally I wonder if we should backport each individual jdk9, 10, 11, 14
> patches into jdk8u as a long train or you would accept 1 large patch
> gathering all patches.
> Of course, the latter approach seems simpler to me, but it requires a
> careful review to ascertain the code is the same with the current jdk
> repository (jdk 14).
> Once jdk8u & jdk14 have Marlin renderer 0.9.1.2, we will simply backport
> future changes.
> 

The individual changes should be backported. This assures that there is
a record that these bugs are fixed in 8u, which is important for
long-term maintainability.

I would suspect the original JDK9 patch would be an easier backport too,
given how much the build system has changed and the files have moved
around in JDK head.

A halfway option between the two would be to build up the required
changes in a staging repository and then ask for a bulk review before
integration into 8u. I believe this is how JFR is being backported.

Thanks,
-- 
Andrew :)

Senior Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222
https://keybase.io/gnu_andrew