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

[NETBEANS-4928] EAR deployment considers outputFileNameMapping Maven EAR plugin property #4930

Merged

Conversation

OndroMih
Copy link
Contributor

@OndroMih OndroMih commented Nov 6, 2022

If the old property isn't found, try the new one. They can't both be specified, the Maven EAR plugin 3+ will refuse the build if the old property is specified. Therefore it doesn't matter which property is looked up first.

Fixes #4928.

@mbien mbien added Java EE/Jakarta EE [ci] enable enterprise job Maven [ci] enable "build tools" tests labels Nov 6, 2022
@mbien mbien added this to the NB17 milestone Nov 6, 2022
Copy link
Member

@mbien mbien left a comment

Choose a reason for hiding this comment

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

looks good. Thanks for the update.

@mbien mbien merged commit bb082de into apache:master Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Java EE/Jakarta EE [ci] enable enterprise job Maven [ci] enable "build tools" tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EAR deployment ignores a new property name introduced in Maven EAR plugin v3
2 participants