This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 516911 - [Java 9] account for proposed command line argument --illegal-access=*
Summary: [Java 9] account for proposed command line argument --illegal-access=*
Status: RESOLVED WONTFIX
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: Launcher (show other bugs)
Version: 4.7.0 Oxygen   Edit
Hardware: All All
: P1 enhancement (vote)
Target Milestone: Oxygen RC3   Edit
Assignee: Arun Thondapu CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 493761 516143
Blocks:
  Show dependency tree
 
Reported: 2017-05-18 14:41 EDT by Markus Keller CLA
Modified: 2017-06-26 12:02 EDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Keller CLA 2017-05-18 14:41:32 EDT
http://mail.openjdk.java.net/pipermail/jigsaw-dev/2017-May/012673.html
proposes a new default behavior that

a) would make the workarounds for bug 493761 and bug 516143 obsolete

b) could break Eclipse products on Java 9 if the current --permit-illegal-access gets removed and we still keep it in the default eclipse.ini (scenario from bug 493761 comment 61)

Currently, we can't do much about that, but I'm opening this bug as placeholder to keep it on the radar and follow the developments in Java 9.

If by RC3 (May 30) it looks like --permit-illegal-access is going to be removed, then we should at least remove this argument from the eclipse.ini.
Comment 1 Arun Thondapu CLA 2017-05-19 01:26:43 EDT
Thanks for bringing it to notice Markus!
Comment 2 Markus Keller CLA 2017-05-19 04:25:24 EDT
(In reply to Arun Thondapu from comment #1)
> Thanks for bringing it to notice Markus!

Thanks for fixing the target  milestone, Arun! I'm out of practice using that popup since I started to use http://www.eclipse.org/jdt/ui/dev.php#scripts ;-)
Comment 3 Dani Megert CLA 2017-05-30 11:37:39 EDT
This is a unpredictable moving target.

Decision from the PMC to get maximum safety: revert the launcher to before M7 and remove ini arguments. Add a what's new entry like in 4.6 (https://www.eclipse.org/eclipse/news/4.6/platform.php#java-9) to instruct users how to launch with Java 9,
Comment 4 Martin Oberhuber CLA 2017-05-30 15:37:35 EDT
See https://wiki.eclipse.org/Eclipse/PMC for some rationale of the PMC decision.

See https://bugs.eclipse.org/bugs/show_bug.cgi?id=517452 for an idea improving the Launcher moving forward -- replacing obscure C code that's hard to change by a data-driven approach that can adapt to different VMs. Comments welcome on that bug.
Comment 5 Arun Thondapu CLA 2017-05-31 05:18:49 EDT
Resolving as WONTFIX as we're not going to address "--illegal-access=*" for the Oxygen release.
Comment 6 Arun Thondapu CLA 2017-05-31 07:08:00 EDT
(In reply to Dani Megert from comment #3)
> This is a unpredictable moving target.
> 
> Decision from the PMC to get maximum safety: revert the launcher to before
> M7 and remove ini arguments. 

Opened bug 517482 and bug 517507 for these two tasks.

Add a what's new entry like in 4.6
> (https://www.eclipse.org/eclipse/news/4.6/platform.php#java-9) to instruct
> users how to launch with Java 9,

Dani, do we need to open a bug for this?