Maven 2.x and 3.x Site Plugin

remove copy of plexus-utils sources from site plugin sources since it is a dependency

Details

  • Type: Task Task
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0-beta-6
  • Fix Version/s: 2.0
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

the dependency has been uncommented, but the source code copy hasn't been removed

classes copied are:

  • o.c.p.u.cli
  • o.c.p.u.interpolation
  • o.c.p.u.introspection
  • o.c.p.u.xml (partial: XML encoding support)

Issue Links

depends upon

Improvement - An improvement or enhancement to an existing feature or task. MNG-2892 Use shade to hide the use of plexus-utils internally so that plugins can use their own version

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
is related to

Sub-task - The sub-task of the issue MSITE-300 Using maven-shade-plugin instead of forking plexus-utils

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Bug - A problem which impairs or prevents the functions of the product. PLXUTILS-60 XmlStreamReader barfs at attribues named .*encoding

  • Blocker - Blocks development and/or testing work, production could not run
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Task - A task that needs to be done. MODELLO-110 remove copy of plexus-utils' XML encoding support sources

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Task - A task that needs to be done. MDEPLOY-70 remove copy of plexus-utils' XML encoding support sources

  • 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.

Task - A task that needs to be done. MINSTALL-46 remove copy of plexus-utils' XML encoding support sources

  • 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.

Task - A task that needs to be done. MRELEASE-316 remove copy of plexus-utils' XML encoding support sources

  • 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.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Dennis Lundberg added a comment -

Removing these files would mean that we have to bump the prerequisites to maven 2.0.6. Previous version of maven always use plexus-utils-1.1, even though we declare a dependency on plexus-utils-1.4.5.

I don't think we can set such a high prerequisite at this time, so I'd like to move this issue to a later release of the site-plugin.

Show
Dennis Lundberg added a comment - Removing these files would mean that we have to bump the prerequisites to maven 2.0.6. Previous version of maven always use plexus-utils-1.1, even though we declare a dependency on plexus-utils-1.4.5. I don't think we can set such a high prerequisite at this time, so I'd like to move this issue to a later release of the site-plugin.
Hide
Permalink
Herve Boutemy added a comment -

you're right, I didn't understand this consequence
no problem to let this issue open and wait for a later release

Show
Herve Boutemy added a comment - you're right, I didn't understand this consequence no problem to let this issue open and wait for a later release
Hide
Permalink
Dennis Lundberg added a comment -

Beacuse of MSITE-332, and the fix for PLXUTILS-60 in r662007, we need to depend on plexus-utils-1.5 or newer.

Show
Dennis Lundberg added a comment - Beacuse of MSITE-332, and the fix for PLXUTILS-60 in r662007, we need to depend on plexus-utils-1.5 or newer.
Hide
Permalink
Dennis Lundberg added a comment -

Fixed in r692655.

Show
Dennis Lundberg added a comment - Fixed in r692655 .

People

  • Assignee:
    Dennis Lundberg
    Reporter:
    Herve Boutemy
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: