Re: Review Request: 8204648: test/jdk/tools/launchers/SourceMode.java fails with long shebang line

2018-06-09 Thread joe darcy
Skipping the shebang tests is fine a workaround Mandy; thanks, -Joe On 6/8/2018 9:57 PM, mandy chung wrote: I run into some issue with shebang tests.  Since Jon is on vacation, I revise the patch to skip the shebang test temporarily until he returns. Mandy diff --git

Re: Review Request: 8204648: test/jdk/tools/launchers/SourceMode.java fails with long shebang line

2018-06-08 Thread mandy chung
I run into some issue with shebang tests. Since Jon is on vacation, I revise the patch to skip the shebang test temporarily until he returns. Mandy diff --git a/test/jdk/tools/launcher/SourceMode.java b/test/jdk/tools/launcher/SourceMode.java --- a/test/jdk/tools/launcher/SourceMode.java +++

Review Request: 8204648: test/jdk/tools/launchers/SourceMode.java fails with long shebang line

2018-06-08 Thread mandy chung
JDK-8204588 [1] fixed the test failure caused by long paths to the Java launcher in some test execution environments, causing the shebang line to overflow the underlying system limit of 128 characters. The test needs a small tweak to the max javaCmd length to reduce from 100 to 90 since the