opencv-java / getting-started

Getting started with OpenCV and JavaFX
105 stars 176 forks source link

Gradle #8

Closed fmatosqg closed 7 years ago

fmatosqg commented 7 years ago

Gradle support for HelloCV

luigidr commented 7 years ago

I cannot accept this merge request, sorry. The fixes are obviously about an Android Studio project, while this repository covers Java desktop.

Let me know whether the proposed changes work in other JetBrains IDEs. In this case, I can think to merge the request in a separated branch named "gradle". However, it cannot be an Android-only project.

fmatosqg commented 7 years ago

Changes were made over Intellij community edition. Although I'd suggest a different approach, because this pull request may be incompatible with Eclipse. That said, I recommend not merging this branch, since there's still a lot of people doing Java over Eclipse.

There is a way to write gradle files maintaining the current folder structure, making the project easily compatible with both Eclipse and Intellij over gradle, although that wasn't the path I took here.

luigidr commented 7 years ago

I will merge your request in a separated branch named gradleand I will edit the README to point out the IntelliJ requirement.