If you can create a drop-in replacement for the plexus-utils code go for it. I 
believe this exercise is of little value users and you run the real risk of 
introducing problems for no technical reason, but if you want to do the work 
that's your prerogative.

So much time was spent to preserve the existing signatures and behaviour in the 
container swap and this is not something to be taken lightly, and you should do 
what has been done in the past.

I think the path to do this if you want to get rid of the plexus-utils 
reference is to reduce the foot print to the classes being used in the core and 
then drop in that reduced JAR. Create a replacement with the code you've 
copied, reimplemented, improved yet binary compatible and test that for a 
while. Binary compatible in all respects, package and signature. And from there 
then make any subsequent changes.

I honestly am not overly concerned with plugins because they can be fixed 
relatively quickly, but doing low value changes in the core for swapping out 
one set of code for another is seriously playing with fire.

If you initially are intending to change code in the core to accommodate this 
change I see no technical reason for it, is dangerous and I'm -1.

On Aug 30, 2012, at 8:03 AM, Mark Struberg wrote:

> While digging thru the plexus-utils usage I wonder whether we should place 
> the maven-utils in maven-core or maven-shared.
> 
> maven-core doesn't yet have any maven-shared dependency it seems. Which means 
> if we like to use them in maven-core as well we should relocate 
> maven-shared-utils to the maven-3 core module.
> 
> wdyt?
> 
> LieGrue,
> strub
> 
> 
> 
> 
> ----- Original Message -----
>> From: Kristian Rosenvold <kristian.rosenv...@gmail.com>
>> To: Maven Developers List <dev@maven.apache.org>
>> Cc: 
>> Sent: Thursday, August 30, 2012 10:48 AM
>> Subject: Re: Removing unused code from maven-shared-utils
>> 
>> Well obviously given the current number of failing
>> tests,maven-shared-utils is going nowhere right now ;)
>> 
>> I will do a test-migration and remove unused code in
>> org.apache.maven.shared.utils.io before we release.
>> 
>> Kristian
>> 
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
>> For additional commands, e-mail: dev-h...@maven.apache.org
>> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
> 

Thanks,

Jason

----------------------------------------------------------
Jason van Zyl
Founder & CTO, Sonatype
Founder,  Apache Maven
http://twitter.com/jvanzyl
---------------------------------------------------------

What matters is not ideas, but the people who have them. Good people can fix 
bad ideas, but good ideas can't save bad people. 

 -- Paul Graham





Reply via email to