PlexusConfiguration behavior in ToolchainConverter

• Patch Submitted:

  Yes

Hi,

when packaging the Toolchains plugin to Fedora, I saw that the ToolchainConverter class of the Toolchains plugin depends on the old behavior of the PlexusConfiguration class, i. e. the getValue() and getName() methods throwing a PlexusConfigurationException. However, the new behavior of the PlexusConfiguration class (as found in Sonatype's Sisu, whose goal is to replace Plexus) does not throw this exception, which causes the Toolchain plugin not to compile when using the new library.

The point of this bug is to consult with you moving from plexus-container-default to sisu-inject-plexus, thus removing the try/catch block in src/main/java/org/apache/maven/plugin/toolchain/ToolchainConverter.java, line 84. A patch to perform that action is attached.

Thank you, Tomas Radej