Skip to content

RATIS-2424. Fix javadoc locale configuration for JDK 8 compatibility#1366

Merged
adoroszlai merged 1 commit intoapache:masterfrom
OneSizeFitsQuorum:RATIS-fix-javadoc-jdk8
Mar 8, 2026
Merged

RATIS-2424. Fix javadoc locale configuration for JDK 8 compatibility#1366
adoroszlai merged 1 commit intoapache:masterfrom
OneSizeFitsQuorum:RATIS-fix-javadoc-jdk8

Conversation

@OneSizeFitsQuorum
Copy link
Contributor

JDK 8's javadoc requires the `-locale` option to be the first
argument on the command line. maven-javadoc-plugin cannot
guarantee this parameter order.

This change uses JVM system properties instead:
- `-J-Duser.language=en`
- `-J-Duser.country=US`

This sets the locale at JVM startup time, before javadoc runs,
ensuring compatibility with both JDK 8 and JDK 11+.
Copy link
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

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

+1 the change looks good.

@adoroszlai adoroszlai merged commit 96a8402 into apache:master Mar 8, 2026
16 checks passed
@adoroszlai
Copy link
Contributor

Thanks @OneSizeFitsQuorum for the patch, @szetszwo for the review.

adoroszlai pushed a commit that referenced this pull request Mar 8, 2026
@OneSizeFitsQuorum OneSizeFitsQuorum deleted the RATIS-fix-javadoc-jdk8 branch March 9, 2026 01:53
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.

3 participants