Skip to content

Fix mvnw on macOS Sequoia #3676

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 19, 2025
Merged

Fix mvnw on macOS Sequoia #3676

merged 1 commit into from
May 19, 2025

Conversation

rschmitt
Copy link
Contributor

This change pulls in a fix from a newer version of the maven-wrapper-plugin that fixes a bug in verifying the distributionSha256Sum using the version of sha256sum that ships with macOS Sequoia.

See:

This change pulls in a fix from a newer version of the
maven-wrapper-plugin that fixes a bug in verifying the
distributionSha256Sum using the version of `sha256sum` that ships with
macOS Sequoia.

See:

* https://issues.apache.org/jira/browse/MWRAPPER-150
* apache/maven-wrapper@b2ff9cd
Copy link

Job Requested goals Build Tool Version Build Outcome Build Scan®
build-macos-latest clean install 3.9.8 Build Scan PUBLISHED
build-ubuntu-latest clean install 3.9.8 Build Scan PUBLISHED
build-windows-latest clean install 3.9.8 Build Scan PUBLISHED
Generated by gradle/develocity-actions

Copy link
Member

@vy vy left a comment

Choose a reason for hiding this comment

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

Let's not piss off contributors using macOS Sequoia, but let's also keep an eye on the upcoming Maven Wrapper 3.3.3 release.

@vy vy merged commit cd70cbf into apache:2.x May 19, 2025
11 checks passed
@github-project-automation github-project-automation bot moved this from To triage to Done in Log4j bug tracker May 19, 2025
@vy vy self-assigned this May 19, 2025
@vy vy added the build Affects the build system or other automation label May 19, 2025
@ppkarwasz
Copy link
Contributor

I would rather update the whole mvnw file to a new version.

@rschmitt
Copy link
Contributor Author

@ppkarwasz I tried that first. The fix isn't available yet in any released version of the wrapper plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Affects the build system or other automation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants