Re: [11] Review request: 8203378: JDK build fails to compile javafx.graphics module-info.java if FX was built with OpenJDK

2018-05-21 Thread Nir Lisker
Fine by me. - Nir On Tue, May 22, 2018 at 12:29 AM, Kevin Rushforth < kevin.rushfo...@oracle.com> wrote: > In that case, why don't I revert my changes and let you remove the > references after j.u.l is removed. > > -- Kevin > > > > On 5/21/2018 2:13 PM, Nir Lisker wrote: > > Since the packager

Re: [11] Review request: 8203378: JDK build fails to compile javafx.graphics module-info.java if FX was built with OpenJDK

2018-05-21 Thread Kevin Rushforth
In that case, why don't I revert my changes and let you remove the references after j.u.l is removed. -- Kevin On 5/21/2018 2:13 PM, Nir Lisker wrote: Since the packager is being removed I don't see a reason to keep any references to it. In any case, the Eclipse files are planned to be

Re: [11] Review request: 8203378: JDK build fails to compile javafx.graphics module-info.java if FX was built with OpenJDK

2018-05-21 Thread Nir Lisker
Since the packager is being removed I don't see a reason to keep any references to it. In any case, the Eclipse files are planned to be updated [1] after j.u.l. is removed [2]. [1] http://mail.openjdk.java.net/pipermail/openjfx-dev/2018-April/021740.html [2]

[11] Review request: 8203378: JDK build fails to compile javafx.graphics module-info.java if FX was built with OpenJDK

2018-05-21 Thread Kevin Rushforth
Phil or Ajit, Please review the following simple fix: https://bugs.openjdk.java.net/browse/JDK-8203378 http://cr.openjdk.java.net/~kcr/8203378/webrev.00/ This removes one qualified export from javafx.graphics to the no-longer-built jdk.packager module. More details in JBS. Note to Eclipse