> On Feb 14, 2017, at 5:52 AM, Sundararajan Athijegannathan
> wrote:
>
> Please review http://cr.openjdk.java.net/~sundar/8174718/webrev.00/ for
> https://bugs.openjdk.java.net/browse/JDK-8174718
JLinkNegativeTest.java - line 352 is not really needed. Otherwise looks fine
to me.
Mandy
+1
> On Feb 14, 2017, at 9:52 AM, Sundararajan Athijegannathan
> wrote:
>
> Please review http://cr.openjdk.java.net/~sundar/8174718/webrev.00/ for
> https://bugs.openjdk.java.net/browse/JDK-8174718
>
> Thanks,
> -Sundar
On 14/02/2017 15:56, Sundararajan Athijegannathan wrote:
Yes - because the test module's name is "inconsistentJar" and jlink
adds module name to the error message.
-Sundar
Got it, I thought it was a typo initially.
-Alan.
Yes - because the test module's name is "inconsistentJar" and jlink adds
module name to the error message.
-Sundar
On 14/02/17, 9:08 PM, Alan Bateman wrote:
On 14/02/2017 13:52, Sundararajan Athijegannathan wrote:
Please review http://cr.openjdk.java.net/~sundar/8174718/webrev.00/
for https:
On 14/02/2017 13:52, Sundararajan Athijegannathan wrote:
Please review http://cr.openjdk.java.net/~sundar/8174718/webrev.00/
for https://bugs.openjdk.java.net/browse/JDK-8174718
Is the output matching in the test correct to check for "Module
inconsistentJar's descriptor ..."?
-Alan
Please review http://cr.openjdk.java.net/~sundar/8174718/webrev.00/ for
https://bugs.openjdk.java.net/browse/JDK-8174718
Thanks,
-Sundar