Hi, I understand the effects of a whitelist. But the test is just intended to dig up tools that don't adhere to the CSR. It's easy enough to fix if you add a tool of yourself, as you can see in this change.
Thanks for testing webrev 03. Feel free to push it if it solves your problems in case it's night in Europe :) Best regards, Goetz. > -----Original Message----- > From: Kumar Srinivasan [mailto:[email protected]] > Sent: Freitag, 19. Januar 2018 14:32 > To: David Holmes <[email protected]>; Lindenmaier, Goetz > <[email protected]> > Cc: Weijun Wang <[email protected]>; core-libs- > [email protected] > Subject: Re: RFR(XS): 8195663: jdk/tools/launcher/HelpFlagsTest.java fails > with java.lang.AssertionError: HelpFlagsTest failed: > > Hello, > > >> Posting this to core-libs-dev. > >> > >> Please review > >> http://cr.openjdk.java.net/~goetz/wr18/8195663- > fixHelpTest/webrev.02/ > >> which excludes the Oracle proprietary tools from the test. > >> I can not verify this. > > > > Wouldn't a whitelist work better here than a blacklist? Other > > distributions might add other tools (in theory). > > Yes, that was my suggestion as well in the JBS entry for this bug. > > Kumar > > > > > David > > > >> Best regards, > >> Goetz. > >> > >> > >>> -----Original Message----- > >>> From: jdk-dev [mailto:[email protected]] On Behalf > Of > >>> Lindenmaier, Goetz > >>> Sent: Donnerstag, 18. Januar 2018 17:02 > >>> To: Weijun Wang <[email protected]> > >>> Cc: [email protected] > >>> Subject: RE: RFR(XS): 8195663: jdk/tools/launcher/HelpFlagsTest.java > >>> fails > >>> with java.lang.AssertionError: HelpFlagsTest failed: > >>> > >>> Hi, > >>> > >>> Added ssvagent: > >>> http://cr.openjdk.java.net/~goetz/wr18/8195663- > fixHelpTest/webrev.02/ > >>> > >>> Best regards, > >>> Goetz. > >>> > >>>> -----Original Message----- > >>>> From: Weijun Wang [mailto:[email protected]] > >>>> Sent: Donnerstag, 18. Januar 2018 16:44 > >>>> To: Lindenmaier, Goetz <[email protected]> > >>>> Cc: Kumar Srinivasan <[email protected]>; jdk- > >>>> [email protected] > >>>> Subject: Re: RFR(XS): 8195663: > >>>> jdk/tools/launcher/HelpFlagsTest.java fails > >>>> with java.lang.AssertionError: HelpFlagsTest failed: > >>>> > >>>> On Windows, there is also a ssvagent.exe. > >>>> > >>>>> On Jan 18, 2018, at 11:37 PM, Lindenmaier, Goetz > >>>> <[email protected]> wrote: > >>>>> > >>>>> Hi, > >>>>> > >>>>> the most simple fix is to just exclude the four tools: > >>>>> http://cr.openjdk.java.net/~goetz/wr18/8195663- > >>> fixHelpTest/webrev.01/ > >>>>> Unfortunately I can't test this, as my build does not contain these > >>>>> tools. > >>>>> > >>>>> If somebody tells me how to build these tools, I will adapt > >>>>> the test to check them, and eventually adapt the tools to > >>>>> CSR 8191477 https://bugs.openjdk.java.net/browse/JDK-8191477. > >>>>> > >>>>> What list should this be reviewed on? I just posted to jdk-dev for > >>>>> now. > >>>>> > >>>>> Best regards, > >>>>> Goetz. > >>
