On Thu, 12 Nov 2020 13:50:41 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> Issue was broken build in Eclipse after fix of 
>> [JDK-8244297](https://bugs.openjdk.java.net/browse/JDK-8244297): 
>> 
>> - the new memory test class requires jdk.management/java.management
>> - users of the test class (in controls) need access to its package
>> 
>> Changed .classpath files in base and controls to fix this.
>
> The change looks reasonable to me, but since I don't use Eclipse it would be 
> better for someone who does to review.
> @nlisker can you review this?

Yes, but that's a problem of no reviewers. If I submit it myself then there 
will also be no one to review. @kevinrushforth What do we do? I think that 
previously when I updated the Eclipse files, there's wasn't a thorough review.

-------------

PR: https://git.openjdk.java.net/jfx/pull/352

Reply via email to