Thanks, I will proceed integrating JDK-8161171 then.
Valerie
On 7/14/2016 12:59 AM, Mandy Chung wrote:
On Jul 14, 2016, at 8:10 AM, Valerie Peng <valerie.p...@oracle.com> wrote:
Sean,
Can you please review the following two webrevs?
Bug: https://bugs.openjdk.java.net/browse/JDK-8159488
Webrev: http://cr.openjdk.java.net/~valeriep/8159488/
Update to Modules.gmk looks good.
While making changes for 8159488, I noticed a problem with my earlier putback
of 8154191 - the top level Modules.gmk was not integrated.
So, I filed 8161171: Missed the make/common/Modules.gmk file when integrating
JDK-8154191.
Can you also review this? It's essentially the same change as the one reviewed.
Bug: https://bugs.openjdk.java.net/browse/JDK-8161171
Webrev: http://cr.openjdk.java.net/~valeriep/8161171/webrev.00/
+1
You can go ahead and push the fix for JDK-8161171.
Mandy
