[8u] RFR: 8059038: Create new launcher for SA tools

[Andrew Hughes]

On Wed, 19 Dec 2018 at 12:15, Severin Gehwolf <sgehwolf at redhat.com> wrote:
>
> On Wed, 2018-12-19 at 06:46 +0000, Andrew Hughes wrote:
> > On Thu, 13 Dec 2018 at 17:19, Severin Gehwolf <sgehwolf at redhat.com>
> > wrote:
> > >
> > > On Thu, 2018-12-13 at 15:49 +0000, Andrew Hughes wrote:
> > > > Ok, I presume that's not a change you're also going to make in
> > > > OpenJDK 12?
> > >
> > > Yes. I don't intend to change this in JDK 12. JDK 8 and JDK 9+ are
> > > not
> > > compatible in this regard. That's the case before and after this
> > > patch.
> > >
> > > > I'm fine with them being linked by the same bug ID, as they
> > > > resolve the same
> > > > underlying issue, but this really needs to be clear in the
> > > > summary
> > > > text, as, to the
> > > > uninformed, they look like completely different patches.
> > >
> > > OK will do. Can I consider this reviewed?
> > >
> > > Thanks,
> > > Severin
> > >
> >
> > I'd like to see an updated webrev with a more detailed commit message
> > first.
>
> Here you go:
> http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8059038/03/
>
> Thanks,
> Severin
>

I was hoping for a note on why it differs from 9 :-)
-- 
Andrew :)

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

Web Site: http://fuseyism.com
Twitter: https://twitter.com/gnu_andrew_java
PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222