-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Doesn't work with gradle 7.x #17
Comments
Do you know of an alternative way of doing this that would be compatible with Gradle 5.x? |
I think that However, |
I'm all for supporting Gradle 7.x and even 8.x when it comes out, but let's see if we can keep backward compatibility. Can you try to use getLibsDirName() to see if it works for now? |
I've made the change in the latest commit and it now with Gralde 7.x. I'm still getting a few warnings for Gradle 8.0, and it looks like it will be hard to keep compatibility with 5.0, but we still have something like year before that happens, and by then we might as well just drop support for it.
The Gradle Wrapper doesn't help for plugins. Those essentially become part of Gradle at runtime and are typically tied to a very narrow range of versions of Gradle. In any case, thanks for reporting this issue and for your help solving it! |
Thanks, I'll give it a try and update.
I meant for the examples, sorry I wasn't clear. |
👍
Ah, the scripts for that are pretty large and I find it just creates noise in the repository here since the sample code is meant to be part of the "documentation". For complete sample projects like that, we can put those on /~https://github.com/bytedeco/sample-projects, even if it's essentially the same thing but with Gradle Wrapper. |
It works, many thanks |
getLibsDir
was deprecated in 6.x and removed in 7.xgradle-javacpp/src/main/java/org/bytedeco/gradle/javacpp/BuildExtension.java
Line 75 in d289424
The text was updated successfully, but these errors were encountered: