Re: Please avoid non-backward compatible API changes - Was: Re: svn commit: r617944 - in /maven/artifact/trunk/src: main/java/org/apache/maven/artifact/ main/java/org/apache/maven/artifact/deployer/ m

2008-02-03 Thread Jason van Zyl
On 3-Feb-08, at 3:45 AM, Milos Kleint wrote: Hello Jason, Please revert this change or make it backward comaptible. Removing getArtifactHandler() from Artifact interface might be conceptually right, but it breaks lots of thing. Not intentional, I ran everything on this side and the ITs so

Re: Please avoid non-backward compatible API changes - Was: Re: svn commit: r617944 - in /maven/artifact/trunk/src: main/java/org/apache/maven/artifact/ main/java/org/apache/maven/artifact/deployer/ m

2008-02-03 Thread Jason van Zyl
I rolled them back for the time being and put them on a branch as I'm headed out today. I'll put an aspect in, and then add a test for the artifact handler use in maven-artifact and make an IT like one of the plugins. I've got a few branches here in GIT, that didn't go in correctly but I