Skip to content

Add use-apache-staging profile#585

Open
slawekjaranowski wants to merge 1 commit into
apache:masterfrom
slawekjaranowski:staging-in-profile
Open

Add use-apache-staging profile#585
slawekjaranowski wants to merge 1 commit into
apache:masterfrom
slawekjaranowski:staging-in-profile

Conversation

@slawekjaranowski

Copy link
Copy Markdown
Member

Own setting.xml brak build on Jenkins - missing authorization for deployment.

Simply, we can use profile like we do for snapshot repository.

fix for #578

Own setting.xml brak build on Jenkins - missing authorization for deployment.

Simply, we can use profile like we do for snapshot repository.

fix for apache#578
@slawekjaranowski slawekjaranowski self-assigned this Jul 15, 2026
@slawekjaranowski slawekjaranowski added the build Pull requests that change the build process label Jul 15, 2026
@ascheman

Copy link
Copy Markdown
Contributor

Approve — nice, this fixes the root cause rather than papering over it.

The real problem in #578 was that -s .github/settings.xml replaced the ASF-managed settings.xml, dropping the deploy server credentials → "missing authorization for deployment". Moving the staging repo into a POM profile lets the default settings (with deploy auth) apply again, and activating it via .mvn/maven.config means it reaches every mvn invocation — including the wrapper bootstrap that a -s arg is easy to miss.

use-apache-staging mirrors use-apache-snapshots 1:1 (symmetric: releases on / snapshots off), and the cleanup is complete — no leftover references to the deleted settings.xml.

One thought: since this profile lives in the parent POM, downstream repos can now activate -Dapache.staging during simultaneous parent votes instead of carrying their own settings.xml. Might be worth a line in the docs/issue.

(Non-binding review — not a PMC member.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Pull requests that change the build process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants