Skip to content
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

Plugin is incompatible with Gradle 7 #92

Closed
dubinsky opened this issue Apr 2, 2021 · 11 comments
Closed

Plugin is incompatible with Gradle 7 #92

dubinsky opened this issue Apr 2, 2021 · 11 comments

Comments

@dubinsky
Copy link

dubinsky commented Apr 2, 2021

ScalaTestAction calls org/gradle/process/internal/DefaultExecActionFactory.root() with null for gradleUserHome; this is no longer supported starting with Gradle 7 (commit with the change).

Also, the method called is deprecated; the comment says:

Do not use this. It's here because some of the services this type needs are not easily accessed in certain cases and will be removed ay some point. Use one of the other methods instead

@DanySK
Copy link

DanySK commented Apr 12, 2021

The problem seems to be here:

JavaExecAction javaExecHandleBuilder = DefaultExecActionFactory.root().newJavaExecAction()

@maiflai
Copy link
Owner

maiflai commented Apr 14, 2021

Yes, I'm trying to work out how to update this without having to rewrite all the tests.

@dubinsky
Copy link
Author

Yes, I'm trying to work out how to update this without having to rewrite all the tests.

Thank you!

@josecsotomorales
Copy link

Just got this issue when updating my project to Gradle 7.0, glad that you folks are already tracking it!

@Habitats
Copy link

Any update on this? 🙏

@maiflai
Copy link
Owner

maiflai commented Apr 22, 2021

I've uploaded 0.31 with a partial fix (it still uses a deprecated api).

Please could you take a look?

@dubinsky
Copy link
Author

@maiflai yay! it works!!! thanks!

@josecsotomorales
Copy link

@maiflai it's working! thanks! 🚀

@dpsoft
Copy link

dpsoft commented Apr 22, 2021

@maiflai works like a charm!! thanks!

@Habitats
Copy link

works like a charm indeed!

thanks for fixing! :D

@maiflai
Copy link
Owner

maiflai commented Apr 23, 2021

thanks folks.

@maiflai maiflai closed this as completed Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants