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

New feature: usefinalname #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

albertdev
Copy link

This plugin already has lots of goodness, but there is one thing which I haven't seen in any Maven plugin to date: using the finalName of a copied artifact as destination file name so that the copy plugin doesn't need a verbose configuration to get pretty filenames.

A short introduction: Maven allows to set a "finalName" for an artifact in the build section. This name will be used when a project builds jars or other files in the target folder. When an artifact gets deployed though, it uses the GAV Maven coordinates (and it should, otherwise Maven couldn't find it). However, no plugins exist which can download the artifact, resolve its POM and use the finalName to rename the artifact to its "human readable" form.

This set of commits does all of this to get rid of the default "artifactId-version-classifier" scheme, at least if you put a "<useFinalName>true</useFinalName>" parameter in a CopyDependency or CopyResource block.

Bert Jacobs added 2 commits March 3, 2013 12:03
Maven allows to set a "finalName" - the name which will be used when a
project builds jars or other files in the /target folder.
This commit allows to set a <useFinalName/> parameter which will
load the POM of a dependency and use the finalName field from it
instead of the default "artifactId-version-classifier" scheme.
Add PomResolver to resolve parent POMs, build would otherwise fail.
@evgeny-goldin
Copy link
Owner

Many thanks for a detailed explanation, Bert! Will review it soon.

@albertdev
Copy link
Author

Seems Markdown eats unknown XML tags... I edited my comment, I meant a "<useFinalName>true</useFinalName>" parameter.

@ghost ghost assigned evgeny-goldin Mar 15, 2013
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