apache / apache/maven-toolchains-plugin

New select-jdk-toolchain goal should warn about empty config

Open Beginner friendly
#183 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Java
Stars
27
Forks
31
PR merge metrics
No merged PRs in 30d

Description

### Affected version

3.2.0

### Bug description

Old `toolchain` goal was warning about empty config: [ToolchainMojo.java:77](https://github.com/apache/maven-toolchains-plugin/blob/7318c8874d46417065139bd8a6ede47511526799/src/main/java/org/apache/maven/plugins/toolchain/ToolchainMojo.java#L77):

`getLog().warn("No toolchains requirements configured.");`

Maybe the new goal should do something similar. Right now it just silently exits.
Found it after mixing old goal configuration with the new select- goal.

Maybe it could be beneficial to parse old configuration in the new goal as a fallback?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the referenced ToolchainMojo.java warning and locate the implementation of the new select-jdk-toolchain goal. Compare how each goal handles an empty configuration, then determine whether the fix should only warn or also consider the suggested legacy-configuration fallback. Done means the silent exit is addressed consistently with the issue’s intended behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
66/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.