Maven 2.x Source Plugin

Test Source Jar has wrong name.

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 2.2
  • Labels:
    None
  • Number of attachments :
    1

Description

maven-jar-plugin generates:

module-1.0-tests.jar

maven-sources plugin generates:

module-1.0-test-sources.jar

I would expect:

module-1.0-tests-sources.jar

IntelliJ also expects this, and cannot bind to the test sources JAR.

Either change the classifier, or make it configurable.

Issue Links

duplicates

New Feature - A new feature of the product, which has yet to be developed. MSOURCES-50 Add optional 'classifier' for source artifact

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Arnaud Heritier added a comment -

I think that changing the classifier will have impacts on existing builds even if this one is erroneous. I would like to have the feedback of some others committers before trying to fix this one.

Show
Arnaud Heritier added a comment - I think that changing the classifier will have impacts on existing builds even if this one is erroneous. I would like to have the feedback of some others committers before trying to fix this one.
Hide
Permalink
Grégory Joseph added a comment -

Is there a related issue opened with Jetbrains, so we can follow on both sides and see who blames who ?

Show
Grégory Joseph added a comment - Is there a related issue opened with Jetbrains, so we can follow on both sides and see who blames who ?
Hide
Permalink
Sebastien Bonnet added a comment -

Could we have an optional parameter to specify the jar suffix instead of hardcoding it to "-sources" for the source:jar goal and "-test-sources" for the source:test-jar goal?
This way we avoid any backward compatibility issues.

Show
Sebastien Bonnet added a comment - Could we have an optional parameter to specify the jar suffix instead of hardcoding it to "-sources" for the source:jar goal and "-test-sources" for the source:test-jar goal? This way we avoid any backward compatibility issues.
Hide
Permalink
Grégory Joseph added a comment - - edited

Now using IntelliJ 9.0.2, and I can see it's expecting module-1.0-test-sources.jar, so maybe this issue can be close (despite the inconsistency)

edit: hmm no, it's still expecting m-tests-sources.jar, my bad

Show
Grégory Joseph added a comment - - edited Now using IntelliJ 9.0.2, and I can see it's expecting module-1.0-test-sources.jar, so maybe this issue can be close (despite the inconsistency) edit: hmm no, it's still expecting m-tests-sources.jar, my bad
Hide
Permalink
Grégory Joseph added a comment -

For the record, there IS a related issue on Jetbrains' side: http://youtrack.jetbrains.net/issue/IDEA-26928

Show
Grégory Joseph added a comment - For the record, there IS a related issue on Jetbrains' side: http://youtrack.jetbrains.net/issue/IDEA-26928
Hide
Permalink
Olivier Lamy added a comment -

classifier is now configurable.

Show
Olivier Lamy added a comment - classifier is now configurable.

People

  • Assignee:
    Olivier Lamy
    Reporter:
    Jason Zaugg
Vote (4)
Watch (6)

Dates

  • Created:
    Updated:
    Resolved: