Skip to content

Change the maven plugin to allow use a source with classifier #11061

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

Closed
wants to merge 3 commits into from

Conversation

axtavt
Copy link

@axtavt axtavt commented Nov 17, 2017

Fixes #7659.

When classifier is specified, uses attached artifact with the same classifier as source, if present. Otherwise uses main artifact.

Fixed version of rejected pull request #9987.

@pivotal-issuemaster
Copy link

@axtavt Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@axtavt Thank you for signing the Contributor License Agreement!

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Nov 17, 2017
@philwebb philwebb added priority: normal type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Nov 17, 2017
@philwebb philwebb self-assigned this Nov 17, 2017
@snicoll snicoll changed the title Change the maven plugin to allow use a source with classifier. Change the maven plugin to allow use a source with classifier Dec 28, 2017
@snicoll snicoll self-assigned this May 15, 2018
@snicoll snicoll added this to the 2.1.0.M1 milestone May 15, 2018
snicoll pushed a commit that referenced this pull request May 15, 2018
Previously, the "classifier" attribute was only used to determine the
target classifier of the repackaged archive, always using the main
artifact as the source.

This commit changes the semantic of the attribute so that an existing
archive matching the "classifier" attribute can be used as source,
replacing the archive the same way the goal replaces the main archive
if no classifier is found.

If no artifact with the specified classifier exists, the repackaged
archive is still processed based on the main archive and attached to
the lifecycle using the value of the classifier attribute.

See gh-11061
@snicoll snicoll closed this in ed02d02 May 15, 2018
snicoll added a commit that referenced this pull request May 15, 2018
* pr/11061:
  Polish "Allow repackage maven goal to take a source classifier"
  Allow repackage maven goal to take a source classifier
@snicoll
Copy link
Member

snicoll commented May 15, 2018

@axtavt thank you so much for your first contribution to Spring Boot. This is now merged in master with a polish commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants