Skip to content

Update dependencies - #207

Merged
cstamas merged 2 commits into
mainfrom
update-dependencies
May 20, 2025
Merged

cstamas merged 2 commits into
mainfrom
update-dependencies

Conversation

@cstamas

@cstamas cstamas commented May 19, 2025 •

Copy link
Copy Markdown
Member

Changes:

  • dropped plexus-xml, we use p-u 3.5.1 and no need for p-x
  • maven-common-artifacts-filters 3.4.0
  • junit 5.12.2

@cstamas cstamas self-assigned this May 19, 2025
@cstamas
cstamas marked this pull request as ready for review May 19, 2025 19:04
@kwin

kwin commented May 19, 2025 •

Copy link
Copy Markdown
Contributor

Usually in dynamic runtimes like OSGi you try to keep the dependency versions to the minimally required versions (as the import-package version ranges are calculated based on the compile classpath). In Maven context though the POM dependencies also define the runtime classpath (which should rather require latest&greatest). Not sure how to find middle ground and support older versions of deps at runtime outside Maven world….

@cstamas

cstamas commented May 19, 2025

Copy link
Copy Markdown
Member Author

Oh, so you say to leave things as is? is fine to me... maybe just up JUnit then?

Comment thread pom.xml Resolved
Comment thread pom.xml Resolved
Comment thread pom.xml Resolved
Comment thread pom.xml Resolved
Comment thread pom.xml Resolved
@sonarqubecloud

Copy link
Copy Markdown

@gnodet

gnodet commented May 19, 2025

Copy link
Copy Markdown
Contributor

Usually in dynamic runtimes like OSGi you try to keep the dependency versions to the minimally required versions (as the import-package version ranges are calculated based on the compile classpath). In Maven context though the POM dependencies also define the runtime classpath (which should rather require latest&greatest). Not sure how to find middle ground and support older versions of deps at runtime outside Maven world….

Ideally, dependencies would be upgraded to their latest version, and if that's a problem for the OSGi metadata, we can upgrade the range to use a lower bound.

@cstamas

cstamas commented May 19, 2025 •

Copy link
Copy Markdown
Member Author

Undone all, except for maven-common-artifact-filters and JUnit. Dropped plexus-xml as that one is not needed with plexus-util 3.x (how did that one got in at all?)

@cstamas
cstamas merged commit f92c669 into main May 20, 2025
@cstamas
cstamas deleted the update-dependencies branch May 20, 2025 09:37
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.

3 participants