Re: [11] Review request: 8199841: Add gradle wrapper files to build

2018-03-29 Thread Johan Vos
Looks good.

- Johan

On Thu, Mar 29, 2018 at 6:40 PM Kevin Rushforth 
wrote:

> Hi Johan,
>
> Please review the following to add gradle wrapper:
>
> https://bugs.openjdk.java.net/browse/JDK-8199841
> http://cr.openjdk.java.net/~kcr/8199841/webrev.00/
>
> This is the aggregation of two changesets pushed to the github mirror:
>
>
> https://github.com/javafxports/openjdk-jfx/commit/7ff32048e62e7338ed6e2142ca0887b1ebc8aaa7
>
> https://github.com/javafxports/openjdk-jfx/commit/5bb8f3f25d31af37dedb040da7e4cd3c91a31919
>
> so there should be no merge conflict. It includes the associated
> third-party legal file.
>
> -- Kevin
>
>


[11] Review request: 8199841: Add gradle wrapper files to build

2018-03-29 Thread Kevin Rushforth

Hi Johan,

Please review the following to add gradle wrapper:

https://bugs.openjdk.java.net/browse/JDK-8199841
http://cr.openjdk.java.net/~kcr/8199841/webrev.00/

This is the aggregation of two changesets pushed to the github mirror:

https://github.com/javafxports/openjdk-jfx/commit/7ff32048e62e7338ed6e2142ca0887b1ebc8aaa7
https://github.com/javafxports/openjdk-jfx/commit/5bb8f3f25d31af37dedb040da7e4cd3c91a31919

so there should be no merge conflict. It includes the associated 
third-party legal file.


-- Kevin