Skip to content

Fix core source-jar, added maven-source-plugin#283

Closed
spelbergnl wants to merge 1 commit intoeasymock:masterfrom
spelbergnl:Fix-core-source-jar
Closed

Fix core source-jar, added maven-source-plugin#283
spelbergnl wants to merge 1 commit intoeasymock:masterfrom
spelbergnl:Fix-core-source-jar

Conversation

@spelbergnl
Copy link

Hi, I noticed that the file easymock-4.3-sources.jar does not contain the sources for easymock itself. This merge request fixes this.

@henri-tremblay
Copy link
Contributor

Are you sure about that? I think it does, but only during the deployBuild (see the profile at the end of the pom.xml)

@henri-tremblay henri-tremblay self-assigned this May 31, 2022
@henri-tremblay
Copy link
Contributor

It seems you are right. However, I still struggle to understand why.

@henri-tremblay henri-tremblay added this to the 4.4 milestone Jun 1, 2022
@spelbergnl
Copy link
Author

spelbergnl commented Jun 1, 2022

The problem is that the maven-source-plugin is only specified in the <pluginManagement> section but not in the <plugins> section.

@henri-tremblay
Copy link
Contributor

No. Look at line 555 in the pom file

@spelbergnl
Copy link
Author

You are right, in the deployBuild profile.

What probably happened is that the 4.3 release was not created using that profile. Would there be a way to fix it and release 4.3.1 with sources included?

Also, what would be a way to prevent this from happening again?

@henri-tremblay
Copy link
Contributor

Good question. But I will investigate. And try to push the missing version. The same deployment process has been used for years without this issue. So I think Maven or the source plugin changed.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants