RFR 8168925: MODULES property should be topologically ordered and space-separated list
sundararajan.athijegannathan at oracle.com
Mon Dec 12 05:58:27 UTC 2016
I'll refactor (and revisit) TOP/OTHER types in my next jlink change.
Thanks for the review.
On 12/12/16, 11:04 AM, Mandy Chung wrote:
>> On Dec 11, 2016, at 9:30 PM, Sundararajan Athijegannathan<sundararajan.athijegannathan at oracle.com> wrote:
>> Please review the updated webrev: http://cr.openjdk.java.net/~sundar/8168925/webrev.02/
> This looks better. Thanks for making the change.
> One thing to mention is that the storeRelease(ResourcePool pool) method to write the release file because the accept(ResourcePoolEntry file) method excludes “/java.base/release” entry:
> 384 case TOP:
> 385 break;
> This release file entry could have been written in the same way as other entries now and no need for this storeRelease method. I’m okay if you want to leave this as is and revisit the TOP and OTHER entry type in the future.
More information about the jigsaw-dev