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

Declare system property usage (for Gradle 7 configuration cache) #56

Closed
C-Otto opened this issue Mar 16, 2021 · 4 comments
Closed

Declare system property usage (for Gradle 7 configuration cache) #56

C-Otto opened this issue Mar 16, 2021 · 4 comments
Assignees
Milestone

Comments

@C-Otto
Copy link
Contributor

C-Otto commented Mar 16, 2021

With Gradle 7 M3 and --configuration-cache I get an error that the CPD plugin reads a system property that is not declared.

org.gradle.api.InvalidUserCodeException: Read system property 'file.encoding'
	at org.gradle.configurationcache.SystemPropertyAccessListener.systemPropertyQueried(SystemPropertyAccessListener.kt:92)
	at org.gradle.internal.classpath.Instrumented.systemProperty(Instrumented.java:83)
	at org.gradle.internal.classpath.Instrumented.systemProperty(Instrumented.java:77)
	at de.aaschmid.gradle.plugins.cpd.CpdExtension.<init>(CpdExtension.java:26)
	at de.aaschmid.gradle.plugins.cpd.CpdExtension_Decorated.<init>(Unknown Source)

Possible fix: https://docs.gradle.org/7.0-milestone-3/userguide/configuration_cache.html#config_cache:requirements:undeclared_sys_prop_read

@aaschmid
Copy link
Owner

hm ... actually thought that this is already fixed. However it seems not to.
Thanks for reporting @C-Otto. I will try to fix it asap for staying compatible with the next version.

@aaschmid aaschmid self-assigned this Mar 16, 2021
@aaschmid aaschmid added the bug label Mar 16, 2021
@aaschmid aaschmid added this to the next version milestone Mar 16, 2021
@aaschmid
Copy link
Owner

aaschmid commented Mar 27, 2021

Duplicate of #54 and there I at least partly fixed it ;-)

However never released it as you did not confirm the fix...

@C-Otto
Copy link
Contributor Author

C-Otto commented Mar 27, 2021

Oh sorry, I forgot about the other issue... is there anything I can do now?

@aaschmid
Copy link
Owner

aaschmid commented Mar 27, 2021

No, I thing I got the remaining problem about this and I think about fixing it will by using default encoding of UTF-8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants