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

Fix RpmPlugin so that using the "with" method does not throw an exceptio... #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

melodious
Copy link

...n and can be used.

Added getters to the CopySpecImpl.WrapperCopySpec to allow the RPM plugin to support a CopySpec's "with" method. Aliased the Redline Enums and constants to the CopySpec class so that they are available in a generic CopySpec definition.

This pull request addresses this issue:
#39

@AlanKrueger
Copy link
Member

This has been deployed to Sonatype OSS snapshots as 1.5-SNAPSHOT https://oss.sonatype.org/content/repositories/snapshots/

@melodious
Copy link
Author

Alan,
Will you be merging these changes with the main for 1.5?

Thanks!

@quidryan
Copy link
Contributor

The idea of supporting "with" is great, but I'm concerned about the modification of the Gradle class WrapperCopySpec via metaclass. You're adding fields to EVERY CopySpec, even though they aren't relevant to everyone, and they could conflict with other plugins.

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.

3 participants