RFR 8220700: jlink generated launcher script needs quoting to avoid parameter expansion

Severin Gehwolf sgehwolf at redhat.com
Wed Aug 21 08:45:25 UTC 2019


Hi,

On Wed, 2019-08-21 at 12:23 +0530, sundararajan.athijegannathan at oracle.com wrote:
> Updated to add a test:
> 
> http://cr.openjdk.java.net/~sundar/8220700/webrev.01
> 
> PS. There are launcher tests in BasicTest.java and arguments are passed 
> even - but args are only printed. The new test passes args and checks 
> the return value that is computed using the args.

While that's true it's not a regression test for this particular bug,
is it? It's not failing prior this patch! I think what you'd need is
something like this on top:

http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8220700-test-additions.patch

This fails prior 8220700 and passes after.

Thanks,
Severin



More information about the jigsaw-dev mailing list