RFR: 8205438: Re-enable shebang tests in test/jdk/tools/launchers/SourceMode.java
James Laskey
james.laskey at oracle.com
Wed Jun 27 21:15:40 UTC 2018
We went through the same exercise with jjs. Tough coming up with shebang tests that worked on all platforms. Shebang args vs command line args was a nightmare.
Sent from my iPhone
> On Jun 27, 2018, at 6:02 PM, mandy chung <mandy.chung at oracle.com> wrote:
>
> Looks good. It's amazing to find out 3 different behavior on these 3 platforms.
>
> Mandy
>
>> On 6/27/18 1:37 PM, Jonathan Gibbons wrote:
>> Please review a test fix to re-enable shebang tests in test/jdk/tools/launchers/SourceMode.java .
>> The test cases for invoking the source launcher via the shebang mechanism have been disabled because they were adversely affected by the very long pathnames on our internal test infastructure, causing the test cases to overflow the system limit of 128 characters for the first line of a shebang file
>> The fix is to run jlink to create a small image that can be addressed by a short relative path.
>> Some of the test cases on macOS and Solaris remain explicitly disabled because of the inconsistencies in the system-level support for shebang execution.
>> Finally, the use of hard-coded version numbers in the test is changed to avoid having to add this test to the list of tests that need to be modified at the beginning of each release.
>> -- Jon
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8205438
>> Webrev: http://cr.openjdk.java.net/~jjg/8205438/webrev.00/
>
>
More information about the core-libs-dev
mailing list