JDK-8160835 : Deprecate -makeall from Java Packager
  • Type: Enhancement
  • Component: deploy
  • Sub-Component: packager
  • Affected Version: 9
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2016-07-05
  • Updated: 2017-05-17
  • Resolved: 2016-08-18
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 9
9Fixed
Related Reports
Relates :  
Sub Tasks
JDK-8179642 :  
Description
Background

Java Packager's -makeall is one stop shopping: It accepts one java file, a few arguments (-makeall -appclass -preloader -classpath -name -width -height -v), performs compilation by invoking javac, creates a jar, and a JNLP file is generated all in one step. It is designed to be an extremely simple build tool.


Justification

-makeall defines almost all options allowing for very little control by the developer. Java Packager was enhanced to add support for native bundles but no -makeall arguments were added for native bundles and no Ant tasks exist for -makeall. It is only a CLI argument. -makeall has been buggy since 7u and is only useful for Web Start Applications. -makeall is unlikely to have modular support (which was not added since this is primarily a JNLP only feature). Keeping -makeall adds development time to fix or enhance the feature, and at least a week of SQE time. By deprecating we save SQE time testing an entire option and sub options -makeall -appclass -preloader -classpath -name -width -height -v for Windows, Linux and Mac, with and without modules.

Deprecation would take one new line of code:

460: System.err.println("-makeall is deprecated");

As you can see from the review, the diff provided below all existing deprecations use stdout, so those will be fixed as well.

Risk is extremely low.
Comments
Changeset: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/9f0c717f2ce7
18-08-2016

Looks good. +1
15-08-2016