On Tue, 5 Sep 2023 19:56:56 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
>> Michael Strauß has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Removed application preferences implementation > > modules/javafx.graphics/src/test/java/test/com/sun/javafx/application/preferences/PlatformPreferencesTest.java > line 41: > >> 39: >> 40: import static org.junit.jupiter.api.Assertions.*; >> 41: import static test.javafx.collections.MockMapObserver.Tuple.tup; > > this generates 9 errors in eclipse, starting with > > Description Resource Type > The type test.javafx.collections.MockMapObserver.Tuple.tup is not accessible > PlatformPreferencesTest.java Java Problem > > the following change to graphics/.classpath should fix it (inc. complete > file): > > > <?xml version="1.0" encoding="UTF-8"?> > <classpath> > <classpathentry kind="src" path="src/main/java"/> > <classpathentry kind="src" path="build/gensrc/jsl-prism"/> > <classpathentry kind="src" path="build/gensrc/jsl-decora"/> > <classpathentry kind="src" path="build/hlsl/Decora"/> > <classpathentry kind="src" path="build/hlsl/Prism"/> > <classpathentry kind="src" output="testbin" path="src/shims/java"> > <attributes> > <attribute name="test" value="true"/> > </attributes> > </classpathentry> > <classpathentry kind="src" output="testbin" path="src/test/java"> > <attributes> > <attribute name="test" value="true"/> > <attribute name="optional" value="true"/> > </attributes> > </classpathentry> > <classpathentry kind="src" path="src/main/resources"> > <attributes> > <attribute name="optional" value="true"/> > </attributes> > </classpathentry> > <classpathentry kind="src" output="testbin" path="src/test/resources"> > <attributes> > <attribute name="test" value="true"/> > <attribute name="optional" value="true"/> > </attributes> > </classpathentry> > <classpathentry combineaccessrules="false" kind="src" path="/base"> > <attributes> > <attribute name="module" value="true"/> > <attribute name="add-exports" > value="javafx.base/com.sun.javafx.property=javafx.graphics:javafx.base/test.javafx.collections=javafx.graphics:javafx.base/test.util.memory=javafx.graphics"/> > </attributes> > </classpathentry> > <classpathentry kind="con" > path="org.eclipse.jdt.junit.JUNIT_CONTAINER/5"> > <attributes> > <attribute name="test" value="true"/> > </attributes> > </classpathentry> > <classpathentry kind="con" > path="org.eclipse.jdt.launching.JRE_CONTAINER"> > <attributes> > <attribute name="module" value="true"/> > <attribute name="add-exports" > value="java.base/sun.security.util=javafx.graphics"/> > </attributes> > </classpathentry> > <classpathentry kind="output" path="bin"/> > </classpath> I've updated the `.classpath` file as shown. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1014#discussion_r1316539556