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 duplicate declaration #5461

Merged
merged 1 commit into from Nov 22, 2018

Conversation

Projects
None yet
3 participants
@ahorek
Copy link
Contributor

ahorek commented Nov 20, 2018

[WARNING] Some problems were encountered while building the effective model for org.jruby:jruby-core:jar:9.2.5.0-SNAPSHOT
[WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)' must be unique but found duplicate declaration of plugin org.apache.felix:maven-bundle-plugin @ org.jruby:jruby-parent:9.2.5.0-SNAPSHOT, /jruby/pom.xml, line 338, column 17
[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.jruby:jruby-parent:pom:9.2.5.0-SNAPSHOT
[WARNING] 'build.pluginManagement.plugins.plugin.(groupId:artifactId)' must be unique but found duplicate declaration of plugin org.apache.felix:maven-bundle-plugin
[WARNING]
[WARNING] It is highly recommended to fix these problems because they threaten the stability of your build.
[WARNING]
[WARNING] For this reason, future Maven versions might no longer support building such malformed projects.

@ahorek ahorek force-pushed the ahorek:duplicate_declaration branch from afe43ea to 48e582a Nov 20, 2018

pavel

@ahorek ahorek force-pushed the ahorek:duplicate_declaration branch from 48e582a to 309c537 Nov 21, 2018

@headius headius added this to the JRuby 9.2.5.0 milestone Nov 22, 2018

@headius

This comment has been minimized.

Copy link
Member

headius commented Nov 22, 2018

Oh nice...seems green too. I'll proactively merge this and @mkristian can tell us if there's anything wrong.

@headius headius merged commit e57741e into jruby:master Nov 22, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@mkristian

This comment has been minimized.

Copy link
Member

mkristian commented Nov 22, 2018

@ahorek that is cool to get rid of these warnings ! thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.