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

Replace buildDir with layout.buildDirectory and add an excludedModules property #621

Merged
merged 5 commits into from
Nov 16, 2023

Conversation

labkey-susanh
Copy link
Contributor

Rationale

In Gradle 8.3, the use of project.buildDir was deprecated in favor of project.layout.buildDirectory.

Related Pull Requests

Changes

  • Replace deprecated buildDir with layout.buildDirectory
  • Add support for new property excludeModules that allows specifying a full path to a module (or set of modules) to exclude

Comment on lines +110 to +113
if (hasProperty('excludedModules'))
{
excludedModules.addAll("${excludedModules}".split(","))
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could the BuildUtils.include* methods automatically check the excludedModules property instead of having to check it in every settings file?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried that, but this hasProperty method always returns null inside the Gradle plugin. Not sure why, but I didn't want to spend more time fighting it.

@labkey-susanh labkey-susanh merged commit 539249d into develop Nov 16, 2023
4 checks passed
@labkey-susanh labkey-susanh deleted the fb_buildDirBeGone branch November 16, 2023 16:18
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

Successfully merging this pull request may close these issues.

2 participants