Erik, Mandy:

Looks good to me.

/Tim


Thanks for handling it this quickly. Here is the proper fix. I had missed using double dollars inside the macro definition. I have verified that the javafx modules show up in the jre and that the ModuleLoaderMap.java adds them to the platform modules list.

Bug: https://bugs.openjdk.java.net/browse/JDK-8153261
Webrev: http://cr.openjdk.java.net/~erikj/8153261/webrev.01/

/Erik

On 2016-03-31 22:03, Mandy Chung wrote:
I have yet to understand why the eval doesn’t do its work. This patch includes ReadImportMetaData in make/Images.gmk and make/gensrc/GensrcModuleLoaderMap.gmk. I need to push this fix to unblock jdk9/client testing.

http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8153217/webrev.00/

Mandy

Reply via email to