JDK-8203379 : Remove javapackager sources from OpenJFX repo
  • Type: Enhancement
  • Component: javafx
  • Sub-Component: other
  • Affected Version: openjfx11
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2018-05-17
  • Updated: 2018-08-20
  • Resolved: 2018-08-20
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.
Other
openjfx12Fixed
Related Reports
Relates :  
JDK-8195798 - ��� Address dependencies in javafx.* modules on internal APIs of core modules
Relates :  
JDK-8200758 - JEP 343: Packaging Tool (Incubator)
Description
As part of removing JavaFX from the JDK the javapackager tool, and the jdk.packager and jdk.packager.services modules were necessarily removed as well.

The javapackager is not included with OpenJDK builds, nor will it be included with standalone JavaFX builds. Instead we are looking to provide a replacement tool in OpenJDK.

In the mean time, developers can use the existing javapackager tool from JDK 10.

I note that the sources will still be available, either in the 11-dev/rt repo or in the jfx-dev/rt repo by cloning (or updating) the repo from the 11+23 tag.
Comments
Changeset: 1c9c3ac09110 Author: kcr Date: 2018-08-20 08:15 -0700 URL: http://hg.openjdk.java.net/openjfx/jfx-dev/rt/rev/1c9c3ac09110 8203379: Remove javapackager sources from OpenJFX repo Reviewed-by: mbilla, prr, nlisker
20-08-2018
Yes, it was a lot of build logic for packager. It will be nice to get rid of it...
18-08-2018
That is a lot of build logic for packager. +1 from me. -phil.
18-08-2018
Thanks for the review. The SceneBuilder ones were left alone intentionally (since they are scheduled for removal...I have a patch for that ready to go, too). Edited to add: the SceneBuilder build files in question are broken with or without the removal of the packager sources.
18-08-2018
Thanks for the review. I'll update the copyright years when I push (which won't be until Monday since I want to give others a chance to comment).
18-08-2018
webrev.00 covered all fxpackager references.. +1 Nit : (though optional) you can correct copyright header for armv6hf.gradle, ios.gradle and mac.gradle
18-08-2018
The only leftover references I could find are in files under SceneBuilder, which is scheduled to be removed. Looks good otherwise.
18-08-2018
Webrev: http://cr.openjdk.java.net/~kcr/8203379/webrev.00/ The bulk of the webrev was a simple `hg rm` of the following: .idea/fxpackager.iml dependencies/java.base/ dependencies/jdk.jlink/ modules/jdk.packager.services/ modules/jdk.packager/ netbeans/antplugin/ netbeans/fxpackager/ netbeans/fxpackagerTests/ netbeans/jdk.packager.services/ netbeans/jdk.packager/ The rest of the changes were to remove references from the build logic and IDE scripts.
17-08-2018
Thanks for the offer, but I have a patch all ready to go, and will send the webrev today or Monday.
17-08-2018
Kevin, would you like me to work on this?
17-08-2018