Re: 8174823: Module system implementation refresh (3/2017)

2017-03-24 Thread Magnus Ihse Bursie
Alright. Thanks for the answers. I'm happy with the build system changes. /Magnus On 2017-03-24 08:58, Alan Bateman wrote: On 23/03/2017 22:24, Mandy Chung wrote: REQUIRED_OS_NAME was used as OS_NAME in `release` file. Previously it was “Darwin” which is inconsistent with the value of “os.

Re: 8174823: Module system implementation refresh (3/2017)

2017-03-24 Thread Alan Bateman
On 23/03/2017 22:24, Mandy Chung wrote: REQUIRED_OS_NAME was used as OS_NAME in `release` file. Previously it was “Darwin” which is inconsistent with the value of “os.name” system property (“Mac OS X”). This change made OS_NAME to contain the same value as “os.name” system property. Right,

Re: 8174823: Module system implementation refresh (3/2017)

2017-03-23 Thread Mandy Chung
REQUIRED_OS_NAME was used as OS_NAME in `release` file. Previously it was “Darwin” which is inconsistent with the value of “os.name” system property (“Mac OS X”). This change made OS_NAME to contain the same value as “os.name” system property. Mandy > On Mar 23, 2017, at 2:51 PM, Magnus Ihse

Re: 8174823: Module system implementation refresh (3/2017)

2017-03-23 Thread Magnus Ihse Bursie
What is the REQUIRED_OS_NAME used for, and should it not be "macOS" instead of "Mac OS X" nowadays? /Magnus On 2017-03-22 12:30, Erik Joelsson wrote: (adding build-dev) Build changes look good to me. /Erik On 2017-03-21 22:42, Alan Bateman wrote: We have been accumulating changes in the ja

Re: 8174823: Module system implementation refresh (3/2017)

2017-03-22 Thread Erik Joelsson
(adding build-dev) Build changes look good to me. /Erik On 2017-03-21 22:42, Alan Bateman wrote: We have been accumulating changes in the jake forest for the last few weeks to align with the proposals in JSR 376 and also to pick up API changes and bug fixes. It's time to bring these changes