RFR JDK-8213909: jdeps --print-module-deps should report missing dependences

Mandy Chung mandy.chung at oracle.com
Mon Nov 19 17:22:48 UTC 2018


Thanks Sundar.  Here is the CSR:
    https://bugs.openjdk.java.net/browse/JDK-8213915

Mandy

On 11/19/18 6:48 AM, Sundararajan Athijegannathan wrote:
> 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



More information about the core-libs-dev mailing list