> Refactor jpackage to separate the configuration and execution phases.
> At the configuration phase, jpackage parses command-line arguments and 
> validates them.
> At the execution phase, jpackage builds a bundle based on data collected at 
> the configuration phase.
> 
> There was no clear separation between these phases. Both used the same data 
> type (`Map<String, Object>`), making it hard to understand and use properly.
> 
> This change introduces data model to jpackage (classes in 
> "jdk.jpackage.internal.model" package). The output of the configuration phase 
> is either an instance of 
> [jdk.jpackage.internal.model.Application](https://github.com/openjdk/jdk/pull/19668/files#diff-e4e7717f1978a09ac4806eded5c7f94aa29b2ea56671545dc053cb83eba86919)
>  interface for app image bundling or 
> [jdk.jpackage.internal.model.Package](https://github.com/openjdk/jdk/pull/19668/files#diff-9908b5648e03bd8a8104f6f6f5aa08e5df78fbc0508823774d3458b22927b721)
>  for native package bundling.
> 
> The execution phase has been reworked to get configuration properties from 
> the new `jdk.jpackage.internal.model.Application` and 
> `jdk.jpackage.internal.model.Package` interfaces instead of extracting data 
> from `Map<String, Object>` "params".
> 
> Additionally, a notion of "packaging pipeline" 
> (jdk.jpackage.internal.PackagingPipeline class) was added to configure 
> packaging declaratively with more code sharing between bundlers.
> 
> jdk.jpackage module javadoc - 
> https://alexeysemenyukoracle.github.io/jpackage-javadoc/jdk.jpackage/module-summary.html
> 
> **Functional changes**
> jpackage behavior 99% remains the same, i.e., it produces the same bundles 
> for the given parameters. This change affects only the implementation. Still, 
> there are some changes in jpackage behavior. They are outlined below.
> 
>  - Minimize copying of the source app image when doing native packaging.
> 
> Before this change, native package bundlers made redundant copies of the 
> source app image. E.g., msi and linux package bundlers copied the external 
> app image (the one specified with `--app-image` parameter); linux package 
> bundlers always transformed the source app image if the installation 
> directory was in the "/usr" tree (`--install-dir /usr`). This change 
> eliminates all redundant app image copy/transformations.
> 
> - PKG bundler: change "preinstall" and "postinstall" scripts in app bundles.
> 
> post- and pre- install PKG scripts for SimplePackageTest package before and 
> after the change:
> <table>
> <thead>
> <tr>
> <th>Script</th>
> <th>New</th>
> <th>Old</th>
> </tr>
> </thead>
> <tbody...

Alexey Semenyuk has updated the pull request incrementally with one additional 
commit since the last revision:

  Remove redundant StandardBundlerParam.createResource()

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/19668/files
  - new: https://git.openjdk.org/jdk/pull/19668/files/b7865ff0..30e2a722

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=19668&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=19668&range=01-02

  Stats: 12 lines in 1 file changed: 0 ins; 12 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/19668.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/19668/head:pull/19668

PR: https://git.openjdk.org/jdk/pull/19668

Reply via email to