Skip to content

bugfix exported bundle version in upload to maven repo#7102

Merged
peterkir merged 2 commits intomasterfrom
bugfix-exported-bundle-versions
Feb 26, 2026
Merged

bugfix exported bundle version in upload to maven repo#7102
peterkir merged 2 commits intomasterfrom
bugfix-exported-bundle-versions

Conversation

@peterkir
Copy link
Copy Markdown
Contributor

@peterkir peterkir commented Feb 17, 2026

This pull request adds comprehensive test coverage for the -snapshot instruction's effect on Maven POM version generation and implements the logic to apply the -snapshot transformation to the version in PomResource. The changes ensure that when building POM files, any SNAPSHOT qualifier in the version is replaced or removed according to the -snapshot property, and this is now thoroughly tested for various scenarios.

Test coverage improvements

  • Added multiple tests in MavenTest.java to verify that the -snapshot instruction correctly transforms SNAPSHOT versions in generated POM files, including cases with custom snapshot values, empty snapshots (removal), and qualifiers ending in -SNAPSHOT. These tests also cover integration with Maven repository layouts.

Core logic enhancement

  • Updated PomResource to apply the -snapshot property transformation to the version string when generating the POM, using the Builder.doSnapshot method. This ensures the generated POM version matches the intended release or snapshot semantics. [1] [2]

Comment thread biz.aQute.bndlib/src/aQute/bnd/maven/PomResource.java Outdated
@peterkir peterkir force-pushed the bugfix-exported-bundle-versions branch 5 times, most recently from 6ffeac8 to ef312ed Compare February 22, 2026 19:28
Signed-off-by: Peter Kirschner <peter@klib.io>
@peterkir peterkir force-pushed the bugfix-exported-bundle-versions branch from ef312ed to d4e9f83 Compare February 22, 2026 20:20
@peterkir peterkir merged commit 2b180c5 into master Feb 26, 2026
17 of 24 checks passed
@peterkir peterkir deleted the bugfix-exported-bundle-versions branch February 26, 2026 12:11
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.

2 participants