RFR [9] Remove add exports from ModuleSummary build

[Claes Redestad]

+1

/Claes

On 01/18/2017 01:27 PM, Magnus Ihse Bursie wrote:
> On 2017-01-18 13:11, Chris Hegarty wrote:
>> After a recent change [1], I noticed a warning during compilation of
>> the ModuleSummary build tool:
>>
>>    warning: [options] module name in --add-exports option not found: 
>> jdk.jdeps
>>
>> It can be seen from the history of ModuleSummary.java in jake that at
>> one point this class used types from com.sun.tools.classfile ( in the 
>> jdeps
>> module ), but no longer does. The add exports can simply be removed,
>> as it it not needed any more.
>>
>>
>> diff --git a/make/ModuleTools.gmk b/make/ModuleTools.gmk
>> --- a/make/ModuleTools.gmk
>> +++ b/make/ModuleTools.gmk
>> @@ -39,7 +39,6 @@
>>       build.tools.jigsaw.GenGraphs
>>     TOOL_MODULESUMMARY := $(BUILD_JAVA) -esa -ea -cp 
>> $(TOOLS_CLASSES_DIR) \
>> -    --add-exports jdk.jdeps/com.sun.tools.classfile=ALL-UNNAMED \
>>       build.tools.jigsaw.ModuleSummary
>>     TOOL_ADD_PACKAGES_ATTRIBUTE := $(BUILD_JAVA) $(JAVA_FLAGS_SMALL) \
>>
>> -Chris.
>>
>> [1] https://bugs.openjdk.java.net/browse/JDK-8171380
>
> Looks good to me.
>
> /Magnus
>