Skip to content

Fix maven-source-plugin duplicate artifact issue #24

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

Merged
merged 2 commits into from
Jun 9, 2025

Conversation

basmussen
Copy link
Contributor

Description

This PR fixes the build failure in the dependabot PR #16 for updating maven-source-plugin from 2.4 to 3.3.1.

Problem

The original PR was failing with the error:
\\

Root Cause

The pom.xml had a redundant execution configuration with an empty \ that conflicted with the \ execution already defined in the parent POM.

Solution

Removed the redundant execution configuration, keeping only the version specification. The parent POM already defines the proper execution configuration.

Testing

  • ✅ Tested with Java 8
  • ✅ Tested with Java 11
  • ✅ Tested with Java 17
  • ✅ Tested with Java 21

All tests pass successfully with the maven-source-plugin 3.3.1.

Fixes the issue in #16

dependabot bot and others added 2 commits June 9, 2025 16:04
Bumps [org.apache.maven.plugins:maven-source-plugin](https://github.com/apache/maven-source-plugin) from 2.4 to 3.3.1.
- [Release notes](https://github.com/apache/maven-source-plugin/releases)
- [Commits](apache/maven-source-plugin@maven-source-plugin-2.4...maven-source-plugin-3.3.1)

---
updated-dependencies:
- dependency-name: org.apache.maven.plugins:maven-source-plugin
  dependency-version: 3.3.1
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
The maven-source-plugin is already properly configured in the parent POM,
so there's no need to define it in the child POM. This removes the
redundant configuration that was causing duplicate artifact issues.
@basmussen basmussen force-pushed the pr-16-maven-source-plugin branch from 6d410c0 to 54e9388 Compare June 9, 2025 17:10
@basmussen basmussen merged commit a69b092 into develop Jun 9, 2025
5 checks passed
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.

1 participant