Skip to content

bugfix: maven 4 consumers need sonatypeOssDistMgmtStagingUrl to be …#2420

Merged
rfelcman merged 2 commits intoeclipse-ee4j:masterfrom
flowlogix:fix-maven4-consumer
May 30, 2025
Merged

bugfix: maven 4 consumers need sonatypeOssDistMgmtStagingUrl to be …#2420
rfelcman merged 2 commits intoeclipse-ee4j:masterfrom
flowlogix:fix-maven4-consumer

Conversation

@lprimak
Copy link
Copy Markdown
Contributor

@lprimak lprimak commented May 29, 2025

…defined

Fixes #2416

Comment thread pom.xml Outdated
<build.type>SNAPSHOT</build.type>
<maven.build.timestamp.format>yyyyMMddHHmm</maven.build.timestamp.format>
<build.qualifier>v${maven.build.timestamp}</build.qualifier>
<sonatypeOssDistMgmtStagingUrl/>
Copy link
Copy Markdown
Contributor

@rfelcman rfelcman May 29, 2025

Choose a reason for hiding this comment

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

Please put there <sonatypeOssDistMgmtStagingUrl>https://jakarta.oss.sonatype.org/content/repositories/staging/</sonatypeOssDistMgmtStagingUrl> as this value is needed for snapshosts.
Without this builds will fails as You can see bellow.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yup, doesn't hurt. However, IMHO more appropriate place to put this is CI.

Copy link
Copy Markdown
Contributor

@rfelcman rfelcman left a comment

Choose a reason for hiding this comment

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

LGTM

@rfelcman rfelcman merged commit 6d1a171 into eclipse-ee4j:master May 30, 2025
6 checks passed
rfelcman pushed a commit that referenced this pull request May 30, 2025
…defined

* bugfix: maven 4 consumers need `sonatypeOssDistMgmtStagingUrl` to be defined
* chore: add actual values to `sonatypeOssDistMgmtStagingUrl`
[backport] #2420 to 4.x branch (#2421)
@lprimak lprimak deleted the fix-maven4-consumer branch May 30, 2025 13:23
andi-huber added a commit to apache/causeway that referenced this pull request Jun 30, 2025
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.

[Blocker] EclipseLink root POM contains unresolved variable breaking the build on Maven 4

2 participants