Skip to content

Fix release-profile duplicate sources jar after finos parent bump to 10 - #738

Merged
hugohills-regnosys merged 1 commit into
finos:dependabot/maven/org.finos-finos-10from
REGnosys:hugo/fix-release-sources-duplicate
Sep 15, 2026
Merged

hugohills-regnosys merged 1 commit into
finos:dependabot/maven/org.finos-finos-10from
REGnosys:hugo/fix-release-sources-duplicate

Conversation

@hugohills-regnosys

Copy link
Copy Markdown
Contributor

Summary

  • Bump org.finos:finos from 7 to 10 #732 (bump org.finos:finos from 7 to 10) fails the FINOS/rune-common Codefresh build, which runs mvn clean install ... -P release.
  • finos:finos 10 renamed its release profile's maven-source-plugin execution from attach-sources to release-attach-sources. Our own release profile in pom.xml disables that parent execution by overriding it by id (to avoid duplicating our own default/jar sources-attach execution) — after the bump, the id no longer matches, so the parent's execution ran unblocked alongside ours and both tried to attach a sources-classified artifact:
    [ERROR] Failed to execute goal org.apache.maven.plugins:maven-source-plugin:3.3.1:jar (default) on project serialization: Presumably you have configured maven-source-plugin to execute twice in your build. You have to configure a classifier for at least one of them.
    
  • Fix: rename our override to release-attach-sources to match the parent's new id.

This targets #732's own branch (dependabot/maven/org.finos-finos-10), not main — the fix only makes sense combined with the version-10 bump; merged standalone against the current parent (v7) it would reintroduce the same duplicate-artifact failure in reverse.

Test plan

  • mvn -P release -Dgpg.skip=true -pl common -am package — reproduced the failure before the fix, passed after.

finos:finos 10 renamed its release profile's maven-source-plugin
execution from "attach-sources" to "release-attach-sources". Our
override in the release profile (added to avoid the duplicate sources
jar this parent execution produces alongside our own "default"
execution) still targeted the old id, so after the bump the parent's
execution ran unblocked and duplicated `default`'s sources jar:

  [ERROR] Failed to execute goal
  org.apache.maven.plugins:maven-source-plugin:3.3.1:jar (default) on
  project serialization: Presumably you have configured
  maven-source-plugin to execute twice in your build. You have to
  configure a classifier for at least one of them.

Verified locally with:
  mvn -P release -Dgpg.skip=true -pl common -am package
@hugohills-regnosys
hugohills-regnosys merged commit f4a6456 into finos:dependabot/maven/org.finos-finos-10 Sep 15, 2026
6 checks passed
@hugohills-regnosys
hugohills-regnosys deleted the hugo/fix-release-sources-duplicate branch September 15, 2026 14:04
hugohills-regnosys added a commit that referenced this pull request Sep 15, 2026
* Bump org.finos:finos from 7 to 10

Bumps [org.finos:finos](https://github.com/finos/finos-parent-pom) from 7 to 10.
- [Release notes](https://github.com/finos/finos-parent-pom/releases)
- [Commits](https://github.com/finos/finos-parent-pom/commits)

---
updated-dependencies:
- dependency-name: org.finos:finos
  dependency-version: '10'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

* Match the finos parent's renamed release-sources execution id (#738)

finos:finos 10 renamed its release profile's maven-source-plugin
execution from "attach-sources" to "release-attach-sources". Our
override in the release profile (added to avoid the duplicate sources
jar this parent execution produces alongside our own "default"
execution) still targeted the old id, so after the bump the parent's
execution ran unblocked and duplicated `default`'s sources jar:

  [ERROR] Failed to execute goal
  org.apache.maven.plugins:maven-source-plugin:3.3.1:jar (default) on
  project serialization: Presumably you have configured
  maven-source-plugin to execute twice in your build. You have to
  configure a classifier for at least one of them.

Verified locally with:
  mvn -P release -Dgpg.skip=true -pl common -am package

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Hugo Hills <39260692+hugohills-regnosys@users.noreply.github.com>
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