Add gradle wrapper files

[aalmiray]

The build (currently) has a strict check on Gradle 4.3 as supported version. It's recommended to use the Gradle wrapper to ensure a project is built with the correct Gradle version for it.

Consider adding gradle wrapper files.

[johanvos]

I'm in favor of this too. There was some discussion previously where people argued against it, but I forgot the reasons.

[hendrikebbers]

+1 Von meinem iPhone gesendet

…

Am 15.03.2018 um 15:13 schrieb Andres Almiray ***@***.***>: The build (currently) has a strict check on Gradle 4.3 as supported version. It's recommended to use the Gradle wrapper to ensure a project is built with the correct Gradle version for it. Consider adding gradle wrapper files. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

[eugener]

+1. But before merging, both builds at TravisCI and AppVeyor should be fixed/tested.

[johanvos]

Can you create an issue in JBS and refer to here?

[aalmiray]

Sure, I can, one question though: what's JBS?

[johanvos]

:) Java Bug System, see
https://github.com/javafxports/openjdk-jfx/blob/develop/.github/CONTRIBUTING.md

I am very well aware of the double administration of this, but in order to have patches that can go upstream, this is an important step.

[aalmiray]

Done. internal review ID : 9053024

[hendrikebbers]

Just one question: Why do we use Travis AND AppVeyor? Shouldn't one CI build be enough?

[aalmiray]

Appveyor runs on Windows while Travis runs on Linux by default.

[hendrikebbers]

ah, THX :)

[aalmiray]

https://bugs.java.com/bugdatabase/view_bug.do?bug_id=JDK-8199841

[johanvos]

covered by PR #26 which can be merges as soon as builds are successful.

[johanvos]

builds passed (against develop branch), PR merged