Re: svn commit: r1685226 - in /maven/plugins/trunk/maven-assembly-plugin/src: main/java/org/apache/maven/plugin/assembly/artifact/ test/java/org/apache/maven/plugin/assembly/archive/phase/ test/java/o

2015-07-02 Thread Robert Scholte
I'm working on a DependencyCollector, so we need to to change the artifactId. They all depend on a RepositorySystem from any Aether, so maven-repository-system? Only one problem: it will probably be harder to find or to recognize why/when you need it. Robert Op Sat, 13 Jun 2015 22:02:44

Re: svn commit: r1685226 - in /maven/plugins/trunk/maven-assembly-plugin/src: main/java/org/apache/maven/plugin/assembly/artifact/ test/java/org/apache/maven/plugin/assembly/archive/phase/ test/java/o

2015-06-14 Thread Robert Scholte
Assumptions... http://mvnrepository.org/artifact/org.apache.maven.shared/maven-dependency-tree says enough usage. although I can't tell if it uses the (removed) tree package or the current graph package. Anyhow, since we're pushing this to 3.0 as well, I need to change the methods of

Re: svn commit: r1685226 - in /maven/plugins/trunk/maven-assembly-plugin/src: main/java/org/apache/maven/plugin/assembly/artifact/ test/java/org/apache/maven/plugin/assembly/archive/phase/ test/java/o

2015-06-13 Thread Robert Scholte
Hi Kristian, org.apache.maven.artifact.resolver.ArtifactResolver is part of maven-compat. I've fixed maven-artifact-transfer, which should contain the preferred ArtifactResolver. Let me know it that'll work for you. thanks, Robert Op Sat, 13 Jun 2015 11:28:03 +0200 schreef

Re: svn commit: r1685226 - in /maven/plugins/trunk/maven-assembly-plugin/src: main/java/org/apache/maven/plugin/assembly/artifact/ test/java/org/apache/maven/plugin/assembly/archive/phase/ test/java/o

2015-06-13 Thread Kristian Rosenvold
I looked at maven-artifact-transfer and the code seems to be focused on install/deploy. The description tag in the project also says An API to either install or deploy artifacts with Maven3 It would appear we should either change the description of the project (to include download) or create a

Re: svn commit: r1685226 - in /maven/plugins/trunk/maven-assembly-plugin/src: main/java/org/apache/maven/plugin/assembly/artifact/ test/java/org/apache/maven/plugin/assembly/archive/phase/ test/java/o

2015-06-13 Thread Robert Scholte
Hervé already worked on the graph-stuff. It's part of the shared maven-dependency-tree. Robert Op Sat, 13 Jun 2015 22:02:44 +0200 schreef Kristian Rosenvold kristian.rosenv...@gmail.com: +1 for changing the description. Maybe even the artifact id :) There is the small problem of the

Re: svn commit: r1685226 - in /maven/plugins/trunk/maven-assembly-plugin/src: main/java/org/apache/maven/plugin/assembly/artifact/ test/java/org/apache/maven/plugin/assembly/archive/phase/ test/java/o

2015-06-13 Thread Robert Scholte
There's no release yet, so we're even free to change/rename classes/methods/constructors etc. First intention was indeed install/deploy, but then I discovered I had to do more. So let's change the description and let this become that main aether-neutral project. Robert Op Sat, 13 Jun

Re: svn commit: r1685226 - in /maven/plugins/trunk/maven-assembly-plugin/src: main/java/org/apache/maven/plugin/assembly/artifact/ test/java/org/apache/maven/plugin/assembly/archive/phase/ test/java/o

2015-06-13 Thread Kristian Rosenvold
+1 for changing the description. Maybe even the artifact id :) There is the small problem of the complex dependency graph stuff; there does not appear to be a smart way to wrap the advanced functions. The issue is ProjectBuildingResult#getDependencyResolutionResult. There does not immediately

Re: svn commit: r1685226 - in /maven/plugins/trunk/maven-assembly-plugin/src: main/java/org/apache/maven/plugin/assembly/artifact/ test/java/org/apache/maven/plugin/assembly/archive/phase/ test/java/o

2015-06-13 Thread Kristian Rosenvold
Cool. I suppose not much other stuff uses that anyway :) K 2015-06-13 22:22 GMT+02:00 Robert Scholte rfscho...@apache.org: Hervé already worked on the graph-stuff. It's part of the shared maven-dependency-tree. Robert Op Sat, 13 Jun 2015 22:02:44 +0200 schreef Kristian Rosenvold