Skip to content

Conversation

@cstamas
Copy link
Member

@cstamas cstamas commented Oct 14, 2025

Maven embedder uses system default encoding, making builds non portable.

Fixes #11258

This MUST be emphasized in release notes, as issue describes, people may circumvented this by using -Dfile.encoding=... that will break if they did not use UTF8.

Maven embedder uses system default encoding, making builds non portable.

Fixes apache#11258

This MUST be emphasized in release notes, as issue describes, people
may circumvented this by using `-Dfile.encoding=...` that will
break if they did not use UTF8.
@cstamas cstamas added this to the 3.9.12 milestone Oct 14, 2025
@cstamas cstamas self-assigned this Oct 14, 2025
@cstamas cstamas added the enhancement New feature or request label Oct 14, 2025
@cstamas cstamas linked an issue Oct 14, 2025 that may be closed by this pull request
@cstamas
Copy link
Member Author

cstamas commented Oct 14, 2025

Nota bene: AFAIK on macos and (fairly modern) linux distros default encoding is UTF8 since a while, but not on Windows. Hence, this change, if affect anyone, is most probably those on OS not using UTF8 encoding already.

@cstamas cstamas marked this pull request as ready for review October 14, 2025 10:48
@slawekjaranowski
Copy link
Member

I'm not sure to adding a prefix like [maven-x.x.x] to PR title .... it will be in release notes, but we have a release notes per branch
maybe a label will be better for mark PR

@cstamas cstamas merged commit d47bdcc into apache:maven-3.9.x Oct 14, 2025
18 checks passed
@cstamas cstamas deleted the maven-3.9.x-config-files-utf8 branch October 14, 2025 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Maven uses system default encoding for maven.config

2 participants