Re: RFR: 8160881: Remove jvisualvm from JDK9

2017-01-18 Thread Erik Joelsson
Looks good. /Erik On 2017-01-18 18:57, Henry Jen wrote: Hi, Please review the webrev to remove jvisualvm from JDK9. http://cr.openjdk.java.net/~henryjen/jdk9/8160881/jdk/webrev/ Cheers, Henry

RFR: 8160881: Remove jvisualvm from JDK9

2017-01-18 Thread Henry Jen
Hi, Please review the webrev to remove jvisualvm from JDK9. http://cr.openjdk.java.net/~henryjen/jdk9/8160881/jdk/webrev/ Cheers, Henry

Re: Building OpenJDK 8 on macOS 10.12 - Path provided in --with-xcode-path not being used to locate Xcode?

2017-01-18 Thread David DeHaven
>> I'm trying to build OpenJDK 8 on macOS 10.12, and having some trouble >> getting it to accept the path to Xcode. >> >> $ ./configure >> "--with-xcode-path=$HOME/DevEnv/Applications/Xcode4/Xcode.app" >> >> checking Determining if we need to set DEVELOPER_DIR... yes >>

Re: Building OpenJDK 8 on macOS 10.12 - Path provided in --with-xcode-path not being used to locate Xcode?

2017-01-18 Thread David DeHaven
> I'm trying to build OpenJDK 8 on macOS 10.12, and having some trouble > getting it to accept the path to Xcode. > >$ ./configure > "--with-xcode-path=$HOME/DevEnv/Applications/Xcode4/Xcode.app" > >checking Determining if we need to set DEVELOPER_DIR... yes >

Re: RFR [9] Remove add exports from ModuleSummary build

2017-01-18 Thread 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

Re: RFR [9] Remove add exports from ModuleSummary build

2017-01-18 Thread Magnus Ihse Bursie
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

RFR [9] Remove add exports from ModuleSummary build

2017-01-18 Thread Chris Hegarty
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