Maven 2 & 3

toolchains not found in extensions

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0.9, 2.1.0-M1
  • Fix Version/s: 3.0-alpha-3
  • Component/s: Settings
  • Labels:
    None
  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

There's currently no way to plugin in new toolchains without placing them in M2_HOME/lib. For wagons to be available in extensions, the extension manager explicitly registers them - so I propose to do the same in 2.1.0-M2 for toolchains, and only support the Java toolchains in 2.0.9.

Issue Links

is duplicated by

Bug - A problem which impairs or prevents the functions of the product. MNG-3429 Toolchain classes not being picked up in uber jar

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
is related to

Improvement - An improvement or enhancement to an existing feature or task. NMAVEN-147 DotnetToolchain Improvement, plus dotnet toolchain manager

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
John Casey added a comment -

As I documented here:

http://docs.codehaus.org/display/MAVEN/Maven+2.1+Plugin+and+Extension+Loading+Design

The extension-loading mechanism in maven 2.x is horrible, and prone to all sorts of weird behavioral changes depending on things like the number of artifacts in the extension's transitive dependency closure. IMO it needs to be replaced with something sensible. If we don't take the time to fix this mess, it's just going to extend the confusion surrounding extensions (which are problematic for a lot of users I've talked to) into the realm of toolchains as well.

I'm definitely -1 for patching the extension-loading code to support toolchains the same way wagons are supported now.

Show
John Casey added a comment - As I documented here: http://docs.codehaus.org/display/MAVEN/Maven+2.1+Plugin+and+Extension+Loading+Design The extension-loading mechanism in maven 2.x is horrible, and prone to all sorts of weird behavioral changes depending on things like the number of artifacts in the extension's transitive dependency closure. IMO it needs to be replaced with something sensible. If we don't take the time to fix this mess, it's just going to extend the confusion surrounding extensions (which are problematic for a lot of users I've talked to) into the realm of toolchains as well. I'm definitely -1 for patching the extension-loading code to support toolchains the same way wagons are supported now.
Hide
Permalink
Brian Fox added a comment -

We have a ton of issues in 2.0.10 with votes, lets bump this and move on with the release.

Show
Brian Fox added a comment - We have a ton of issues in 2.0.10 with votes, lets bump this and move on with the release.
Hide
Permalink
Brett Porter added a comment -

agreed. I'd originally intended this only for 2.1.0-M2, had just thought to investigate the impact of a potential fix before pulling the trigger on 2.0.10

Show
Brett Porter added a comment - agreed. I'd originally intended this only for 2.1.0-M2, had just thought to investigate the impact of a potential fix before pulling the trigger on 2.0.10
Hide
Permalink
John Casey added a comment -

Consolidating to 2.1.0-M1 so we can then rename to 2.1.0. We can weed out any issues we want to push to a later release from this set once we've done the consolidation.

Show
John Casey added a comment - Consolidating to 2.1.0-M1 so we can then rename to 2.1.0. We can weed out any issues we want to push to a later release from this set once we've done the consolidation.
Hide
Permalink
Brett Porter added a comment -

let's check if this works in 3.0's extension loading

Show
Brett Porter added a comment - let's check if this works in 3.0's extension loading

People

  • Assignee:
    Benjamin Bentmann
    Reporter:
    Brett Porter
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: