Looks good to me.
-Sundar
On 15/11/18, 5:46 AM, Mandy Chung wrote:
This patch improves `jdeps --print-module-deps`, `--list-deps` and
`--list-reduced-deps` to report missing dependences and also do
transitive
dependence analysis as the default.
Webrev at:
http://cr.openjdk.java.net/~mchung/jdk12/webrevs/8213909/webrev.00/
$ jdeps --class-path libs --print-module-deps app.jar
`--print-module-deps` finds the modules required by the specified
application.
Its result can be used to create a runtime image for such application
to run.
The current behavior does not report missing dependences. In addition,
`--print-module-deps` only reports module dependences required by
app.jar.
To include the transitive module dependences required by libs, if
referenced,
-R option can be used.
If -R option is not specified, app.jar will fail to run on the runtime
image
created by the output from `jdeps --print-module-deps`. The patch
changes
the default behavior to do transitive analysis. In addition, it will
report
as an error if any dependence is missing and not found.
The --ignore-missing-deps option can be used to ignore missing deps.
This patch also includes a simple fix in
JdepsTask.ResourceBundleHelper for:
JDK-8168869: jdeps: localized messages don't use proper line breaks
I can separate it in its own changeset when I push.
Thanks
Mandy
