Skip to content
This repository has been archived by the owner on Feb 11, 2022. It is now read-only.

Change deprecated classesDir for classesDirs #170

Closed
wants to merge 1 commit into from

Conversation

pablisco
Copy link
Contributor

@pablisco pablisco commented Mar 4, 2019

classesDir is deprecated in Gradle 4.x and we should use classesDirs instead as Gradle 5.x removes this property.

Without this change, Gradle complains about it if we use Gradle 5.x on the client:

* What went wrong:
Could not determine the dependencies of task ':core-java:findbugsMain'.
> Could not get unknown property 'classesDir' for main classes of type org.gradle.api.internal.tasks.DefaultSourceSetOutput.

Fixes #94

@pablisco pablisco added the bug label Mar 4, 2019
@pablisco pablisco force-pushed the fix-deprecated-classesDir-property branch from 0872bfb to 5f72e72 Compare March 4, 2019 16:49
@pablisco pablisco changed the title Change deprecated classesDir for classesDirs Change deprecated classesDir for classesDirs Mar 4, 2019
@tasomaniac
Copy link
Contributor

tasomaniac commented Mar 5, 2019

This is duplicate of #94 but I am closing #94 since you have more details here.

@tasomaniac
Copy link
Contributor

The fix for this is easy actually. We should just use classesDirs instead.

@tasomaniac
Copy link
Contributor

Silly me. I thought this was an issue. Didn't realize it was a PR 🤦‍♂️ haha 😂

I couldn't see why the CI was failing. There might be multiple usages of classesDir

@pablisco
Copy link
Contributor Author

pablisco commented Mar 5, 2019

Moved into #169 as part of the upgrade

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

Successfully merging this pull request may close these issues.

2 participants