Skip to content

Conversation

paulochang
Copy link

@paulochang paulochang commented Aug 6, 2025

The homebrew cask recipe installs the maven demon version 2.x by default, which expects maven 4 to be installed, is unstable and breaks most ci/cd systems. This behavior is non standard and goes against the homebrew cask recommendations (see https://docs.brew.sh/Acceptable-Casks#beta-unstable-development-nightly-or-legacy and https://docs.brew.sh/Versions#acceptable-versioned-formulae). As it is expected that users will install the mvnd recipe right after mvn, this will likely cause them to get unexpected errors.

This updates the readme to warn future users more explicitly about the non-standard behavior and that they will likely need to use the mvnd@1 alternative.

… recipe

The homebrew cask recipe installs the maven demon 2.x by default, which expects maven 4 to be installed. This updates the readme to warn future users that they will likely need to use the `mvnd@1` alternative.
@slachiewicz slachiewicz requested a review from cstamas August 22, 2025 19:05
@slachiewicz slachiewicz added the breaking Pull requests that break existing features label Aug 22, 2025
@elharo elharo changed the title Update README.adoc to warn about non-standard defaults on hombrew cask recipe Update README.adoc to warn about non-standard defaults on homebrew cask recipe Sep 17, 2025
----

Note: There are two formulae: the `mvnd` that install latest, and `mvnd@1` that installs 1.x line.
Note: There are two formulae. Contrary to the recommendation by the [homebrew cask standards](https://docs.brew.sh/Acceptable-Casks#beta-unstable-development-nightly-or-legacy), the `mvnd` installs the preview 2.x version , and `mvnd@1` installs the stable 1.x version.
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove "Contrary to the recommendation by the homebrew cask standards" as it's not relevant to the target audience, though you can file a bug for that if you want to reverse this.

the mvnd --> mvnd

@cstamas cstamas mentioned this pull request Sep 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Pull requests that break existing features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants