[RFR] 8231171: remove remaining sun.java.launcher.pid references - was RE: sun.java.launcher.pid property usage

[Baesken, Matthias]

Thanks for the review !
Alan, may I add you as reviewer too ?

Best regards, Matthias

> -----Original Message-----
> From: David Holmes <david.holmes at oracle.com>
> Sent: Freitag, 20. September 2019 11:41
> To: Baesken, Matthias <matthias.baesken at sap.com>; 'hotspot-
> dev at openjdk.java.net' <hotspot-dev at openjdk.java.net>; core-libs-
> dev at openjdk.java.net
> Cc: Alan Bateman <Alan.Bateman at oracle.com>
> Subject: Re: [RFR] 8231171: remove remaining sun.java.launcher.pid
> references - was RE: sun.java.launcher.pid property usage
> 
> That looks fine to me.
> 
> Thanks,
> David
> 
> On 20/09/2019 7:14 pm, Baesken, Matthias wrote:
> > Hi David   , I adjusted the  test  (
> test/jdk/tools/launcher/TestSpecialArgs.java )   and removed the comments
> in os_bsd.cpp  (suggested by you) .
> > New webrev :
> >
> > http://cr.openjdk.java.net/~mbaesken/webrevs/8231171.1/
> >
> > Best regards, Matthias
> >
> >
> >
> >
> >> Hi Matthias,
> >>
> >> On 20/09/2019 5:03 pm, Baesken, Matthias wrote:
> >>>
> >>> Hello,  looks like   on Linux  there is a special check   in
> TestSpecialArgs.java
> >> for
> >>>
> >>>       launcherPidString = "launcher.pid="
> >>>
> >>> that  fails after   8231171  .
> >>> Should I adjust the test ?  Or  keep the  setting in the launcher on Linux ?
> >>
> >> IMHO adjust the test please.
> >>
> >> Thanks,
> >> David
> >> -----
> >>
> >