| org.apache.maven.project.DefaultMavenProjectBuilder |
Line |
|---|
| remove |
145 |
| make it a component |
166 |
| mkleint - use the (Container, Properties) constructor to make system properties embeddable |
271 |
| such a call in MavenMetadataSource too - packaging not really the intention of type |
371 |
| this is gross. Would like to give it the whole model, but maven-artifact shouldn't depend on that Can a maven-core implementation of the Artifact interface store it, and be used in the exceptions? |
579 |
| configurable actions dependant on status |
666 |
| re-enable [MNG-798/865] |
671 |
| not correct in some instances |
716 |
| the aRWR can get out of sync with project.model.repositories. We should do all the processing of profiles, etc on the models then recreate the aggregated sets at the end from the project repositories (they must still be created along the way so that parent poms can be discovered, however) Use a TreeSet to ensure ordering is retained |
772 |
| maybe not strictly correct, while we should enfore that packaging has a type handler of the same id, we don't |
1016 |
| these aren't taking active project artifacts into consideration in the reactor |
1084 |
| mkleint - use the (Container, Properties constructor to make system properties embeddable |
1175 |
| share with createPluginArtifacts? |
1686 |
| share with createPluginArtifacts? |
1732 |
| MNG-3731 |
1887 |
| org.apache.maven.project.MavenProject |
Line |
|---|
| let the scope handler deal with this |
555 |
| classpath check doesn't belong here - that's the other method |
574 |
| let the scope handler deal with this |
577 |
| let the scope handler deal with this |
603 |
| let the scope handler deal with this NOTE: [jc] scope == 'test' is the widest possible scope, so we don't really need to perform this check... if ( Artifact.SCOPE_TEST.equals( a.getScope() ) || Artifact.SCOPE_COMPILE.equals( a.getScope() ) || Artifact.SCOPE_RUNTIME.equals( a.getScope() ) ) { } |
637 |
| classpath check doesn't belong here - that's the other method |
663 |
| let the scope handler deal with this NOTE: [jc] scope == 'test' is the widest possible scope, so we don't really need to perform this check... if ( Artifact.SCOPE_TEST.equals( a.getScope() ) || Artifact.SCOPE_COMPILE.equals( a.getScope() ) || Artifact.SCOPE_RUNTIME.equals( a.getScope() ) ) { list.add( a ); } |
666 |
| let the scope handler deal with this NOTE: [jc] scope == 'test' is the widest possible scope, so we don't really need to perform this check... if ( Artifact.SCOPE_TEST.equals( a.getScope() ) || Artifact.SCOPE_COMPILE.equals( a.getScope() ) || Artifact.SCOPE_RUNTIME.equals( a.getScope() ) ) { } |
696 |
| let the scope handler deal with this |
731 |
| classpath check doesn't belong here - that's the other method |
754 |
| let the scope handler deal with this |
757 |
| let the scope handler deal with this |
782 |
| let the scope handler deal with this |
813 |
| classpath check doesn't belong here - that's the other method |
831 |
| let the scope handler deal with this |
834 |
| let the scope handler deal with this |
859 |
| this should not be allowed to be null. |
930 |
| org.apache.maven.project.MavenProjectDynamismTest |
Line |
|---|
| MNG-3731 assertTrue( build.getScriptSourceDirectory() + " doesn't start with base-path: " + basepath, build.getScriptSourceDirectory().startsWith( basepath ) ); |
72 |
| org.apache.maven.project.ModelUtils |
Line |
|---|
| it's pretty much a copy of {@link #mergePluginLists(PluginContainer, PluginContainer, boolean)} |
610 |
| org.apache.maven.project.ProjectSorter |
Line |
|---|
| Shouldn't we add an edge between the plugin and its dependency? Note that doing this may result in cycles...run ProjectSorterTest.testPluginDependenciesInfluenceSorting_DeclarationInParent() for more information, if you change this: |
173 |
| !![jc; 28-jul-2005] check this; if we're using '-r' and there are aggregator tasks, this will result in weirdness. |
220 |
| org.apache.maven.project.ProjectUtils |
Line |
|---|
| make this a map inside the factory instead, so no lookup needed |
77 |
| make this a map inside the factory instead, so no lookup needed |
109 |
| org.apache.maven.project.artifact.MavenMetadataSource |
Line |
|---|
| can we just modify the original? |
151 |
| this could come straight from the project, negating the need to set it in the project itself? |
312 |
| we could possibly use p.getDependencyArtifacts instead of this call, but they haven't been filtered or used the inherited scope (should that be passed to the buildFromRepository method above?) |
330 |
| org.apache.maven.project.inheritance.DefaultModelInheritanceAssembler |
Line |
|---|
| This should eventually be migrated to DefaultPathTranslator. |
474 |
| Move this to plexus-utils' PathTool. |
515 |
| org.apache.maven.project.inheritance.DefaultModelInheritanceAssemblerTest |
Line |
|---|
| this is probably what we should be appending... child.setUrl( "/child.dir" ); |
65 |
| test inheritence/super pom? List childExcludesTest = child.getBuild().getUnitTest().getExcludes(); |
346 |
| a lot easier if modello generated equals() :) |
496 |
| a lot easier if modello generated equals() :) |
512 |
| a lot easier if modello generated equals() :) |
527 |
| org.apache.maven.project.path.DefaultPathTranslator |
Line |
|---|
| MNG-3731 build.setScriptSourceDirectory( alignToBaseDirectory( build.getScriptSourceDirectory(), basedir ) ); |
61 |
