Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

downgrade maven-source-plugin to make release build pass #345

Merged
merged 1 commit into from
Jul 10, 2024
Merged

Conversation

tiana528
Copy link
Contributor

No description provided.

pom.xml Outdated
@@ -332,7 +332,7 @@
<execution>
<id>attach-sources</id>
<goals>
<goal>jar</goal>
<goal>jar-no-fork</goal>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Without this change, the mvn command for release fail.

mvn deploy  -DperformRelease=true

[INFO] --- maven-source-plugin:3.3.0:jar (attach-sources) @ td-client ---
[INFO] Building jar: /Users/wang.yan/work/td-client-java/target/td-client-1.1.1-sources.jar
[INFO]
[INFO] --- maven-source-plugin:3.3.0:jar-no-fork (attach-sources) @ td-client ---
[ERROR] We have duplicated artifacts attached.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  10.185 s
[INFO] Finished at: 2024-07-10T09:13:58+09:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-source-plugin:3.3.0:jar-no-fork (attach-sources) on project td-client: Presumably you have configured maven-source-plugn to execute twice times in your build. You have to configure a classifier for at least on of them. -> [Help 1]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@yuokada yuokada self-requested a review July 10, 2024 01:16
@tiana528 tiana528 changed the title make command pass 'mvn deploy -DperformRelease=true' downgrade maven-source-plugin to make release build pass Jul 10, 2024
@tiana528
Copy link
Contributor Author

Thank you!

@tiana528 tiana528 merged commit aae5ebd into master Jul 10, 2024
7 checks passed
@tiana528 tiana528 deleted the fix-build branch July 10, 2024 02:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants