Re: gradle :web:test fails

2018-04-05 Thread Nir Lisker
Yes, I didn't build Webkit and the JDK is 10, but I thought the Webkit tests would be skipped if it weren't built. The filed bug should do it. Thanks. - Nir On Thu, Apr 5, 2018, 15:01 Kevin Rushforth wrote: > Yes, this is very likely the issue. If you take the

Review Request: JDK-8200587: Fix mistakes in FX API docs

2018-04-05 Thread Nir Lisker
Hi Kevin, After JDK-8200749 is committed, please review the fixes for documentation mistakes: https://bugs.openjdk.java.net/browse/JDK-8200587 http://cr.openjdk.java.net/~nlisker/8200587/webrev.00/ Thanks, Nir

RE: CFV: New OpenJFX Committer: Rajath Kamath

2018-04-05 Thread Ambarish Rapte
Vote: Yes -Original Message- From: Guru Sent: Tuesday, April 03, 2018 2:12 PM To: Kevin Rushforth Cc: openjfx-dev@openjdk.java.net Subject: Re: CFV: New OpenJFX Committer: Rajath Kamath Vote: YES > On 29-Mar-2018, at 10:12 PM, Kevin Rushforth

Re: gradle :web:test fails

2018-04-05 Thread Kevin Rushforth
Yes, this is very likely the issue. If you take the jfxwebkit.dll from the latest EA build of JDK 11 then you should be fine. Alternatively, if you have the patience to build webkit from source (at least once) then you can use that. Johan had a good idea that gradle :web:test should produce a

Re: CSSParser Color.parse() for unexpected CSS properties

2018-04-05 Thread Kevin Rushforth
OK, thanks. For the benefit of others who might be interested, this bug is now visible here: https://bugs.openjdk.java.net/browse/JDK-8201135 -- Kevin Matthew Elliot wrote: Hi Kevin, Priyanka from Oracle beat me to it and this small example hit the nail on the head immediately. The below

RE: gradle :web:test fails

2018-04-05 Thread Murali Billa
Hi Lisker, +one more point: I think you are not compiling webkit . The command "gradle :web:test" picks the webkit dll from JDK (which can have older versions like 10 / 9) . You can compile webkit with command " gradle -PCOMPILE_WEBKIT=true :web:test" and in this case webkit dll will be

Re: CSSParser Color.parse() for unexpected CSS properties

2018-04-05 Thread Matthew Elliot
Hi Kevin, Priyanka from Oracle beat me to it and this small example hit the nail on the head immediately. The below will throw and swallow and IllegalArgumentException in CSSParser in the following method. private ParsedValueImpl colorValueOfString(String str) { import

Will OpenJFX have a release ready in time for the Java 11 launch?

2018-04-05 Thread Mikael Christensen - QIAGEN
Hi, This is a question about Oracle's statement that JavaFX will no longer be part of the SDK starting from Java 11: https://blogs.oracle.com/java-platform-group/the-future-of-javafx-and-other-java-client-roadmap-updates Is the plan, that when Java 11 is released (scheduled for September 2018),

Re: Will OpenJFX have a release ready in time for the Java 11 launch?

2018-04-05 Thread Kevin Rushforth
Yes, this is the plan. There has been quite a lot of discussion on this, so I invite you to search the email archives [1]. Specifically, the recent "modules versus SDK's" thread [2]. You are welcome to join the mailing list [3] and contribute to the discussion if you like. -- Kevin [1]

Re: HEADS-UP: Proposal to bump the minimum boot JDK for FX to JDK 10

2018-04-05 Thread Kevin Rushforth
I created a pull request [3] for this, and discovered that the CI builds for the GitHub sandbox are using JDK 9.0.4. Michael Ennen is fixing this by switching both the Travis and Appveyor CI builds to use JDK 10 [4]. Given the disruptive nature of bumping the minimum, I think it is wise to