You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by nicolas de loof <ni...@apache.org> on 2009/05/02 08:44:07 UTC
Re: svn commit: r770570 - in /maven/components/branches/maven-2.2.x:
maven-artifact/src/main/java/org/apache/maven/artifact/ maven-model/
maven-profile/ maven-project/src/main/java/org/apache/maven/project/
maven-repository-metadata/ maven-settings/
Do we prefer plugin developers to use List<?> and get ClassCastExceptions at
runtime or sue type-safe collections to help them create stronger code ?Anyway,
few plugins allready use Java5. Java 1.4 based one will not be broken as the
generics signature is only a compile-time check.
2009/5/1 Brian Fox <br...@infinity.nu>
> I'm not sure if this is in scope of what John is trying to do wrt to 2.2.
>
>
> Jason van Zyl wrote:
>
>> I don't believe anyone actually agreed to this yet. Are you sure this is
>> not going to cause problems for users?
>>
>> On 1-May-09, at 1:04 AM, nicolas@apache.org wrote:
>>
>> Author: nicolas
>>> Date: Fri May 1 08:04:48 2009
>>> New Revision: 770570
>>>
>>> URL: http://svn.apache.org/viewvc?rev=770570&view=rev
>>> Log:
>>> use java5 syntax.
>>> -> set modello to generate java5 syntax
>>> -> MavenProject updated to use generics on collections & methods for
>>> better type safety
>>>
>>> Modified:
>>>
>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>>
>>> maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>> maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>
>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>>
>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
>>> maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>> maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>
>>> Modified:
>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>>
>>> URL:
>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java?rev=770570&r1=770569&r2=770570&view=diff
>>> ==============================================================================
>>>
>>> ---
>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>> (original)
>>> +++
>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>> Fri May 1 08:04:48 2009
>>> @@ -86,13 +86,13 @@
>>> baseVersion;
>>> }
>>>
>>> - public static Map artifactMapByVersionlessId( Collection artifacts )
>>> + public static Map<String, Artifact> artifactMapByVersionlessId(
>>> Collection<Artifact> artifacts )
>>> {
>>> - Map artifactMap = new LinkedHashMap();
>>> + Map<String, Artifact> artifactMap = new LinkedHashMap<String,
>>> Artifact>();
>>>
>>> if ( artifacts != null )
>>> {
>>> - for ( Iterator it = artifacts.iterator(); it.hasNext(); )
>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>> it.hasNext(); )
>>> {
>>> Artifact artifact = (Artifact) it.next();
>>>
>>>
>>> Modified: maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>> URL:
>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-model/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>> ==============================================================================
>>>
>>> --- maven/components/branches/maven-2.2.x/maven-model/pom.xml (original)
>>> +++ maven/components/branches/maven-2.2.x/maven-model/pom.xml Fri May 1
>>> 08:04:48 2009
>>> @@ -47,6 +47,7 @@
>>> <models>
>>> <model>src/main/mdo/maven.mdo</model>
>>> </models>
>>> + <useJava5>true</useJava5>
>>> </configuration>
>>> </plugin>
>>> <plugin>
>>>
>>> Modified: maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>> URL:
>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-profile/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>> ==============================================================================
>>>
>>> --- maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>> (original)
>>> +++ maven/components/branches/maven-2.2.x/maven-profile/pom.xml Fri May
>>> 1 08:04:48 2009
>>> @@ -54,6 +54,7 @@
>>> <models>
>>> <model>src/main/mdo/profiles.mdo</model>
>>> </models>
>>> + <useJava5>true</useJava5>
>>> </configuration>
>>> </plugin>
>>> </plugins>
>>>
>>> Modified:
>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>>
>>> URL:
>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java?rev=770570&r1=770569&r2=770570&view=diff
>>> ==============================================================================
>>>
>>> ---
>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>> (original)
>>> +++
>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>> Fri May 1 08:04:48 2009
>>> @@ -36,10 +36,11 @@
>>> import org.apache.maven.artifact.Artifact;
>>> import org.apache.maven.artifact.ArtifactUtils;
>>> import org.apache.maven.artifact.DependencyResolutionRequiredException;
>>> -import org.apache.maven.artifact.versioning.ManagedVersionMap;
>>> import org.apache.maven.artifact.factory.ArtifactFactory;
>>> import org.apache.maven.artifact.repository.ArtifactRepository;
>>> import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
>>> +import org.apache.maven.artifact.versioning.ArtifactVersion;
>>> +import org.apache.maven.artifact.versioning.ManagedVersionMap;
>>> import org.apache.maven.model.Build;
>>> import org.apache.maven.model.CiManagement;
>>> import org.apache.maven.model.Contributor;
>>> @@ -47,6 +48,7 @@
>>> import org.apache.maven.model.DependencyManagement;
>>> import org.apache.maven.model.Developer;
>>> import org.apache.maven.model.DistributionManagement;
>>> +import org.apache.maven.model.Extension;
>>> import org.apache.maven.model.IssueManagement;
>>> import org.apache.maven.model.License;
>>> import org.apache.maven.model.MailingList;
>>> @@ -59,6 +61,7 @@
>>> import org.apache.maven.model.ReportPlugin;
>>> import org.apache.maven.model.ReportSet;
>>> import org.apache.maven.model.Reporting;
>>> +import org.apache.maven.model.Repository;
>>> import org.apache.maven.model.Resource;
>>> import org.apache.maven.model.Scm;
>>> import org.apache.maven.model.io.xpp3.MavenXpp3Writer;
>>> @@ -96,60 +99,60 @@
>>>
>>> private File file;
>>>
>>> - private Set artifacts;
>>> + private Set<Artifact> artifacts;
>>>
>>> private Artifact parentArtifact;
>>>
>>> - private Set pluginArtifacts;
>>> + private Set<Artifact> pluginArtifacts;
>>>
>>> - private List remoteArtifactRepositories;
>>> + private List<ArtifactRepository> remoteArtifactRepositories;
>>>
>>> - private List collectedProjects = Collections.EMPTY_LIST;
>>> + private List<MavenProject> collectedProjects =
>>> Collections.<MavenProject> emptyList();
>>>
>>> - private List attachedArtifacts;
>>> + private List<Artifact> attachedArtifacts;
>>>
>>> private MavenProject executionProject;
>>>
>>> - private List compileSourceRoots = new ArrayList();
>>> + private List<String> compileSourceRoots = new ArrayList<String>();
>>>
>>> - private List testCompileSourceRoots = new ArrayList();
>>> + private List<String> testCompileSourceRoots = new
>>> ArrayList<String>();
>>>
>>> - private List scriptSourceRoots = new ArrayList();
>>> + private List<String> scriptSourceRoots = new ArrayList<String>();
>>>
>>> - private List pluginArtifactRepositories;
>>> + private List<ArtifactRepository> pluginArtifactRepositories;
>>>
>>> private ArtifactRepository releaseArtifactRepository;
>>>
>>> private ArtifactRepository snapshotArtifactRepository;
>>>
>>> - private List activeProfiles = new ArrayList();
>>> + private List<String> activeProfiles = new ArrayList<String>();
>>>
>>> - private Set dependencyArtifacts;
>>> + private Set<Artifact> dependencyArtifacts;
>>>
>>> private Artifact artifact;
>>>
>>> // calculated.
>>> - private Map artifactMap;
>>> + private Map<String, Artifact> artifactMap;
>>>
>>> private Model originalModel;
>>>
>>> - private Map pluginArtifactMap;
>>> + private Map<String, Artifact> pluginArtifactMap;
>>>
>>> - private Set reportArtifacts;
>>> + private Set<Artifact> reportArtifacts;
>>>
>>> - private Map reportArtifactMap;
>>> + private Map<String, Artifact> reportArtifactMap;
>>>
>>> - private Set extensionArtifacts;
>>> + private Set<Artifact> extensionArtifacts;
>>>
>>> - private Map extensionArtifactMap;
>>> + private Map<String, Artifact> extensionArtifactMap;
>>>
>>> - private Map managedVersionMap;
>>> + private Map<String, ArtifactVersion> managedVersionMap;
>>>
>>> - private Map projectReferences = new HashMap();
>>> + private Map<String, MavenProject> projectReferences = new
>>> HashMap<String, MavenProject>();
>>>
>>> private boolean executionRoot;
>>>
>>> - private Map moduleAdjustments;
>>> + private Map<String, String> moduleAdjustments;
>>>
>>> private File basedir;
>>>
>>> @@ -245,23 +248,23 @@
>>> if ( project.getAttachedArtifacts() != null )
>>> {
>>> // clone properties modifyable by plugins in a forked
>>> lifecycle
>>> - setAttachedArtifacts( new ArrayList(
>>> project.getAttachedArtifacts() ) );
>>> + setAttachedArtifacts( new ArrayList<Artifact>(
>>> project.getAttachedArtifacts() ) );
>>> }
>>>
>>> if ( project.getCompileSourceRoots() != null )
>>> {
>>> // clone source roots
>>> - setCompileSourceRoots( ( new ArrayList(
>>> project.getCompileSourceRoots() ) ) );
>>> + setCompileSourceRoots( ( new ArrayList<String>(
>>> project.getCompileSourceRoots() ) ) );
>>> }
>>>
>>> if ( project.getTestCompileSourceRoots() != null )
>>> {
>>> - setTestCompileSourceRoots( ( new ArrayList(
>>> project.getTestCompileSourceRoots() ) ) );
>>> + setTestCompileSourceRoots( ( new ArrayList<String>(
>>> project.getTestCompileSourceRoots() ) ) );
>>> }
>>>
>>> if ( project.getScriptSourceRoots() != null )
>>> {
>>> - setScriptSourceRoots( ( new ArrayList(
>>> project.getScriptSourceRoots() ) ) );
>>> + setScriptSourceRoots( ( new ArrayList<String>(
>>> project.getScriptSourceRoots() ) ) );
>>> }
>>>
>>> setModel( ( ModelUtils.cloneModel( project.getModel() ) ) );
>>> @@ -298,25 +301,25 @@
>>> setDynamicBuild( ModelUtils.cloneBuild(
>>> project.getDynamicBuild() ) );
>>> setOriginalInterpolatedBuild( ModelUtils.cloneBuild(
>>> project.getOriginalInterpolatedBuild() ) );
>>>
>>> - List dynamicRoots = project.getDynamicCompileSourceRoots();
>>> + List<String> dynamicRoots =
>>> project.getDynamicCompileSourceRoots();
>>> if ( dynamicRoots != null )
>>> {
>>> - setDynamicCompileSourceRoots( new ArrayList(
>>> dynamicRoots ) );
>>> - setOriginalInterpolatedCompileSourceRoots( new
>>> ArrayList( project.getOriginalInterpolatedCompileSourceRoots() ) );
>>> + setDynamicCompileSourceRoots( new ArrayList<String>(
>>> dynamicRoots ) );
>>> + setOriginalInterpolatedCompileSourceRoots( new
>>> ArrayList<String>( project.getOriginalInterpolatedCompileSourceRoots() ) );
>>> }
>>>
>>> dynamicRoots = project.getDynamicTestCompileSourceRoots();
>>> if ( dynamicRoots != null )
>>> {
>>> - setDynamicTestCompileSourceRoots( new ArrayList(
>>> dynamicRoots ) );
>>> - setOriginalInterpolatedTestCompileSourceRoots( new
>>> ArrayList( project.getOriginalInterpolatedTestCompileSourceRoots() ) );
>>> + setDynamicTestCompileSourceRoots( new ArrayList<String>(
>>> dynamicRoots ) );
>>> + setOriginalInterpolatedTestCompileSourceRoots( new
>>> ArrayList<String>( project.getOriginalInterpolatedTestCompileSourceRoots() )
>>> );
>>> }
>>>
>>> dynamicRoots = project.getDynamicScriptSourceRoots();
>>> if ( dynamicRoots != null )
>>> {
>>> - setDynamicScriptSourceRoots( new ArrayList( dynamicRoots
>>> ) );
>>> - setOriginalInterpolatedScriptSourceRoots( new ArrayList(
>>> project.getOriginalInterpolatedScriptSourceRoots() ) );
>>> + setDynamicScriptSourceRoots( new ArrayList<String>(
>>> dynamicRoots ) );
>>> + setOriginalInterpolatedScriptSourceRoots( new
>>> ArrayList<String>( project.getOriginalInterpolatedScriptSourceRoots() ) );
>>> }
>>> }
>>>
>>> @@ -347,12 +350,12 @@
>>>
>>> if ( moduleAdjustments == null )
>>> {
>>> - moduleAdjustments = new HashMap();
>>> + moduleAdjustments = new HashMap<String, String>();
>>>
>>> - List modules = getModules();
>>> + List<String> modules = getModules();
>>> if ( modules != null )
>>> {
>>> - for ( Iterator it = modules.iterator(); it.hasNext(); )
>>> + for ( Iterator<String> it = modules.iterator();
>>> it.hasNext(); )
>>> {
>>> String modulePath = (String) it.next();
>>> String moduleName = modulePath;
>>> @@ -415,12 +418,12 @@
>>> this.parent = parent;
>>> }
>>>
>>> - public void setRemoteArtifactRepositories( List
>>> remoteArtifactRepositories )
>>> + public void setRemoteArtifactRepositories( List<ArtifactRepository>
>>> remoteArtifactRepositories )
>>> {
>>> this.remoteArtifactRepositories = remoteArtifactRepositories;
>>> }
>>>
>>> - public List getRemoteArtifactRepositories()
>>> + public List<ArtifactRepository> getRemoteArtifactRepositories()
>>> {
>>> return remoteArtifactRepositories;
>>> }
>>> @@ -460,12 +463,12 @@
>>> return basedir;
>>> }
>>>
>>> - public void setDependencies( List dependencies )
>>> + public void setDependencies( List<Dependency> dependencies )
>>> {
>>> getModel().setDependencies( dependencies );
>>> }
>>>
>>> - public List getDependencies()
>>> + public List<Dependency> getDependencies()
>>> {
>>> return getModel().getDependencies();
>>> }
>>> @@ -524,29 +527,29 @@
>>> }
>>> }
>>>
>>> - public List getCompileSourceRoots()
>>> + public List<String> getCompileSourceRoots()
>>> {
>>> return compileSourceRoots;
>>> }
>>>
>>> - public List getScriptSourceRoots()
>>> + public List<String> getScriptSourceRoots()
>>> {
>>> return scriptSourceRoots;
>>> }
>>>
>>> - public List getTestCompileSourceRoots()
>>> + public List<String> getTestCompileSourceRoots()
>>> {
>>> return testCompileSourceRoots;
>>> }
>>>
>>> - public List getCompileClasspathElements()
>>> + public List<String> getCompileClasspathElements()
>>> throws DependencyResolutionRequiredException
>>> {
>>> - List list = new ArrayList( getArtifacts().size() );
>>> + List<String> list = new ArrayList<String>( getArtifacts().size()
>>> );
>>>
>>> list.add( getBuild().getOutputDirectory() );
>>>
>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>> + for ( Iterator<?> i = getArtifacts().iterator(); i.hasNext(); )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -563,11 +566,11 @@
>>> return list;
>>> }
>>>
>>> - public List getCompileArtifacts()
>>> + public List<Artifact> getCompileArtifacts()
>>> {
>>> - List list = new ArrayList( getArtifacts().size() );
>>> + List<Artifact> list = new ArrayList<Artifact>(
>>> getArtifacts().size() );
>>>
>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>> i.hasNext(); )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -585,18 +588,18 @@
>>> return list;
>>> }
>>>
>>> - public List getCompileDependencies()
>>> + public List<Dependency> getCompileDependencies()
>>> {
>>> - Set artifacts = getArtifacts();
>>> + Set<Artifact> artifacts = getArtifacts();
>>>
>>> if ( artifacts == null || artifacts.isEmpty() )
>>> {
>>> - return Collections.EMPTY_LIST;
>>> + return Collections.<Dependency> emptyList();
>>> }
>>>
>>> - List list = new ArrayList( artifacts.size() );
>>> + List<Dependency> list = new ArrayList<Dependency>(
>>> artifacts.size() );
>>>
>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>> i.hasNext(); )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -619,16 +622,16 @@
>>> return list;
>>> }
>>>
>>> - public List getTestClasspathElements()
>>> + public List<String> getTestClasspathElements()
>>> throws DependencyResolutionRequiredException
>>> {
>>> - List list = new ArrayList( getArtifacts().size() + 1 );
>>> + List<String> list = new ArrayList<String>( getArtifacts().size()
>>> + 1 );
>>>
>>> list.add( getBuild().getTestOutputDirectory() );
>>>
>>> list.add( getBuild().getOutputDirectory() );
>>>
>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>> i.hasNext(); )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -652,11 +655,11 @@
>>> return list;
>>> }
>>>
>>> - public List getTestArtifacts()
>>> + public List<Artifact> getTestArtifacts()
>>> {
>>> - List list = new ArrayList( getArtifacts().size() );
>>> + List<Artifact> list = new ArrayList<Artifact>(
>>> getArtifacts().size() );
>>>
>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>> i.hasNext(); )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -678,18 +681,18 @@
>>> return list;
>>> }
>>>
>>> - public List getTestDependencies()
>>> + public List<Dependency> getTestDependencies()
>>> {
>>> - Set artifacts = getArtifacts();
>>> + Set<Artifact> artifacts = getArtifacts();
>>>
>>> if ( artifacts == null || artifacts.isEmpty() )
>>> {
>>> - return Collections.EMPTY_LIST;
>>> + return Collections.emptyList();
>>> }
>>>
>>> - List list = new ArrayList( artifacts.size() );
>>> + List<Dependency> list = new ArrayList<Dependency>(
>>> artifacts.size() );
>>>
>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>> i.hasNext(); )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -715,14 +718,14 @@
>>> return list;
>>> }
>>>
>>> - public List getRuntimeClasspathElements()
>>> + public List<String> getRuntimeClasspathElements()
>>> throws DependencyResolutionRequiredException
>>> {
>>> - List list = new ArrayList( getArtifacts().size() + 1 );
>>> + List<String> list = new ArrayList<String>( getArtifacts().size()
>>> + 1 );
>>>
>>> list.add( getBuild().getOutputDirectory() );
>>>
>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>> i.hasNext(); )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -743,11 +746,11 @@
>>> return list;
>>> }
>>>
>>> - public List getRuntimeArtifacts()
>>> + public List<Artifact> getRuntimeArtifacts()
>>> {
>>> - List list = new ArrayList( getArtifacts().size() );
>>> + List<Artifact> list = new ArrayList<Artifact>(
>>> getArtifacts().size() );
>>>
>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>> i.hasNext(); )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -764,18 +767,18 @@
>>> return list;
>>> }
>>>
>>> - public List getRuntimeDependencies()
>>> + public List<Dependency> getRuntimeDependencies()
>>> {
>>> - Set artifacts = getArtifacts();
>>> + Set<Artifact> artifacts = getArtifacts();
>>>
>>> if ( artifacts == null || artifacts.isEmpty() )
>>> {
>>> - return Collections.EMPTY_LIST;
>>> + return Collections.<Dependency> emptyList();
>>> }
>>>
>>> - List list = new ArrayList( artifacts.size() );
>>> + List<Dependency> list = new ArrayList<Dependency>(
>>> artifacts.size() );
>>>
>>> - for ( Iterator i = artifacts.iterator(); i.hasNext(); )
>>> + for ( Iterator<Artifact> i = artifacts.iterator(); i.hasNext();
>>> )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -797,14 +800,14 @@
>>> return list;
>>> }
>>>
>>> - public List getSystemClasspathElements()
>>> + public List<String> getSystemClasspathElements()
>>> throws DependencyResolutionRequiredException
>>> {
>>> - List list = new ArrayList( getArtifacts().size() );
>>> + List<String> list = new ArrayList<String>( getArtifacts().size()
>>> );
>>>
>>> list.add( getBuild().getOutputDirectory() );
>>>
>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>> i.hasNext(); )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -820,11 +823,11 @@
>>> return list;
>>> }
>>>
>>> - public List getSystemArtifacts()
>>> + public List<Artifact> getSystemArtifacts()
>>> {
>>> - List list = new ArrayList( getArtifacts().size() );
>>> + List<Artifact> list = new ArrayList<Artifact>(
>>> getArtifacts().size() );
>>>
>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>> i.hasNext(); )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -841,18 +844,18 @@
>>> return list;
>>> }
>>>
>>> - public List getSystemDependencies()
>>> + public List<Dependency> getSystemDependencies()
>>> {
>>> - Set artifacts = getArtifacts();
>>> + Set<Artifact> artifacts = getArtifacts();
>>>
>>> if ( artifacts == null || artifacts.isEmpty() )
>>> {
>>> - return Collections.EMPTY_LIST;
>>> + return Collections.<Dependency> emptyList();
>>> }
>>>
>>> - List list = new ArrayList( artifacts.size() );
>>> + List<Dependency> list = new ArrayList<Dependency>(
>>> artifacts.size() );
>>>
>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>> i.hasNext(); )
>>> {
>>> Artifact a = (Artifact) i.next();
>>>
>>> @@ -1050,12 +1053,12 @@
>>> return getModel().getScm();
>>> }
>>>
>>> - public void setMailingLists( List mailingLists )
>>> + public void setMailingLists( List<MailingList> mailingLists )
>>> {
>>> getModel().setMailingLists( mailingLists );
>>> }
>>>
>>> - public List getMailingLists()
>>> + public List<?> getMailingLists()
>>> {
>>> return getModel().getMailingLists();
>>> }
>>> @@ -1065,12 +1068,12 @@
>>> getModel().addMailingList( mailingList );
>>> }
>>>
>>> - public void setDevelopers( List developers )
>>> + public void setDevelopers( List<Developer> developers )
>>> {
>>> getModel().setDevelopers( developers );
>>> }
>>>
>>> - public List getDevelopers()
>>> + public List<Developer> getDevelopers()
>>> {
>>> return getModel().getDevelopers();
>>> }
>>> @@ -1080,12 +1083,12 @@
>>> getModel().addDeveloper( developer );
>>> }
>>>
>>> - public void setContributors( List contributors )
>>> + public void setContributors( List<Contributor> contributors )
>>> {
>>> getModel().setContributors( contributors );
>>> }
>>>
>>> - public List getContributors()
>>> + public List<Contributor> getContributors()
>>> {
>>> return getModel().getContributors();
>>> }
>>> @@ -1105,12 +1108,12 @@
>>> return getModelBuild();
>>> }
>>>
>>> - public List getResources()
>>> + public List<Resource> getResources()
>>> {
>>> return getBuild().getResources();
>>> }
>>>
>>> - public List getTestResources()
>>> + public List<Resource> getTestResources()
>>> {
>>> return getBuild().getTestResources();
>>> }
>>> @@ -1135,12 +1138,12 @@
>>> return getModel().getReporting();
>>> }
>>>
>>> - public void setLicenses( List licenses )
>>> + public void setLicenses( List<License> licenses )
>>> {
>>> getModel().setLicenses( licenses );
>>> }
>>>
>>> - public List getLicenses()
>>> + public List<License> getLicenses()
>>> {
>>> return getModel().getLicenses();
>>> }
>>> @@ -1150,7 +1153,7 @@
>>> getModel().addLicense( license );
>>> }
>>>
>>> - public void setArtifacts( Set artifacts )
>>> + public void setArtifacts( Set<Artifact> artifacts )
>>> {
>>> this.artifacts = artifacts;
>>>
>>> @@ -1165,12 +1168,12 @@
>>> * @return {@link Set} < {@link Artifact} >
>>> * @see #getDependencyArtifacts() to get only direct dependencies
>>> */
>>> - public Set getArtifacts()
>>> + public Set<Artifact> getArtifacts()
>>> {
>>> - return artifacts == null ? Collections.EMPTY_SET : artifacts;
>>> + return artifacts == null ? Collections.<Artifact> emptySet() :
>>> artifacts;
>>> }
>>>
>>> - public Map getArtifactMap()
>>> + public Map<String, Artifact> getArtifactMap()
>>> {
>>> if ( artifactMap == null )
>>> {
>>> @@ -1180,19 +1183,19 @@
>>> return artifactMap;
>>> }
>>>
>>> - public void setPluginArtifacts( Set pluginArtifacts )
>>> + public void setPluginArtifacts( Set<Artifact> pluginArtifacts )
>>> {
>>> this.pluginArtifacts = pluginArtifacts;
>>>
>>> this.pluginArtifactMap = null;
>>> }
>>>
>>> - public Set getPluginArtifacts()
>>> + public Set<Artifact> getPluginArtifacts()
>>> {
>>> return pluginArtifacts;
>>> }
>>>
>>> - public Map getPluginArtifactMap()
>>> + public Map<String, Artifact> getPluginArtifactMap()
>>> {
>>> if ( pluginArtifactMap == null )
>>> {
>>> @@ -1202,19 +1205,19 @@
>>> return pluginArtifactMap;
>>> }
>>>
>>> - public void setReportArtifacts( Set reportArtifacts )
>>> + public void setReportArtifacts( Set<Artifact> reportArtifacts )
>>> {
>>> this.reportArtifacts = reportArtifacts;
>>>
>>> this.reportArtifactMap = null;
>>> }
>>>
>>> - public Set getReportArtifacts()
>>> + public Set<Artifact> getReportArtifacts()
>>> {
>>> return reportArtifacts;
>>> }
>>>
>>> - public Map getReportArtifactMap()
>>> + public Map<String, Artifact> getReportArtifactMap()
>>> {
>>> if ( reportArtifactMap == null )
>>> {
>>> @@ -1224,19 +1227,19 @@
>>> return reportArtifactMap;
>>> }
>>>
>>> - public void setExtensionArtifacts( Set extensionArtifacts )
>>> + public void setExtensionArtifacts( Set<Artifact> extensionArtifacts
>>> )
>>> {
>>> this.extensionArtifacts = extensionArtifacts;
>>>
>>> this.extensionArtifactMap = null;
>>> }
>>>
>>> - public Set getExtensionArtifacts()
>>> + public Set<Artifact> getExtensionArtifacts()
>>> {
>>> return this.extensionArtifacts;
>>> }
>>>
>>> - public Map getExtensionArtifactMap()
>>> + public Map<String, Artifact> getExtensionArtifactMap()
>>> {
>>> if ( extensionArtifactMap == null )
>>> {
>>> @@ -1256,7 +1259,7 @@
>>> return parentArtifact;
>>> }
>>>
>>> - public List getRepositories()
>>> + public List<Repository> getRepositories()
>>> {
>>> return getModel().getRepositories();
>>> }
>>> @@ -1265,7 +1268,7 @@
>>> // Plugins
>>> //
>>> ----------------------------------------------------------------------
>>>
>>> - public List getReportPlugins()
>>> + public List<ReportPlugin> getReportPlugins()
>>> {
>>> if ( getModel().getReporting() == null )
>>> {
>>> @@ -1275,7 +1278,7 @@
>>>
>>> }
>>>
>>> - public List getBuildPlugins()
>>> + public List<Plugin> getBuildPlugins()
>>> {
>>> if ( getModel().getBuild() == null )
>>> {
>>> @@ -1284,7 +1287,7 @@
>>> return getModel().getBuild().getPlugins();
>>> }
>>>
>>> - public List getModules()
>>> + public List<String> getModules()
>>> {
>>> return getModel().getModules();
>>> }
>>> @@ -1335,7 +1338,7 @@
>>>
>>> if ( pm != null )
>>> {
>>> - Map pmByKey = pm.getPluginsAsMap();
>>> + Map<String, Plugin> pmByKey = pm.getPluginsAsMap();
>>>
>>> String pluginKey = plugin.getKey();
>>>
>>> @@ -1348,17 +1351,17 @@
>>> }
>>> }
>>>
>>> - public List getCollectedProjects()
>>> + public List<MavenProject> getCollectedProjects()
>>> {
>>> return collectedProjects;
>>> }
>>>
>>> - public void setCollectedProjects( List collectedProjects )
>>> + public void setCollectedProjects( List<MavenProject>
>>> collectedProjects )
>>> {
>>> this.collectedProjects = collectedProjects;
>>> }
>>>
>>> - public void setPluginArtifactRepositories( List
>>> pluginArtifactRepositories )
>>> + public void setPluginArtifactRepositories( List<ArtifactRepository>
>>> pluginArtifactRepositories )
>>> {
>>> this.pluginArtifactRepositories = pluginArtifactRepositories;
>>> }
>>> @@ -1367,7 +1370,7 @@
>>> * @return a list of ArtifactRepository objects constructed
>>> * from the Repository objects returned by
>>> getPluginRepositories.
>>> */
>>> - public List getPluginArtifactRepositories()
>>> + public List<ArtifactRepository> getPluginArtifactRepositories()
>>> {
>>> return pluginArtifactRepositories;
>>> }
>>> @@ -1378,17 +1381,17 @@
>>> : getReleaseArtifactRepository();
>>> }
>>>
>>> - public List getPluginRepositories()
>>> + public List<Repository> getPluginRepositories()
>>> {
>>> return getModel().getPluginRepositories();
>>> }
>>>
>>> - public void setActiveProfiles( List activeProfiles )
>>> + public void setActiveProfiles( List<String> activeProfiles )
>>> {
>>> this.activeProfiles.addAll( activeProfiles );
>>> }
>>>
>>> - public List getActiveProfiles()
>>> + public List<String> getActiveProfiles()
>>> {
>>> return activeProfiles;
>>> }
>>> @@ -1398,11 +1401,11 @@
>>> getAttachedArtifacts().add( artifact );
>>> }
>>>
>>> - public List getAttachedArtifacts()
>>> + public List<Artifact> getAttachedArtifacts()
>>> {
>>> if ( attachedArtifacts == null )
>>> {
>>> - attachedArtifacts = new ArrayList();
>>> + attachedArtifacts = new ArrayList<Artifact>();
>>> }
>>> return attachedArtifacts;
>>> }
>>> @@ -1420,7 +1423,7 @@
>>>
>>> if ( getBuildPlugins() != null )
>>> {
>>> - for ( Iterator iterator = getBuildPlugins().iterator();
>>> iterator.hasNext(); )
>>> + for ( Iterator<Plugin> iterator =
>>> getBuildPlugins().iterator(); iterator.hasNext(); )
>>> {
>>> Plugin plugin = (Plugin) iterator.next();
>>>
>>> @@ -1498,7 +1501,7 @@
>>>
>>> if ( getReportPlugins() != null )
>>> {
>>> - for ( Iterator iterator = getReportPlugins().iterator();
>>> iterator.hasNext(); )
>>> + for ( Iterator<ReportPlugin> iterator =
>>> getReportPlugins().iterator(); iterator.hasNext(); )
>>> {
>>> ReportPlugin plugin = (ReportPlugin) iterator.next();
>>>
>>> @@ -1564,12 +1567,12 @@
>>> * @return {@link Set} < {@link Artifact} >
>>> * @see #getArtifacts() to get all transitive dependencies
>>> */
>>> - public Set getDependencyArtifacts()
>>> + public Set<Artifact> getDependencyArtifacts()
>>> {
>>> return dependencyArtifacts;
>>> }
>>>
>>> - public void setDependencyArtifacts( Set dependencyArtifacts )
>>> + public void setDependencyArtifacts( Set<Artifact>
>>> dependencyArtifacts )
>>> {
>>> this.dependencyArtifacts = dependencyArtifacts;
>>> }
>>> @@ -1594,12 +1597,12 @@
>>> return originalModel;
>>> }
>>>
>>> - public void setManagedVersionMap( Map map )
>>> + public void setManagedVersionMap( Map<String, ArtifactVersion> map )
>>> {
>>> this.managedVersionMap = map;
>>> }
>>>
>>> - public Map getManagedVersionMap()
>>> + public Map<String, ArtifactVersion> getManagedVersionMap()
>>> {
>>> return this.managedVersionMap;
>>> }
>>> @@ -1627,12 +1630,12 @@
>>> return getId().hashCode();
>>> }
>>>
>>> - public List getBuildExtensions()
>>> + public List<Extension> getBuildExtensions()
>>> {
>>> Build build = getBuild();
>>> if ( build == null || build.getExtensions() == null )
>>> {
>>> - return Collections.EMPTY_LIST;
>>> + return Collections.<Extension> emptyList();
>>> }
>>> else
>>> {
>>> @@ -1644,7 +1647,7 @@
>>> * @todo the lazy initialisation of this makes me uneasy.
>>> * @return {@link Set} < {@link Artifact} >
>>> */
>>> - public Set createArtifacts( ArtifactFactory artifactFactory, String
>>> inheritedScope,
>>> + public Set<Artifact> createArtifacts( ArtifactFactory
>>> artifactFactory, String inheritedScope,
>>> ArtifactFilter dependencyFilter )
>>> throws InvalidDependencyVersionException
>>> {
>>> @@ -1674,12 +1677,12 @@
>>> return getModel().getProperties();
>>> }
>>>
>>> - public List getFilters()
>>> + public List<String> getFilters()
>>> {
>>> return getBuild().getFilters();
>>> }
>>>
>>> - public Map getProjectReferences()
>>> + public Map<String, MavenProject> getProjectReferences()
>>> {
>>> return projectReferences;
>>> }
>>> @@ -1705,22 +1708,22 @@
>>> this.model = model;
>>> }
>>>
>>> - protected void setAttachedArtifacts( List attachedArtifacts )
>>> + protected void setAttachedArtifacts( List<Artifact>
>>> attachedArtifacts )
>>> {
>>> this.attachedArtifacts = attachedArtifacts;
>>> }
>>>
>>> - protected void setCompileSourceRoots( List compileSourceRoots )
>>> + protected void setCompileSourceRoots( List<String>
>>> compileSourceRoots )
>>> {
>>> this.compileSourceRoots = compileSourceRoots;
>>> }
>>>
>>> - protected void setTestCompileSourceRoots( List
>>> testCompileSourceRoots )
>>> + protected void setTestCompileSourceRoots( List<String>
>>> testCompileSourceRoots )
>>> {
>>> this.testCompileSourceRoots = testCompileSourceRoots;
>>> }
>>>
>>> - protected void setScriptSourceRoots( List scriptSourceRoots )
>>> + protected void setScriptSourceRoots( List<String> scriptSourceRoots
>>> )
>>> {
>>> this.scriptSourceRoots = scriptSourceRoots;
>>> }
>>> @@ -1737,16 +1740,16 @@
>>>
>>> public void resolveActiveArtifacts()
>>> {
>>> - Set depArtifacts = getDependencyArtifacts();
>>> + Set<Artifact> depArtifacts = getDependencyArtifacts();
>>> if ( depArtifacts == null )
>>> {
>>> return;
>>> }
>>>
>>> - Set updated = new LinkedHashSet( depArtifacts.size() );
>>> + Set<Artifact> updated = new LinkedHashSet<Artifact>(
>>> depArtifacts.size() );
>>> int updatedCount = 0;
>>>
>>> - for ( Iterator it = depArtifacts.iterator(); it.hasNext(); )
>>> + for ( Iterator<Artifact> it = depArtifacts.iterator();
>>> it.hasNext(); )
>>> {
>>> Artifact depArtifact = (Artifact) it.next();
>>> Artifact replaced = replaceWithActiveArtifact( depArtifact );
>>> @@ -1815,13 +1818,13 @@
>>> * @param requestedArtifact The artifact to resolve, must not be
>>> <code>null</code>.
>>> * @return The matching artifact or <code>null</code> if not found.
>>> */
>>> - private Artifact findMatchingArtifact( List artifacts, Artifact
>>> requestedArtifact )
>>> + private Artifact findMatchingArtifact( List<Artifact> artifacts,
>>> Artifact requestedArtifact )
>>> {
>>> if ( artifacts != null && !artifacts.isEmpty() )
>>> {
>>> // first try matching by dependency conflict id
>>> String requestedId =
>>> requestedArtifact.getDependencyConflictId();
>>> - for ( Iterator it = artifacts.iterator(); it.hasNext(); )
>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>> it.hasNext(); )
>>> {
>>> Artifact artifact = (Artifact) it.next();
>>> if ( requestedId.equals(
>>> artifact.getDependencyConflictId() ) )
>>> @@ -1832,7 +1835,7 @@
>>>
>>> // next try matching by repository conflict id
>>> requestedId = getRepositoryConflictId( requestedArtifact );
>>> - for ( Iterator it = artifacts.iterator(); it.hasNext(); )
>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>> it.hasNext(); )
>>> {
>>> Artifact artifact = (Artifact) it.next();
>>> if ( requestedId.equals( getRepositoryConflictId( artifact
>>> ) ) )
>>> @@ -1898,7 +1901,7 @@
>>> }
>>> }
>>>
>>> - private void addArtifactPath(Artifact a, List list) throws
>>> DependencyResolutionRequiredException
>>> + private void addArtifactPath(Artifact a, List<String> list) throws
>>> DependencyResolutionRequiredException
>>> {
>>> File file = a.getFile();
>>> if ( file == null )
>>> @@ -1954,21 +1957,21 @@
>>>
>>> private Build originalInterpolatedBuild;
>>>
>>> - private List dynamicCompileSourceRoots;
>>> + private List<String> dynamicCompileSourceRoots;
>>>
>>> - private List originalInterpolatedCompileSourceRoots;
>>> + private List<String> originalInterpolatedCompileSourceRoots;
>>>
>>> - private List dynamicTestCompileSourceRoots;
>>> + private List<String> dynamicTestCompileSourceRoots;
>>>
>>> - private List originalInterpolatedTestCompileSourceRoots;
>>> + private List<String> originalInterpolatedTestCompileSourceRoots;
>>>
>>> - private List dynamicScriptSourceRoots;
>>> + private List<String> dynamicScriptSourceRoots;
>>>
>>> - private List originalInterpolatedScriptSourceRoots;
>>> + private List<String> originalInterpolatedScriptSourceRoots;
>>>
>>> private boolean isConcrete = false;
>>>
>>> - public boolean isConcrete()
>>> + public boolean isConcrete()
>>> {
>>> return isConcrete;
>>> }
>>> @@ -1988,32 +1991,32 @@
>>> return originalInterpolatedBuild;
>>> }
>>>
>>> - public List getDynamicCompileSourceRoots()
>>> + public List<String> getDynamicCompileSourceRoots()
>>> {
>>> return dynamicCompileSourceRoots;
>>> }
>>>
>>> - public List getOriginalInterpolatedCompileSourceRoots()
>>> + public List<String> getOriginalInterpolatedCompileSourceRoots()
>>> {
>>> return originalInterpolatedCompileSourceRoots;
>>> }
>>>
>>> - public List getDynamicTestCompileSourceRoots()
>>> + public List<String> getDynamicTestCompileSourceRoots()
>>> {
>>> return dynamicTestCompileSourceRoots;
>>> }
>>>
>>> - public List getOriginalInterpolatedTestCompileSourceRoots()
>>> + public List<String> getOriginalInterpolatedTestCompileSourceRoots()
>>> {
>>> return originalInterpolatedTestCompileSourceRoots;
>>> }
>>>
>>> - public List getDynamicScriptSourceRoots()
>>> + public List<String> getDynamicScriptSourceRoots()
>>> {
>>> return dynamicScriptSourceRoots;
>>> }
>>>
>>> - public List getOriginalInterpolatedScriptSourceRoots()
>>> + public List<String> getOriginalInterpolatedScriptSourceRoots()
>>> {
>>> return originalInterpolatedScriptSourceRoots;
>>> }
>>> @@ -2034,19 +2037,19 @@
>>> originalInterpolatedBuild = null;
>>> }
>>>
>>> - public void preserveCompileSourceRoots( List
>>> originalInterpolatedCompileSourceRoots )
>>> + public void preserveCompileSourceRoots( List<String>
>>> originalInterpolatedCompileSourceRoots )
>>> {
>>> dynamicCompileSourceRoots = getCompileSourceRoots();
>>> this.originalInterpolatedCompileSourceRoots =
>>> originalInterpolatedCompileSourceRoots;
>>> }
>>>
>>> - public void preserveTestCompileSourceRoots( List
>>> originalInterpolatedTestCompileSourceRoots )
>>> + public void preserveTestCompileSourceRoots( List<String>
>>> originalInterpolatedTestCompileSourceRoots )
>>> {
>>> dynamicTestCompileSourceRoots = getTestCompileSourceRoots();
>>> this.originalInterpolatedTestCompileSourceRoots =
>>> originalInterpolatedTestCompileSourceRoots;
>>> }
>>>
>>> - public void preserveScriptSourceRoots( List
>>> originalInterpolatedScriptSourceRoots )
>>> + public void preserveScriptSourceRoots( List<String>
>>> originalInterpolatedScriptSourceRoots )
>>> {
>>> dynamicScriptSourceRoots = getScriptSourceRoots();
>>> this.originalInterpolatedScriptSourceRoots =
>>> originalInterpolatedScriptSourceRoots;
>>> @@ -2071,32 +2074,32 @@
>>> this.originalInterpolatedBuild = originalInterpolatedBuild;
>>> }
>>>
>>> - protected void setDynamicCompileSourceRoots( List
>>> dynamicCompileSourceRoots )
>>> + protected void setDynamicCompileSourceRoots( List<String>
>>> dynamicCompileSourceRoots )
>>> {
>>> this.dynamicCompileSourceRoots = dynamicCompileSourceRoots;
>>> }
>>>
>>> - protected void setOriginalInterpolatedCompileSourceRoots( List
>>> originalInterpolatedCompileSourceRoots )
>>> + protected void setOriginalInterpolatedCompileSourceRoots(
>>> List<String> originalInterpolatedCompileSourceRoots )
>>> {
>>> this.originalInterpolatedCompileSourceRoots =
>>> originalInterpolatedCompileSourceRoots;
>>> }
>>>
>>> - protected void setDynamicTestCompileSourceRoots( List
>>> dynamicTestCompileSourceRoots )
>>> + protected void setDynamicTestCompileSourceRoots( List<String>
>>> dynamicTestCompileSourceRoots )
>>> {
>>> this.dynamicTestCompileSourceRoots =
>>> dynamicTestCompileSourceRoots;
>>> }
>>>
>>> - protected void setOriginalInterpolatedTestCompileSourceRoots( List
>>> originalInterpolatedTestCompileSourceRoots )
>>> + protected void setOriginalInterpolatedTestCompileSourceRoots(
>>> List<String> originalInterpolatedTestCompileSourceRoots )
>>> {
>>> this.originalInterpolatedTestCompileSourceRoots =
>>> originalInterpolatedTestCompileSourceRoots;
>>> }
>>>
>>> - protected void setDynamicScriptSourceRoots( List
>>> dynamicScriptSourceRoots )
>>> + protected void setDynamicScriptSourceRoots( List<String>
>>> dynamicScriptSourceRoots )
>>> {
>>> this.dynamicScriptSourceRoots = dynamicScriptSourceRoots;
>>> }
>>>
>>> - protected void setOriginalInterpolatedScriptSourceRoots( List
>>> originalInterpolatedScriptSourceRoots )
>>> + protected void setOriginalInterpolatedScriptSourceRoots(
>>> List<String> originalInterpolatedScriptSourceRoots )
>>> {
>>> this.originalInterpolatedScriptSourceRoots =
>>> originalInterpolatedScriptSourceRoots;
>>> }
>>> @@ -2114,7 +2117,7 @@
>>> if ( p != null )
>>> {
>>> preservedProperties = new Properties();
>>> - for( Enumeration e = p.propertyNames(); e.hasMoreElements();
>>> )
>>> + for( Enumeration<?> e = p.propertyNames();
>>> e.hasMoreElements(); )
>>> {
>>> String key = (String) e.nextElement();
>>> preservedProperties.setProperty( key, p.getProperty( key )
>>> );
>>>
>>> Modified:
>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
>>> URL:
>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>> ==============================================================================
>>>
>>> ---
>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
>>> (original)
>>> +++
>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml Fri
>>> May 1 08:04:48 2009
>>> @@ -47,6 +47,7 @@
>>> <models>
>>> <model>src/main/mdo/metadata.mdo</model>
>>> </models>
>>> + <useJava5>true</useJava5>
>>> </configuration>
>>> </plugin>
>>> </plugins>
>>>
>>> Modified: maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>> URL:
>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-settings/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>> ==============================================================================
>>>
>>> --- maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>> (original)
>>> +++ maven/components/branches/maven-2.2.x/maven-settings/pom.xml Fri May
>>> 1 08:04:48 2009
>>> @@ -58,6 +58,7 @@
>>> <models>
>>> <model>src/main/mdo/settings.mdo</model>
>>> </models>
>>> + <useJava5>true</useJava5>
>>> </configuration>
>>> </plugin>
>>> </plugins>
>>>
>>> Modified: maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>> URL:
>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>> ==============================================================================
>>>
>>> --- maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>> (original)
>>> +++ maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml Fri May
>>> 1 08:04:48 2009
>>> @@ -66,6 +66,7 @@
>>> <models>
>>> <model>src/main/mdo/toolchains.mdo</model>
>>> </models>
>>> + <useJava5>true</useJava5>
>>> </configuration>
>>> </plugin>
>>> <plugin>
>>>
>>>
>>>
>> Thanks,
>>
>> Jason
>>
>> ----------------------------------------------------------
>> Jason van Zyl
>> Founder, Apache Maven
>> http://twitter.com/jvanzyl
>> http://twitter.com/SonatypeNexus
>> http://twitter.com/SonatypeM2E
>> ----------------------------------------------------------
>>
>> We all have problems. How we deal with them is a measure of our worth.
>>
>> -- Unknown
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>> For additional commands, e-mail: dev-help@maven.apache.org
>>
>>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>
Re: svn commit: r770570 - in /maven/components/branches/maven-2.2.x: maven-artifact/src/main/java/org/apache/maven/artifact/ maven-model/ maven-profile/ maven-project/src/main/java/org/apache/maven/project/ maven-repository-metadata/ maven-settings/
Posted by Hervé BOUTEMY <he...@free.fr>.
What is the problem with this commit regarding the scope of what John is
trying to do wrt to 2.2?
I thought Maven 2.2 requiring Java 5 was about to use typed collections: then
every enhancement to use typed collections is in the scopen, no?
Is there something I'm missing?
FYI, I'm not a Java 5 expert (yes, I'm still stuck with 1.4...): my
comprehension of typed collections is still very theoritical, without
knowledge of real world problems when using them.
regards,
Hervé
Le samedi 02 mai 2009, nicolas de loof a écrit :
> Do we prefer plugin developers to use List<?> and get ClassCastExceptions
> at runtime or sue type-safe collections to help them create stronger code
> ?Anyway, few plugins allready use Java5. Java 1.4 based one will not be
> broken as the generics signature is only a compile-time check.
>
>
> 2009/5/1 Brian Fox <br...@infinity.nu>
>
> > I'm not sure if this is in scope of what John is trying to do wrt to 2.2.
> >
> > Jason van Zyl wrote:
> >> I don't believe anyone actually agreed to this yet. Are you sure this is
> >> not going to cause problems for users?
> >>
> >> On 1-May-09, at 1:04 AM, nicolas@apache.org wrote:
> >>
> >> Author: nicolas
> >>
> >>> Date: Fri May 1 08:04:48 2009
> >>> New Revision: 770570
> >>>
> >>> URL: http://svn.apache.org/viewvc?rev=770570&view=rev
> >>> Log:
> >>> use java5 syntax.
> >>> -> set modello to generate java5 syntax
> >>> -> MavenProject updated to use generics on collections & methods for
> >>> better type safety
> >>>
> >>> Modified:
> >>>
> >>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/
> >>>apache/maven/artifact/ArtifactUtils.java
> >>>
> >>> maven/components/branches/maven-2.2.x/maven-model/pom.xml
> >>> maven/components/branches/maven-2.2.x/maven-profile/pom.xml
> >>>
> >>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/a
> >>>pache/maven/project/MavenProject.java
> >>>
> >>>
> >>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
> >>> maven/components/branches/maven-2.2.x/maven-settings/pom.xml
> >>> maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
> >>>
> >>> Modified:
> >>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/
> >>>apache/maven/artifact/ArtifactUtils.java
> >>>
> >>> URL:
> >>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/mave
> >>>n-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java?re
> >>>v=770570&r1=770569&r2=770570&view=diff
> >>> =======================================================================
> >>>=======
> >>>
> >>> ---
> >>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/
> >>>apache/maven/artifact/ArtifactUtils.java (original)
> >>> +++
> >>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/
> >>>apache/maven/artifact/ArtifactUtils.java Fri May 1 08:04:48 2009
> >>> @@ -86,13 +86,13 @@
> >>> baseVersion;
> >>> }
> >>>
> >>> - public static Map artifactMapByVersionlessId( Collection artifacts
> >>> ) + public static Map<String, Artifact> artifactMapByVersionlessId(
> >>> Collection<Artifact> artifacts )
> >>> {
> >>> - Map artifactMap = new LinkedHashMap();
> >>> + Map<String, Artifact> artifactMap = new LinkedHashMap<String,
> >>> Artifact>();
> >>>
> >>> if ( artifacts != null )
> >>> {
> >>> - for ( Iterator it = artifacts.iterator(); it.hasNext(); )
> >>> + for ( Iterator<Artifact> it = artifacts.iterator();
> >>> it.hasNext(); )
> >>> {
> >>> Artifact artifact = (Artifact) it.next();
> >>>
> >>>
> >>> Modified: maven/components/branches/maven-2.2.x/maven-model/pom.xml
> >>> URL:
> >>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/mave
> >>>n-model/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
> >>> =======================================================================
> >>>=======
> >>>
> >>> --- maven/components/branches/maven-2.2.x/maven-model/pom.xml
> >>> (original) +++
> >>> maven/components/branches/maven-2.2.x/maven-model/pom.xml Fri May 1
> >>> 08:04:48 2009
> >>> @@ -47,6 +47,7 @@
> >>> <models>
> >>> <model>src/main/mdo/maven.mdo</model>
> >>> </models>
> >>> + <useJava5>true</useJava5>
> >>> </configuration>
> >>> </plugin>
> >>> <plugin>
> >>>
> >>> Modified: maven/components/branches/maven-2.2.x/maven-profile/pom.xml
> >>> URL:
> >>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/mave
> >>>n-profile/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
> >>> =======================================================================
> >>>=======
> >>>
> >>> --- maven/components/branches/maven-2.2.x/maven-profile/pom.xml
> >>> (original)
> >>> +++ maven/components/branches/maven-2.2.x/maven-profile/pom.xml Fri May
> >>> 1 08:04:48 2009
> >>> @@ -54,6 +54,7 @@
> >>> <models>
> >>> <model>src/main/mdo/profiles.mdo</model>
> >>> </models>
> >>> + <useJava5>true</useJava5>
> >>> </configuration>
> >>> </plugin>
> >>> </plugins>
> >>>
> >>> Modified:
> >>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/a
> >>>pache/maven/project/MavenProject.java
> >>>
> >>> URL:
> >>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/mave
> >>>n-project/src/main/java/org/apache/maven/project/MavenProject.java?rev=7
> >>>70570&r1=770569&r2=770570&view=diff
> >>> =======================================================================
> >>>=======
> >>>
> >>> ---
> >>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/a
> >>>pache/maven/project/MavenProject.java (original)
> >>> +++
> >>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/a
> >>>pache/maven/project/MavenProject.java Fri May 1 08:04:48 2009
> >>> @@ -36,10 +36,11 @@
> >>> import org.apache.maven.artifact.Artifact;
> >>> import org.apache.maven.artifact.ArtifactUtils;
> >>> import org.apache.maven.artifact.DependencyResolutionRequiredException;
> >>> -import org.apache.maven.artifact.versioning.ManagedVersionMap;
> >>> import org.apache.maven.artifact.factory.ArtifactFactory;
> >>> import org.apache.maven.artifact.repository.ArtifactRepository;
> >>> import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
> >>> +import org.apache.maven.artifact.versioning.ArtifactVersion;
> >>> +import org.apache.maven.artifact.versioning.ManagedVersionMap;
> >>> import org.apache.maven.model.Build;
> >>> import org.apache.maven.model.CiManagement;
> >>> import org.apache.maven.model.Contributor;
> >>> @@ -47,6 +48,7 @@
> >>> import org.apache.maven.model.DependencyManagement;
> >>> import org.apache.maven.model.Developer;
> >>> import org.apache.maven.model.DistributionManagement;
> >>> +import org.apache.maven.model.Extension;
> >>> import org.apache.maven.model.IssueManagement;
> >>> import org.apache.maven.model.License;
> >>> import org.apache.maven.model.MailingList;
> >>> @@ -59,6 +61,7 @@
> >>> import org.apache.maven.model.ReportPlugin;
> >>> import org.apache.maven.model.ReportSet;
> >>> import org.apache.maven.model.Reporting;
> >>> +import org.apache.maven.model.Repository;
> >>> import org.apache.maven.model.Resource;
> >>> import org.apache.maven.model.Scm;
> >>> import org.apache.maven.model.io.xpp3.MavenXpp3Writer;
> >>> @@ -96,60 +99,60 @@
> >>>
> >>> private File file;
> >>>
> >>> - private Set artifacts;
> >>> + private Set<Artifact> artifacts;
> >>>
> >>> private Artifact parentArtifact;
> >>>
> >>> - private Set pluginArtifacts;
> >>> + private Set<Artifact> pluginArtifacts;
> >>>
> >>> - private List remoteArtifactRepositories;
> >>> + private List<ArtifactRepository> remoteArtifactRepositories;
> >>>
> >>> - private List collectedProjects = Collections.EMPTY_LIST;
> >>> + private List<MavenProject> collectedProjects =
> >>> Collections.<MavenProject> emptyList();
> >>>
> >>> - private List attachedArtifacts;
> >>> + private List<Artifact> attachedArtifacts;
> >>>
> >>> private MavenProject executionProject;
> >>>
> >>> - private List compileSourceRoots = new ArrayList();
> >>> + private List<String> compileSourceRoots = new ArrayList<String>();
> >>>
> >>> - private List testCompileSourceRoots = new ArrayList();
> >>> + private List<String> testCompileSourceRoots = new
> >>> ArrayList<String>();
> >>>
> >>> - private List scriptSourceRoots = new ArrayList();
> >>> + private List<String> scriptSourceRoots = new ArrayList<String>();
> >>>
> >>> - private List pluginArtifactRepositories;
> >>> + private List<ArtifactRepository> pluginArtifactRepositories;
> >>>
> >>> private ArtifactRepository releaseArtifactRepository;
> >>>
> >>> private ArtifactRepository snapshotArtifactRepository;
> >>>
> >>> - private List activeProfiles = new ArrayList();
> >>> + private List<String> activeProfiles = new ArrayList<String>();
> >>>
> >>> - private Set dependencyArtifacts;
> >>> + private Set<Artifact> dependencyArtifacts;
> >>>
> >>> private Artifact artifact;
> >>>
> >>> // calculated.
> >>> - private Map artifactMap;
> >>> + private Map<String, Artifact> artifactMap;
> >>>
> >>> private Model originalModel;
> >>>
> >>> - private Map pluginArtifactMap;
> >>> + private Map<String, Artifact> pluginArtifactMap;
> >>>
> >>> - private Set reportArtifacts;
> >>> + private Set<Artifact> reportArtifacts;
> >>>
> >>> - private Map reportArtifactMap;
> >>> + private Map<String, Artifact> reportArtifactMap;
> >>>
> >>> - private Set extensionArtifacts;
> >>> + private Set<Artifact> extensionArtifacts;
> >>>
> >>> - private Map extensionArtifactMap;
> >>> + private Map<String, Artifact> extensionArtifactMap;
> >>>
> >>> - private Map managedVersionMap;
> >>> + private Map<String, ArtifactVersion> managedVersionMap;
> >>>
> >>> - private Map projectReferences = new HashMap();
> >>> + private Map<String, MavenProject> projectReferences = new
> >>> HashMap<String, MavenProject>();
> >>>
> >>> private boolean executionRoot;
> >>>
> >>> - private Map moduleAdjustments;
> >>> + private Map<String, String> moduleAdjustments;
> >>>
> >>> private File basedir;
> >>>
> >>> @@ -245,23 +248,23 @@
> >>> if ( project.getAttachedArtifacts() != null )
> >>> {
> >>> // clone properties modifyable by plugins in a forked
> >>> lifecycle
> >>> - setAttachedArtifacts( new ArrayList(
> >>> project.getAttachedArtifacts() ) );
> >>> + setAttachedArtifacts( new ArrayList<Artifact>(
> >>> project.getAttachedArtifacts() ) );
> >>> }
> >>>
> >>> if ( project.getCompileSourceRoots() != null )
> >>> {
> >>> // clone source roots
> >>> - setCompileSourceRoots( ( new ArrayList(
> >>> project.getCompileSourceRoots() ) ) );
> >>> + setCompileSourceRoots( ( new ArrayList<String>(
> >>> project.getCompileSourceRoots() ) ) );
> >>> }
> >>>
> >>> if ( project.getTestCompileSourceRoots() != null )
> >>> {
> >>> - setTestCompileSourceRoots( ( new ArrayList(
> >>> project.getTestCompileSourceRoots() ) ) );
> >>> + setTestCompileSourceRoots( ( new ArrayList<String>(
> >>> project.getTestCompileSourceRoots() ) ) );
> >>> }
> >>>
> >>> if ( project.getScriptSourceRoots() != null )
> >>> {
> >>> - setScriptSourceRoots( ( new ArrayList(
> >>> project.getScriptSourceRoots() ) ) );
> >>> + setScriptSourceRoots( ( new ArrayList<String>(
> >>> project.getScriptSourceRoots() ) ) );
> >>> }
> >>>
> >>> setModel( ( ModelUtils.cloneModel( project.getModel() ) ) );
> >>> @@ -298,25 +301,25 @@
> >>> setDynamicBuild( ModelUtils.cloneBuild(
> >>> project.getDynamicBuild() ) );
> >>> setOriginalInterpolatedBuild( ModelUtils.cloneBuild(
> >>> project.getOriginalInterpolatedBuild() ) );
> >>>
> >>> - List dynamicRoots =
> >>> project.getDynamicCompileSourceRoots(); + List<String>
> >>> dynamicRoots =
> >>> project.getDynamicCompileSourceRoots();
> >>> if ( dynamicRoots != null )
> >>> {
> >>> - setDynamicCompileSourceRoots( new ArrayList(
> >>> dynamicRoots ) );
> >>> - setOriginalInterpolatedCompileSourceRoots( new
> >>> ArrayList( project.getOriginalInterpolatedCompileSourceRoots() ) );
> >>> + setDynamicCompileSourceRoots( new ArrayList<String>(
> >>> dynamicRoots ) );
> >>> + setOriginalInterpolatedCompileSourceRoots( new
> >>> ArrayList<String>( project.getOriginalInterpolatedCompileSourceRoots()
> >>> ) ); }
> >>>
> >>> dynamicRoots = project.getDynamicTestCompileSourceRoots();
> >>> if ( dynamicRoots != null )
> >>> {
> >>> - setDynamicTestCompileSourceRoots( new ArrayList(
> >>> dynamicRoots ) );
> >>> - setOriginalInterpolatedTestCompileSourceRoots( new
> >>> ArrayList( project.getOriginalInterpolatedTestCompileSourceRoots() ) );
> >>> + setDynamicTestCompileSourceRoots( new
> >>> ArrayList<String>( dynamicRoots ) );
> >>> + setOriginalInterpolatedTestCompileSourceRoots( new
> >>> ArrayList<String>(
> >>> project.getOriginalInterpolatedTestCompileSourceRoots() ) );
> >>> }
> >>>
> >>> dynamicRoots = project.getDynamicScriptSourceRoots();
> >>> if ( dynamicRoots != null )
> >>> {
> >>> - setDynamicScriptSourceRoots( new ArrayList(
> >>> dynamicRoots ) );
> >>> - setOriginalInterpolatedScriptSourceRoots( new
> >>> ArrayList( project.getOriginalInterpolatedScriptSourceRoots() ) );
> >>> + setDynamicScriptSourceRoots( new ArrayList<String>(
> >>> dynamicRoots ) );
> >>> + setOriginalInterpolatedScriptSourceRoots( new
> >>> ArrayList<String>( project.getOriginalInterpolatedScriptSourceRoots() )
> >>> ); }
> >>> }
> >>>
> >>> @@ -347,12 +350,12 @@
> >>>
> >>> if ( moduleAdjustments == null )
> >>> {
> >>> - moduleAdjustments = new HashMap();
> >>> + moduleAdjustments = new HashMap<String, String>();
> >>>
> >>> - List modules = getModules();
> >>> + List<String> modules = getModules();
> >>> if ( modules != null )
> >>> {
> >>> - for ( Iterator it = modules.iterator(); it.hasNext();
> >>> ) + for ( Iterator<String> it = modules.iterator();
> >>> it.hasNext(); )
> >>> {
> >>> String modulePath = (String) it.next();
> >>> String moduleName = modulePath;
> >>> @@ -415,12 +418,12 @@
> >>> this.parent = parent;
> >>> }
> >>>
> >>> - public void setRemoteArtifactRepositories( List
> >>> remoteArtifactRepositories )
> >>> + public void setRemoteArtifactRepositories(
> >>> List<ArtifactRepository> remoteArtifactRepositories )
> >>> {
> >>> this.remoteArtifactRepositories = remoteArtifactRepositories;
> >>> }
> >>>
> >>> - public List getRemoteArtifactRepositories()
> >>> + public List<ArtifactRepository> getRemoteArtifactRepositories()
> >>> {
> >>> return remoteArtifactRepositories;
> >>> }
> >>> @@ -460,12 +463,12 @@
> >>> return basedir;
> >>> }
> >>>
> >>> - public void setDependencies( List dependencies )
> >>> + public void setDependencies( List<Dependency> dependencies )
> >>> {
> >>> getModel().setDependencies( dependencies );
> >>> }
> >>>
> >>> - public List getDependencies()
> >>> + public List<Dependency> getDependencies()
> >>> {
> >>> return getModel().getDependencies();
> >>> }
> >>> @@ -524,29 +527,29 @@
> >>> }
> >>> }
> >>>
> >>> - public List getCompileSourceRoots()
> >>> + public List<String> getCompileSourceRoots()
> >>> {
> >>> return compileSourceRoots;
> >>> }
> >>>
> >>> - public List getScriptSourceRoots()
> >>> + public List<String> getScriptSourceRoots()
> >>> {
> >>> return scriptSourceRoots;
> >>> }
> >>>
> >>> - public List getTestCompileSourceRoots()
> >>> + public List<String> getTestCompileSourceRoots()
> >>> {
> >>> return testCompileSourceRoots;
> >>> }
> >>>
> >>> - public List getCompileClasspathElements()
> >>> + public List<String> getCompileClasspathElements()
> >>> throws DependencyResolutionRequiredException
> >>> {
> >>> - List list = new ArrayList( getArtifacts().size() );
> >>> + List<String> list = new ArrayList<String>(
> >>> getArtifacts().size() );
> >>>
> >>> list.add( getBuild().getOutputDirectory() );
> >>>
> >>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
> >>> + for ( Iterator<?> i = getArtifacts().iterator(); i.hasNext();
> >>> ) {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -563,11 +566,11 @@
> >>> return list;
> >>> }
> >>>
> >>> - public List getCompileArtifacts()
> >>> + public List<Artifact> getCompileArtifacts()
> >>> {
> >>> - List list = new ArrayList( getArtifacts().size() );
> >>> + List<Artifact> list = new ArrayList<Artifact>(
> >>> getArtifacts().size() );
> >>>
> >>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
> >>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
> >>> i.hasNext(); )
> >>> {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -585,18 +588,18 @@
> >>> return list;
> >>> }
> >>>
> >>> - public List getCompileDependencies()
> >>> + public List<Dependency> getCompileDependencies()
> >>> {
> >>> - Set artifacts = getArtifacts();
> >>> + Set<Artifact> artifacts = getArtifacts();
> >>>
> >>> if ( artifacts == null || artifacts.isEmpty() )
> >>> {
> >>> - return Collections.EMPTY_LIST;
> >>> + return Collections.<Dependency> emptyList();
> >>> }
> >>>
> >>> - List list = new ArrayList( artifacts.size() );
> >>> + List<Dependency> list = new ArrayList<Dependency>(
> >>> artifacts.size() );
> >>>
> >>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
> >>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
> >>> i.hasNext(); )
> >>> {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -619,16 +622,16 @@
> >>> return list;
> >>> }
> >>>
> >>> - public List getTestClasspathElements()
> >>> + public List<String> getTestClasspathElements()
> >>> throws DependencyResolutionRequiredException
> >>> {
> >>> - List list = new ArrayList( getArtifacts().size() + 1 );
> >>> + List<String> list = new ArrayList<String>(
> >>> getArtifacts().size() + 1 );
> >>>
> >>> list.add( getBuild().getTestOutputDirectory() );
> >>>
> >>> list.add( getBuild().getOutputDirectory() );
> >>>
> >>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
> >>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
> >>> i.hasNext(); )
> >>> {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -652,11 +655,11 @@
> >>> return list;
> >>> }
> >>>
> >>> - public List getTestArtifacts()
> >>> + public List<Artifact> getTestArtifacts()
> >>> {
> >>> - List list = new ArrayList( getArtifacts().size() );
> >>> + List<Artifact> list = new ArrayList<Artifact>(
> >>> getArtifacts().size() );
> >>>
> >>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
> >>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
> >>> i.hasNext(); )
> >>> {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -678,18 +681,18 @@
> >>> return list;
> >>> }
> >>>
> >>> - public List getTestDependencies()
> >>> + public List<Dependency> getTestDependencies()
> >>> {
> >>> - Set artifacts = getArtifacts();
> >>> + Set<Artifact> artifacts = getArtifacts();
> >>>
> >>> if ( artifacts == null || artifacts.isEmpty() )
> >>> {
> >>> - return Collections.EMPTY_LIST;
> >>> + return Collections.emptyList();
> >>> }
> >>>
> >>> - List list = new ArrayList( artifacts.size() );
> >>> + List<Dependency> list = new ArrayList<Dependency>(
> >>> artifacts.size() );
> >>>
> >>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
> >>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
> >>> i.hasNext(); )
> >>> {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -715,14 +718,14 @@
> >>> return list;
> >>> }
> >>>
> >>> - public List getRuntimeClasspathElements()
> >>> + public List<String> getRuntimeClasspathElements()
> >>> throws DependencyResolutionRequiredException
> >>> {
> >>> - List list = new ArrayList( getArtifacts().size() + 1 );
> >>> + List<String> list = new ArrayList<String>(
> >>> getArtifacts().size() + 1 );
> >>>
> >>> list.add( getBuild().getOutputDirectory() );
> >>>
> >>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
> >>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
> >>> i.hasNext(); )
> >>> {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -743,11 +746,11 @@
> >>> return list;
> >>> }
> >>>
> >>> - public List getRuntimeArtifacts()
> >>> + public List<Artifact> getRuntimeArtifacts()
> >>> {
> >>> - List list = new ArrayList( getArtifacts().size() );
> >>> + List<Artifact> list = new ArrayList<Artifact>(
> >>> getArtifacts().size() );
> >>>
> >>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
> >>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
> >>> i.hasNext(); )
> >>> {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -764,18 +767,18 @@
> >>> return list;
> >>> }
> >>>
> >>> - public List getRuntimeDependencies()
> >>> + public List<Dependency> getRuntimeDependencies()
> >>> {
> >>> - Set artifacts = getArtifacts();
> >>> + Set<Artifact> artifacts = getArtifacts();
> >>>
> >>> if ( artifacts == null || artifacts.isEmpty() )
> >>> {
> >>> - return Collections.EMPTY_LIST;
> >>> + return Collections.<Dependency> emptyList();
> >>> }
> >>>
> >>> - List list = new ArrayList( artifacts.size() );
> >>> + List<Dependency> list = new ArrayList<Dependency>(
> >>> artifacts.size() );
> >>>
> >>> - for ( Iterator i = artifacts.iterator(); i.hasNext(); )
> >>> + for ( Iterator<Artifact> i = artifacts.iterator();
> >>> i.hasNext(); )
> >>> {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -797,14 +800,14 @@
> >>> return list;
> >>> }
> >>>
> >>> - public List getSystemClasspathElements()
> >>> + public List<String> getSystemClasspathElements()
> >>> throws DependencyResolutionRequiredException
> >>> {
> >>> - List list = new ArrayList( getArtifacts().size() );
> >>> + List<String> list = new ArrayList<String>(
> >>> getArtifacts().size() );
> >>>
> >>> list.add( getBuild().getOutputDirectory() );
> >>>
> >>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
> >>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
> >>> i.hasNext(); )
> >>> {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -820,11 +823,11 @@
> >>> return list;
> >>> }
> >>>
> >>> - public List getSystemArtifacts()
> >>> + public List<Artifact> getSystemArtifacts()
> >>> {
> >>> - List list = new ArrayList( getArtifacts().size() );
> >>> + List<Artifact> list = new ArrayList<Artifact>(
> >>> getArtifacts().size() );
> >>>
> >>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
> >>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
> >>> i.hasNext(); )
> >>> {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -841,18 +844,18 @@
> >>> return list;
> >>> }
> >>>
> >>> - public List getSystemDependencies()
> >>> + public List<Dependency> getSystemDependencies()
> >>> {
> >>> - Set artifacts = getArtifacts();
> >>> + Set<Artifact> artifacts = getArtifacts();
> >>>
> >>> if ( artifacts == null || artifacts.isEmpty() )
> >>> {
> >>> - return Collections.EMPTY_LIST;
> >>> + return Collections.<Dependency> emptyList();
> >>> }
> >>>
> >>> - List list = new ArrayList( artifacts.size() );
> >>> + List<Dependency> list = new ArrayList<Dependency>(
> >>> artifacts.size() );
> >>>
> >>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
> >>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
> >>> i.hasNext(); )
> >>> {
> >>> Artifact a = (Artifact) i.next();
> >>>
> >>> @@ -1050,12 +1053,12 @@
> >>> return getModel().getScm();
> >>> }
> >>>
> >>> - public void setMailingLists( List mailingLists )
> >>> + public void setMailingLists( List<MailingList> mailingLists )
> >>> {
> >>> getModel().setMailingLists( mailingLists );
> >>> }
> >>>
> >>> - public List getMailingLists()
> >>> + public List<?> getMailingLists()
> >>> {
> >>> return getModel().getMailingLists();
> >>> }
> >>> @@ -1065,12 +1068,12 @@
> >>> getModel().addMailingList( mailingList );
> >>> }
> >>>
> >>> - public void setDevelopers( List developers )
> >>> + public void setDevelopers( List<Developer> developers )
> >>> {
> >>> getModel().setDevelopers( developers );
> >>> }
> >>>
> >>> - public List getDevelopers()
> >>> + public List<Developer> getDevelopers()
> >>> {
> >>> return getModel().getDevelopers();
> >>> }
> >>> @@ -1080,12 +1083,12 @@
> >>> getModel().addDeveloper( developer );
> >>> }
> >>>
> >>> - public void setContributors( List contributors )
> >>> + public void setContributors( List<Contributor> contributors )
> >>> {
> >>> getModel().setContributors( contributors );
> >>> }
> >>>
> >>> - public List getContributors()
> >>> + public List<Contributor> getContributors()
> >>> {
> >>> return getModel().getContributors();
> >>> }
> >>> @@ -1105,12 +1108,12 @@
> >>> return getModelBuild();
> >>> }
> >>>
> >>> - public List getResources()
> >>> + public List<Resource> getResources()
> >>> {
> >>> return getBuild().getResources();
> >>> }
> >>>
> >>> - public List getTestResources()
> >>> + public List<Resource> getTestResources()
> >>> {
> >>> return getBuild().getTestResources();
> >>> }
> >>> @@ -1135,12 +1138,12 @@
> >>> return getModel().getReporting();
> >>> }
> >>>
> >>> - public void setLicenses( List licenses )
> >>> + public void setLicenses( List<License> licenses )
> >>> {
> >>> getModel().setLicenses( licenses );
> >>> }
> >>>
> >>> - public List getLicenses()
> >>> + public List<License> getLicenses()
> >>> {
> >>> return getModel().getLicenses();
> >>> }
> >>> @@ -1150,7 +1153,7 @@
> >>> getModel().addLicense( license );
> >>> }
> >>>
> >>> - public void setArtifacts( Set artifacts )
> >>> + public void setArtifacts( Set<Artifact> artifacts )
> >>> {
> >>> this.artifacts = artifacts;
> >>>
> >>> @@ -1165,12 +1168,12 @@
> >>> * @return {@link Set} < {@link Artifact} >
> >>> * @see #getDependencyArtifacts() to get only direct dependencies
> >>> */
> >>> - public Set getArtifacts()
> >>> + public Set<Artifact> getArtifacts()
> >>> {
> >>> - return artifacts == null ? Collections.EMPTY_SET : artifacts;
> >>> + return artifacts == null ? Collections.<Artifact> emptySet() :
> >>> artifacts;
> >>> }
> >>>
> >>> - public Map getArtifactMap()
> >>> + public Map<String, Artifact> getArtifactMap()
> >>> {
> >>> if ( artifactMap == null )
> >>> {
> >>> @@ -1180,19 +1183,19 @@
> >>> return artifactMap;
> >>> }
> >>>
> >>> - public void setPluginArtifacts( Set pluginArtifacts )
> >>> + public void setPluginArtifacts( Set<Artifact> pluginArtifacts )
> >>> {
> >>> this.pluginArtifacts = pluginArtifacts;
> >>>
> >>> this.pluginArtifactMap = null;
> >>> }
> >>>
> >>> - public Set getPluginArtifacts()
> >>> + public Set<Artifact> getPluginArtifacts()
> >>> {
> >>> return pluginArtifacts;
> >>> }
> >>>
> >>> - public Map getPluginArtifactMap()
> >>> + public Map<String, Artifact> getPluginArtifactMap()
> >>> {
> >>> if ( pluginArtifactMap == null )
> >>> {
> >>> @@ -1202,19 +1205,19 @@
> >>> return pluginArtifactMap;
> >>> }
> >>>
> >>> - public void setReportArtifacts( Set reportArtifacts )
> >>> + public void setReportArtifacts( Set<Artifact> reportArtifacts )
> >>> {
> >>> this.reportArtifacts = reportArtifacts;
> >>>
> >>> this.reportArtifactMap = null;
> >>> }
> >>>
> >>> - public Set getReportArtifacts()
> >>> + public Set<Artifact> getReportArtifacts()
> >>> {
> >>> return reportArtifacts;
> >>> }
> >>>
> >>> - public Map getReportArtifactMap()
> >>> + public Map<String, Artifact> getReportArtifactMap()
> >>> {
> >>> if ( reportArtifactMap == null )
> >>> {
> >>> @@ -1224,19 +1227,19 @@
> >>> return reportArtifactMap;
> >>> }
> >>>
> >>> - public void setExtensionArtifacts( Set extensionArtifacts )
> >>> + public void setExtensionArtifacts( Set<Artifact>
> >>> extensionArtifacts )
> >>> {
> >>> this.extensionArtifacts = extensionArtifacts;
> >>>
> >>> this.extensionArtifactMap = null;
> >>> }
> >>>
> >>> - public Set getExtensionArtifacts()
> >>> + public Set<Artifact> getExtensionArtifacts()
> >>> {
> >>> return this.extensionArtifacts;
> >>> }
> >>>
> >>> - public Map getExtensionArtifactMap()
> >>> + public Map<String, Artifact> getExtensionArtifactMap()
> >>> {
> >>> if ( extensionArtifactMap == null )
> >>> {
> >>> @@ -1256,7 +1259,7 @@
> >>> return parentArtifact;
> >>> }
> >>>
> >>> - public List getRepositories()
> >>> + public List<Repository> getRepositories()
> >>> {
> >>> return getModel().getRepositories();
> >>> }
> >>> @@ -1265,7 +1268,7 @@
> >>> // Plugins
> >>> //
> >>> ----------------------------------------------------------------------
> >>>
> >>> - public List getReportPlugins()
> >>> + public List<ReportPlugin> getReportPlugins()
> >>> {
> >>> if ( getModel().getReporting() == null )
> >>> {
> >>> @@ -1275,7 +1278,7 @@
> >>>
> >>> }
> >>>
> >>> - public List getBuildPlugins()
> >>> + public List<Plugin> getBuildPlugins()
> >>> {
> >>> if ( getModel().getBuild() == null )
> >>> {
> >>> @@ -1284,7 +1287,7 @@
> >>> return getModel().getBuild().getPlugins();
> >>> }
> >>>
> >>> - public List getModules()
> >>> + public List<String> getModules()
> >>> {
> >>> return getModel().getModules();
> >>> }
> >>> @@ -1335,7 +1338,7 @@
> >>>
> >>> if ( pm != null )
> >>> {
> >>> - Map pmByKey = pm.getPluginsAsMap();
> >>> + Map<String, Plugin> pmByKey = pm.getPluginsAsMap();
> >>>
> >>> String pluginKey = plugin.getKey();
> >>>
> >>> @@ -1348,17 +1351,17 @@
> >>> }
> >>> }
> >>>
> >>> - public List getCollectedProjects()
> >>> + public List<MavenProject> getCollectedProjects()
> >>> {
> >>> return collectedProjects;
> >>> }
> >>>
> >>> - public void setCollectedProjects( List collectedProjects )
> >>> + public void setCollectedProjects( List<MavenProject>
> >>> collectedProjects )
> >>> {
> >>> this.collectedProjects = collectedProjects;
> >>> }
> >>>
> >>> - public void setPluginArtifactRepositories( List
> >>> pluginArtifactRepositories )
> >>> + public void setPluginArtifactRepositories(
> >>> List<ArtifactRepository> pluginArtifactRepositories )
> >>> {
> >>> this.pluginArtifactRepositories = pluginArtifactRepositories;
> >>> }
> >>> @@ -1367,7 +1370,7 @@
> >>> * @return a list of ArtifactRepository objects constructed
> >>> * from the Repository objects returned by
> >>> getPluginRepositories.
> >>> */
> >>> - public List getPluginArtifactRepositories()
> >>> + public List<ArtifactRepository> getPluginArtifactRepositories()
> >>> {
> >>> return pluginArtifactRepositories;
> >>> }
> >>> @@ -1378,17 +1381,17 @@
> >>>
> >>> : getReleaseArtifactRepository();
> >>>
> >>> }
> >>>
> >>> - public List getPluginRepositories()
> >>> + public List<Repository> getPluginRepositories()
> >>> {
> >>> return getModel().getPluginRepositories();
> >>> }
> >>>
> >>> - public void setActiveProfiles( List activeProfiles )
> >>> + public void setActiveProfiles( List<String> activeProfiles )
> >>> {
> >>> this.activeProfiles.addAll( activeProfiles );
> >>> }
> >>>
> >>> - public List getActiveProfiles()
> >>> + public List<String> getActiveProfiles()
> >>> {
> >>> return activeProfiles;
> >>> }
> >>> @@ -1398,11 +1401,11 @@
> >>> getAttachedArtifacts().add( artifact );
> >>> }
> >>>
> >>> - public List getAttachedArtifacts()
> >>> + public List<Artifact> getAttachedArtifacts()
> >>> {
> >>> if ( attachedArtifacts == null )
> >>> {
> >>> - attachedArtifacts = new ArrayList();
> >>> + attachedArtifacts = new ArrayList<Artifact>();
> >>> }
> >>> return attachedArtifacts;
> >>> }
> >>> @@ -1420,7 +1423,7 @@
> >>>
> >>> if ( getBuildPlugins() != null )
> >>> {
> >>> - for ( Iterator iterator = getBuildPlugins().iterator();
> >>> iterator.hasNext(); )
> >>> + for ( Iterator<Plugin> iterator =
> >>> getBuildPlugins().iterator(); iterator.hasNext(); )
> >>> {
> >>> Plugin plugin = (Plugin) iterator.next();
> >>>
> >>> @@ -1498,7 +1501,7 @@
> >>>
> >>> if ( getReportPlugins() != null )
> >>> {
> >>> - for ( Iterator iterator = getReportPlugins().iterator();
> >>> iterator.hasNext(); )
> >>> + for ( Iterator<ReportPlugin> iterator =
> >>> getReportPlugins().iterator(); iterator.hasNext(); )
> >>> {
> >>> ReportPlugin plugin = (ReportPlugin) iterator.next();
> >>>
> >>> @@ -1564,12 +1567,12 @@
> >>> * @return {@link Set} < {@link Artifact} >
> >>> * @see #getArtifacts() to get all transitive dependencies
> >>> */
> >>> - public Set getDependencyArtifacts()
> >>> + public Set<Artifact> getDependencyArtifacts()
> >>> {
> >>> return dependencyArtifacts;
> >>> }
> >>>
> >>> - public void setDependencyArtifacts( Set dependencyArtifacts )
> >>> + public void setDependencyArtifacts( Set<Artifact>
> >>> dependencyArtifacts )
> >>> {
> >>> this.dependencyArtifacts = dependencyArtifacts;
> >>> }
> >>> @@ -1594,12 +1597,12 @@
> >>> return originalModel;
> >>> }
> >>>
> >>> - public void setManagedVersionMap( Map map )
> >>> + public void setManagedVersionMap( Map<String, ArtifactVersion> map
> >>> ) {
> >>> this.managedVersionMap = map;
> >>> }
> >>>
> >>> - public Map getManagedVersionMap()
> >>> + public Map<String, ArtifactVersion> getManagedVersionMap()
> >>> {
> >>> return this.managedVersionMap;
> >>> }
> >>> @@ -1627,12 +1630,12 @@
> >>> return getId().hashCode();
> >>> }
> >>>
> >>> - public List getBuildExtensions()
> >>> + public List<Extension> getBuildExtensions()
> >>> {
> >>> Build build = getBuild();
> >>> if ( build == null || build.getExtensions() == null )
> >>> {
> >>> - return Collections.EMPTY_LIST;
> >>> + return Collections.<Extension> emptyList();
> >>> }
> >>> else
> >>> {
> >>> @@ -1644,7 +1647,7 @@
> >>> * @todo the lazy initialisation of this makes me uneasy.
> >>> * @return {@link Set} < {@link Artifact} >
> >>> */
> >>> - public Set createArtifacts( ArtifactFactory artifactFactory,
> >>> String inheritedScope,
> >>> + public Set<Artifact> createArtifacts( ArtifactFactory
> >>> artifactFactory, String inheritedScope,
> >>> ArtifactFilter dependencyFilter )
> >>> throws InvalidDependencyVersionException
> >>> {
> >>> @@ -1674,12 +1677,12 @@
> >>> return getModel().getProperties();
> >>> }
> >>>
> >>> - public List getFilters()
> >>> + public List<String> getFilters()
> >>> {
> >>> return getBuild().getFilters();
> >>> }
> >>>
> >>> - public Map getProjectReferences()
> >>> + public Map<String, MavenProject> getProjectReferences()
> >>> {
> >>> return projectReferences;
> >>> }
> >>> @@ -1705,22 +1708,22 @@
> >>> this.model = model;
> >>> }
> >>>
> >>> - protected void setAttachedArtifacts( List attachedArtifacts )
> >>> + protected void setAttachedArtifacts( List<Artifact>
> >>> attachedArtifacts )
> >>> {
> >>> this.attachedArtifacts = attachedArtifacts;
> >>> }
> >>>
> >>> - protected void setCompileSourceRoots( List compileSourceRoots )
> >>> + protected void setCompileSourceRoots( List<String>
> >>> compileSourceRoots )
> >>> {
> >>> this.compileSourceRoots = compileSourceRoots;
> >>> }
> >>>
> >>> - protected void setTestCompileSourceRoots( List
> >>> testCompileSourceRoots )
> >>> + protected void setTestCompileSourceRoots( List<String>
> >>> testCompileSourceRoots )
> >>> {
> >>> this.testCompileSourceRoots = testCompileSourceRoots;
> >>> }
> >>>
> >>> - protected void setScriptSourceRoots( List scriptSourceRoots )
> >>> + protected void setScriptSourceRoots( List<String>
> >>> scriptSourceRoots )
> >>> {
> >>> this.scriptSourceRoots = scriptSourceRoots;
> >>> }
> >>> @@ -1737,16 +1740,16 @@
> >>>
> >>> public void resolveActiveArtifacts()
> >>> {
> >>> - Set depArtifacts = getDependencyArtifacts();
> >>> + Set<Artifact> depArtifacts = getDependencyArtifacts();
> >>> if ( depArtifacts == null )
> >>> {
> >>> return;
> >>> }
> >>>
> >>> - Set updated = new LinkedHashSet( depArtifacts.size() );
> >>> + Set<Artifact> updated = new LinkedHashSet<Artifact>(
> >>> depArtifacts.size() );
> >>> int updatedCount = 0;
> >>>
> >>> - for ( Iterator it = depArtifacts.iterator(); it.hasNext(); )
> >>> + for ( Iterator<Artifact> it = depArtifacts.iterator();
> >>> it.hasNext(); )
> >>> {
> >>> Artifact depArtifact = (Artifact) it.next();
> >>> Artifact replaced = replaceWithActiveArtifact( depArtifact
> >>> ); @@ -1815,13 +1818,13 @@
> >>> * @param requestedArtifact The artifact to resolve, must not be
> >>> <code>null</code>.
> >>> * @return The matching artifact or <code>null</code> if not found.
> >>> */
> >>> - private Artifact findMatchingArtifact( List artifacts, Artifact
> >>> requestedArtifact )
> >>> + private Artifact findMatchingArtifact( List<Artifact> artifacts,
> >>> Artifact requestedArtifact )
> >>> {
> >>> if ( artifacts != null && !artifacts.isEmpty() )
> >>> {
> >>> // first try matching by dependency conflict id
> >>> String requestedId =
> >>> requestedArtifact.getDependencyConflictId();
> >>> - for ( Iterator it = artifacts.iterator(); it.hasNext(); )
> >>> + for ( Iterator<Artifact> it = artifacts.iterator();
> >>> it.hasNext(); )
> >>> {
> >>> Artifact artifact = (Artifact) it.next();
> >>> if ( requestedId.equals(
> >>> artifact.getDependencyConflictId() ) )
> >>> @@ -1832,7 +1835,7 @@
> >>>
> >>> // next try matching by repository conflict id
> >>> requestedId = getRepositoryConflictId( requestedArtifact );
> >>> - for ( Iterator it = artifacts.iterator(); it.hasNext(); )
> >>> + for ( Iterator<Artifact> it = artifacts.iterator();
> >>> it.hasNext(); )
> >>> {
> >>> Artifact artifact = (Artifact) it.next();
> >>> if ( requestedId.equals( getRepositoryConflictId(
> >>> artifact ) ) )
> >>> @@ -1898,7 +1901,7 @@
> >>> }
> >>> }
> >>>
> >>> - private void addArtifactPath(Artifact a, List list) throws
> >>> DependencyResolutionRequiredException
> >>> + private void addArtifactPath(Artifact a, List<String> list) throws
> >>> DependencyResolutionRequiredException
> >>> {
> >>> File file = a.getFile();
> >>> if ( file == null )
> >>> @@ -1954,21 +1957,21 @@
> >>>
> >>> private Build originalInterpolatedBuild;
> >>>
> >>> - private List dynamicCompileSourceRoots;
> >>> + private List<String> dynamicCompileSourceRoots;
> >>>
> >>> - private List originalInterpolatedCompileSourceRoots;
> >>> + private List<String> originalInterpolatedCompileSourceRoots;
> >>>
> >>> - private List dynamicTestCompileSourceRoots;
> >>> + private List<String> dynamicTestCompileSourceRoots;
> >>>
> >>> - private List originalInterpolatedTestCompileSourceRoots;
> >>> + private List<String> originalInterpolatedTestCompileSourceRoots;
> >>>
> >>> - private List dynamicScriptSourceRoots;
> >>> + private List<String> dynamicScriptSourceRoots;
> >>>
> >>> - private List originalInterpolatedScriptSourceRoots;
> >>> + private List<String> originalInterpolatedScriptSourceRoots;
> >>>
> >>> private boolean isConcrete = false;
> >>>
> >>> - public boolean isConcrete()
> >>> + public boolean isConcrete()
> >>> {
> >>> return isConcrete;
> >>> }
> >>> @@ -1988,32 +1991,32 @@
> >>> return originalInterpolatedBuild;
> >>> }
> >>>
> >>> - public List getDynamicCompileSourceRoots()
> >>> + public List<String> getDynamicCompileSourceRoots()
> >>> {
> >>> return dynamicCompileSourceRoots;
> >>> }
> >>>
> >>> - public List getOriginalInterpolatedCompileSourceRoots()
> >>> + public List<String> getOriginalInterpolatedCompileSourceRoots()
> >>> {
> >>> return originalInterpolatedCompileSourceRoots;
> >>> }
> >>>
> >>> - public List getDynamicTestCompileSourceRoots()
> >>> + public List<String> getDynamicTestCompileSourceRoots()
> >>> {
> >>> return dynamicTestCompileSourceRoots;
> >>> }
> >>>
> >>> - public List getOriginalInterpolatedTestCompileSourceRoots()
> >>> + public List<String>
> >>> getOriginalInterpolatedTestCompileSourceRoots() {
> >>> return originalInterpolatedTestCompileSourceRoots;
> >>> }
> >>>
> >>> - public List getDynamicScriptSourceRoots()
> >>> + public List<String> getDynamicScriptSourceRoots()
> >>> {
> >>> return dynamicScriptSourceRoots;
> >>> }
> >>>
> >>> - public List getOriginalInterpolatedScriptSourceRoots()
> >>> + public List<String> getOriginalInterpolatedScriptSourceRoots()
> >>> {
> >>> return originalInterpolatedScriptSourceRoots;
> >>> }
> >>> @@ -2034,19 +2037,19 @@
> >>> originalInterpolatedBuild = null;
> >>> }
> >>>
> >>> - public void preserveCompileSourceRoots( List
> >>> originalInterpolatedCompileSourceRoots )
> >>> + public void preserveCompileSourceRoots( List<String>
> >>> originalInterpolatedCompileSourceRoots )
> >>> {
> >>> dynamicCompileSourceRoots = getCompileSourceRoots();
> >>> this.originalInterpolatedCompileSourceRoots =
> >>> originalInterpolatedCompileSourceRoots;
> >>> }
> >>>
> >>> - public void preserveTestCompileSourceRoots( List
> >>> originalInterpolatedTestCompileSourceRoots )
> >>> + public void preserveTestCompileSourceRoots( List<String>
> >>> originalInterpolatedTestCompileSourceRoots )
> >>> {
> >>> dynamicTestCompileSourceRoots = getTestCompileSourceRoots();
> >>> this.originalInterpolatedTestCompileSourceRoots =
> >>> originalInterpolatedTestCompileSourceRoots;
> >>> }
> >>>
> >>> - public void preserveScriptSourceRoots( List
> >>> originalInterpolatedScriptSourceRoots )
> >>> + public void preserveScriptSourceRoots( List<String>
> >>> originalInterpolatedScriptSourceRoots )
> >>> {
> >>> dynamicScriptSourceRoots = getScriptSourceRoots();
> >>> this.originalInterpolatedScriptSourceRoots =
> >>> originalInterpolatedScriptSourceRoots;
> >>> @@ -2071,32 +2074,32 @@
> >>> this.originalInterpolatedBuild = originalInterpolatedBuild;
> >>> }
> >>>
> >>> - protected void setDynamicCompileSourceRoots( List
> >>> dynamicCompileSourceRoots )
> >>> + protected void setDynamicCompileSourceRoots( List<String>
> >>> dynamicCompileSourceRoots )
> >>> {
> >>> this.dynamicCompileSourceRoots = dynamicCompileSourceRoots;
> >>> }
> >>>
> >>> - protected void setOriginalInterpolatedCompileSourceRoots( List
> >>> originalInterpolatedCompileSourceRoots )
> >>> + protected void setOriginalInterpolatedCompileSourceRoots(
> >>> List<String> originalInterpolatedCompileSourceRoots )
> >>> {
> >>> this.originalInterpolatedCompileSourceRoots =
> >>> originalInterpolatedCompileSourceRoots;
> >>> }
> >>>
> >>> - protected void setDynamicTestCompileSourceRoots( List
> >>> dynamicTestCompileSourceRoots )
> >>> + protected void setDynamicTestCompileSourceRoots( List<String>
> >>> dynamicTestCompileSourceRoots )
> >>> {
> >>> this.dynamicTestCompileSourceRoots =
> >>> dynamicTestCompileSourceRoots;
> >>> }
> >>>
> >>> - protected void setOriginalInterpolatedTestCompileSourceRoots( List
> >>> originalInterpolatedTestCompileSourceRoots )
> >>> + protected void setOriginalInterpolatedTestCompileSourceRoots(
> >>> List<String> originalInterpolatedTestCompileSourceRoots )
> >>> {
> >>> this.originalInterpolatedTestCompileSourceRoots =
> >>> originalInterpolatedTestCompileSourceRoots;
> >>> }
> >>>
> >>> - protected void setDynamicScriptSourceRoots( List
> >>> dynamicScriptSourceRoots )
> >>> + protected void setDynamicScriptSourceRoots( List<String>
> >>> dynamicScriptSourceRoots )
> >>> {
> >>> this.dynamicScriptSourceRoots = dynamicScriptSourceRoots;
> >>> }
> >>>
> >>> - protected void setOriginalInterpolatedScriptSourceRoots( List
> >>> originalInterpolatedScriptSourceRoots )
> >>> + protected void setOriginalInterpolatedScriptSourceRoots(
> >>> List<String> originalInterpolatedScriptSourceRoots )
> >>> {
> >>> this.originalInterpolatedScriptSourceRoots =
> >>> originalInterpolatedScriptSourceRoots;
> >>> }
> >>> @@ -2114,7 +2117,7 @@
> >>> if ( p != null )
> >>> {
> >>> preservedProperties = new Properties();
> >>> - for( Enumeration e = p.propertyNames();
> >>> e.hasMoreElements(); )
> >>> + for( Enumeration<?> e = p.propertyNames();
> >>> e.hasMoreElements(); )
> >>> {
> >>> String key = (String) e.nextElement();
> >>> preservedProperties.setProperty( key, p.getProperty( key
> >>> ) );
> >>>
> >>> Modified:
> >>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
> >>> URL:
> >>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/mave
> >>>n-repository-metadata/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
> >>> =======================================================================
> >>>=======
> >>>
> >>> ---
> >>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
> >>> (original)
> >>> +++
> >>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
> >>> Fri May 1 08:04:48 2009
> >>> @@ -47,6 +47,7 @@
> >>> <models>
> >>> <model>src/main/mdo/metadata.mdo</model>
> >>> </models>
> >>> + <useJava5>true</useJava5>
> >>> </configuration>
> >>> </plugin>
> >>> </plugins>
> >>>
> >>> Modified: maven/components/branches/maven-2.2.x/maven-settings/pom.xml
> >>> URL:
> >>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/mave
> >>>n-settings/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
> >>> =======================================================================
> >>>=======
> >>>
> >>> --- maven/components/branches/maven-2.2.x/maven-settings/pom.xml
> >>> (original)
> >>> +++ maven/components/branches/maven-2.2.x/maven-settings/pom.xml Fri
> >>> May 1 08:04:48 2009
> >>> @@ -58,6 +58,7 @@
> >>> <models>
> >>> <model>src/main/mdo/settings.mdo</model>
> >>> </models>
> >>> + <useJava5>true</useJava5>
> >>> </configuration>
> >>> </plugin>
> >>> </plugins>
> >>>
> >>> Modified: maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
> >>> URL:
> >>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/mave
> >>>n-toolchain/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
> >>> =======================================================================
> >>>=======
> >>>
> >>> --- maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
> >>> (original)
> >>> +++ maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml Fri
> >>> May 1 08:04:48 2009
> >>> @@ -66,6 +66,7 @@
> >>> <models>
> >>> <model>src/main/mdo/toolchains.mdo</model>
> >>> </models>
> >>> + <useJava5>true</useJava5>
> >>> </configuration>
> >>> </plugin>
> >>> <plugin>
> >>
> >> Thanks,
> >>
> >> Jason
> >>
> >> ----------------------------------------------------------
> >> Jason van Zyl
> >> Founder, Apache Maven
> >> http://twitter.com/jvanzyl
> >> http://twitter.com/SonatypeNexus
> >> http://twitter.com/SonatypeM2E
> >> ----------------------------------------------------------
> >>
> >> We all have problems. How we deal with them is a measure of our worth.
> >>
> >> -- Unknown
> >>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> >> For additional commands, e-mail: dev-help@maven.apache.org
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> > For additional commands, e-mail: dev-help@maven.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: svn commit: r770570 - in /maven/components/branches/maven-2.2.x:
maven-artifact/src/main/java/org/apache/maven/artifact/ maven-model/ maven-profile/
maven-project/src/main/java/org/apache/maven/project/ maven-repository-metadata/
maven-settings/
Posted by John Casey <jd...@commonjava.org>.
Yeah, we're using the version '2.2.0' partially because JDK 1.5 is a
requirement for one of the regression fixes.
Jason van Zyl wrote:
>
> On 2-May-09, at 1:58 PM, John Casey wrote:
>
>> The ITs all looked fine with this commit, so it looks like this stuff
>> won't cause too much grief...but hopefully the testing of the RC will
>> tell.
>>
>
> If it was decided that this should run in only in 1.5 and you don't have
> to worry about a 1.4 runtime then it should be fine. I lost track of
> whether it was decided a 1.4 runtime should continue to work.
>
>> I've created a 2.2.0-RC branch for subsequent RCs and the final
>> release of 2.2.0, so we can continue converting the syntax on the
>> 2.2.x branch.
>>
>> -john
>>
>> BRIAN FOX wrote:
>>> In general, no. I just wanted to draw John's attention to the changes
>>> since I know he was preparing to cut an RC. I am suspicious of all
>>> changes when a release is drawing near.
>>> On May 2, 2009, at 2:44 AM, nicolas de loof wrote:
>>>> Do we prefer plugin developers to use List<?> and get
>>>> ClassCastExceptions at
>>>> runtime or sue type-safe collections to help them create stronger
>>>> code ?Anyway,
>>>> few plugins allready use Java5. Java 1.4 based one will not be
>>>> broken as the
>>>> generics signature is only a compile-time check.
>>>>
>>>>
>>>> 2009/5/1 Brian Fox <br...@infinity.nu>
>>>>
>>>>> I'm not sure if this is in scope of what John is trying to do wrt
>>>>> to 2.2.
>>>>>
>>>>>
>>>>> Jason van Zyl wrote:
>>>>>
>>>>>> I don't believe anyone actually agreed to this yet. Are you sure
>>>>>> this is
>>>>>> not going to cause problems for users?
>>>>>>
>>>>>> On 1-May-09, at 1:04 AM, nicolas@apache.org wrote:
>>>>>>
>>>>>> Author: nicolas
>>>>>>> Date: Fri May 1 08:04:48 2009
>>>>>>> New Revision: 770570
>>>>>>>
>>>>>>> URL: http://svn.apache.org/viewvc?rev=770570&view=rev
>>>>>>> Log:
>>>>>>> use java5 syntax.
>>>>>>> -> set modello to generate java5 syntax
>>>>>>> -> MavenProject updated to use generics on collections & methods for
>>>>>>> better type safety
>>>>>>>
>>>>>>> Modified:
>>>>>>>
>>>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>>>
>>>>>>>
>>>>>>> maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>>>>> maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>>>>>
>>>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>>>>>>
>>>>>>>
>>>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
>>>>>>>
>>>>>>> maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>>>>> maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>>>>>
>>>>>>> Modified:
>>>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>>>
>>>>>>>
>>>>>>> URL:
>>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java?rev=770570&r1=770569&r2=770570&view=diff
>>>>>>>
>>>>>>> ==============================================================================
>>>>>>>
>>>>>>>
>>>>>>> ---
>>>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>>>
>>>>>>> (original)
>>>>>>> +++
>>>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>>>
>>>>>>> Fri May 1 08:04:48 2009
>>>>>>> @@ -86,13 +86,13 @@
>>>>>>> baseVersion;
>>>>>>> }
>>>>>>>
>>>>>>> - public static Map artifactMapByVersionlessId( Collection
>>>>>>> artifacts )
>>>>>>> + public static Map<String, Artifact> artifactMapByVersionlessId(
>>>>>>> Collection<Artifact> artifacts )
>>>>>>> {
>>>>>>> - Map artifactMap = new LinkedHashMap();
>>>>>>> + Map<String, Artifact> artifactMap = new
>>>>>>> LinkedHashMap<String,
>>>>>>> Artifact>();
>>>>>>>
>>>>>>> if ( artifacts != null )
>>>>>>> {
>>>>>>> - for ( Iterator it = artifacts.iterator();
>>>>>>> it.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>>>>> it.hasNext(); )
>>>>>>> {
>>>>>>> Artifact artifact = (Artifact) it.next();
>>>>>>>
>>>>>>>
>>>>>>> Modified: maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>>>>> URL:
>>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-model/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>>>
>>>>>>> ==============================================================================
>>>>>>>
>>>>>>>
>>>>>>> --- maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>>>>> (original)
>>>>>>> +++ maven/components/branches/maven-2.2.x/maven-model/pom.xml Fri
>>>>>>> May 1
>>>>>>> 08:04:48 2009
>>>>>>> @@ -47,6 +47,7 @@
>>>>>>> <models>
>>>>>>> <model>src/main/mdo/maven.mdo</model>
>>>>>>> </models>
>>>>>>> + <useJava5>true</useJava5>
>>>>>>> </configuration>
>>>>>>> </plugin>
>>>>>>> <plugin>
>>>>>>>
>>>>>>> Modified:
>>>>>>> maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>>>>> URL:
>>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-profile/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>>>
>>>>>>> ==============================================================================
>>>>>>>
>>>>>>>
>>>>>>> --- maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>>>>> (original)
>>>>>>> +++ maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>>>>> Fri May
>>>>>>> 1 08:04:48 2009
>>>>>>> @@ -54,6 +54,7 @@
>>>>>>> <models>
>>>>>>> <model>src/main/mdo/profiles.mdo</model>
>>>>>>> </models>
>>>>>>> + <useJava5>true</useJava5>
>>>>>>> </configuration>
>>>>>>> </plugin>
>>>>>>> </plugins>
>>>>>>>
>>>>>>> Modified:
>>>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>>>>>>
>>>>>>>
>>>>>>> URL:
>>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java?rev=770570&r1=770569&r2=770570&view=diff
>>>>>>>
>>>>>>> ==============================================================================
>>>>>>>
>>>>>>>
>>>>>>> ---
>>>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>>>>>>
>>>>>>> (original)
>>>>>>> +++
>>>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>>>>>>
>>>>>>> Fri May 1 08:04:48 2009
>>>>>>> @@ -36,10 +36,11 @@
>>>>>>> import org.apache.maven.artifact.Artifact;
>>>>>>> import org.apache.maven.artifact.ArtifactUtils;
>>>>>>> import
>>>>>>> org.apache.maven.artifact.DependencyResolutionRequiredException;
>>>>>>> -import org.apache.maven.artifact.versioning.ManagedVersionMap;
>>>>>>> import org.apache.maven.artifact.factory.ArtifactFactory;
>>>>>>> import org.apache.maven.artifact.repository.ArtifactRepository;
>>>>>>> import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
>>>>>>> +import org.apache.maven.artifact.versioning.ArtifactVersion;
>>>>>>> +import org.apache.maven.artifact.versioning.ManagedVersionMap;
>>>>>>> import org.apache.maven.model.Build;
>>>>>>> import org.apache.maven.model.CiManagement;
>>>>>>> import org.apache.maven.model.Contributor;
>>>>>>> @@ -47,6 +48,7 @@
>>>>>>> import org.apache.maven.model.DependencyManagement;
>>>>>>> import org.apache.maven.model.Developer;
>>>>>>> import org.apache.maven.model.DistributionManagement;
>>>>>>> +import org.apache.maven.model.Extension;
>>>>>>> import org.apache.maven.model.IssueManagement;
>>>>>>> import org.apache.maven.model.License;
>>>>>>> import org.apache.maven.model.MailingList;
>>>>>>> @@ -59,6 +61,7 @@
>>>>>>> import org.apache.maven.model.ReportPlugin;
>>>>>>> import org.apache.maven.model.ReportSet;
>>>>>>> import org.apache.maven.model.Reporting;
>>>>>>> +import org.apache.maven.model.Repository;
>>>>>>> import org.apache.maven.model.Resource;
>>>>>>> import org.apache.maven.model.Scm;
>>>>>>> import org.apache.maven.model.io.xpp3.MavenXpp3Writer;
>>>>>>> @@ -96,60 +99,60 @@
>>>>>>>
>>>>>>> private File file;
>>>>>>>
>>>>>>> - private Set artifacts;
>>>>>>> + private Set<Artifact> artifacts;
>>>>>>>
>>>>>>> private Artifact parentArtifact;
>>>>>>>
>>>>>>> - private Set pluginArtifacts;
>>>>>>> + private Set<Artifact> pluginArtifacts;
>>>>>>>
>>>>>>> - private List remoteArtifactRepositories;
>>>>>>> + private List<ArtifactRepository> remoteArtifactRepositories;
>>>>>>>
>>>>>>> - private List collectedProjects = Collections.EMPTY_LIST;
>>>>>>> + private List<MavenProject> collectedProjects =
>>>>>>> Collections.<MavenProject> emptyList();
>>>>>>>
>>>>>>> - private List attachedArtifacts;
>>>>>>> + private List<Artifact> attachedArtifacts;
>>>>>>>
>>>>>>> private MavenProject executionProject;
>>>>>>>
>>>>>>> - private List compileSourceRoots = new ArrayList();
>>>>>>> + private List<String> compileSourceRoots = new
>>>>>>> ArrayList<String>();
>>>>>>>
>>>>>>> - private List testCompileSourceRoots = new ArrayList();
>>>>>>> + private List<String> testCompileSourceRoots = new
>>>>>>> ArrayList<String>();
>>>>>>>
>>>>>>> - private List scriptSourceRoots = new ArrayList();
>>>>>>> + private List<String> scriptSourceRoots = new
>>>>>>> ArrayList<String>();
>>>>>>>
>>>>>>> - private List pluginArtifactRepositories;
>>>>>>> + private List<ArtifactRepository> pluginArtifactRepositories;
>>>>>>>
>>>>>>> private ArtifactRepository releaseArtifactRepository;
>>>>>>>
>>>>>>> private ArtifactRepository snapshotArtifactRepository;
>>>>>>>
>>>>>>> - private List activeProfiles = new ArrayList();
>>>>>>> + private List<String> activeProfiles = new ArrayList<String>();
>>>>>>>
>>>>>>> - private Set dependencyArtifacts;
>>>>>>> + private Set<Artifact> dependencyArtifacts;
>>>>>>>
>>>>>>> private Artifact artifact;
>>>>>>>
>>>>>>> // calculated.
>>>>>>> - private Map artifactMap;
>>>>>>> + private Map<String, Artifact> artifactMap;
>>>>>>>
>>>>>>> private Model originalModel;
>>>>>>>
>>>>>>> - private Map pluginArtifactMap;
>>>>>>> + private Map<String, Artifact> pluginArtifactMap;
>>>>>>>
>>>>>>> - private Set reportArtifacts;
>>>>>>> + private Set<Artifact> reportArtifacts;
>>>>>>>
>>>>>>> - private Map reportArtifactMap;
>>>>>>> + private Map<String, Artifact> reportArtifactMap;
>>>>>>>
>>>>>>> - private Set extensionArtifacts;
>>>>>>> + private Set<Artifact> extensionArtifacts;
>>>>>>>
>>>>>>> - private Map extensionArtifactMap;
>>>>>>> + private Map<String, Artifact> extensionArtifactMap;
>>>>>>>
>>>>>>> - private Map managedVersionMap;
>>>>>>> + private Map<String, ArtifactVersion> managedVersionMap;
>>>>>>>
>>>>>>> - private Map projectReferences = new HashMap();
>>>>>>> + private Map<String, MavenProject> projectReferences = new
>>>>>>> HashMap<String, MavenProject>();
>>>>>>>
>>>>>>> private boolean executionRoot;
>>>>>>>
>>>>>>> - private Map moduleAdjustments;
>>>>>>> + private Map<String, String> moduleAdjustments;
>>>>>>>
>>>>>>> private File basedir;
>>>>>>>
>>>>>>> @@ -245,23 +248,23 @@
>>>>>>> if ( project.getAttachedArtifacts() != null )
>>>>>>> {
>>>>>>> // clone properties modifyable by plugins in a forked
>>>>>>> lifecycle
>>>>>>> - setAttachedArtifacts( new ArrayList(
>>>>>>> project.getAttachedArtifacts() ) );
>>>>>>> + setAttachedArtifacts( new ArrayList<Artifact>(
>>>>>>> project.getAttachedArtifacts() ) );
>>>>>>> }
>>>>>>>
>>>>>>> if ( project.getCompileSourceRoots() != null )
>>>>>>> {
>>>>>>> // clone source roots
>>>>>>> - setCompileSourceRoots( ( new ArrayList(
>>>>>>> project.getCompileSourceRoots() ) ) );
>>>>>>> + setCompileSourceRoots( ( new ArrayList<String>(
>>>>>>> project.getCompileSourceRoots() ) ) );
>>>>>>> }
>>>>>>>
>>>>>>> if ( project.getTestCompileSourceRoots() != null )
>>>>>>> {
>>>>>>> - setTestCompileSourceRoots( ( new ArrayList(
>>>>>>> project.getTestCompileSourceRoots() ) ) );
>>>>>>> + setTestCompileSourceRoots( ( new ArrayList<String>(
>>>>>>> project.getTestCompileSourceRoots() ) ) );
>>>>>>> }
>>>>>>>
>>>>>>> if ( project.getScriptSourceRoots() != null )
>>>>>>> {
>>>>>>> - setScriptSourceRoots( ( new ArrayList(
>>>>>>> project.getScriptSourceRoots() ) ) );
>>>>>>> + setScriptSourceRoots( ( new ArrayList<String>(
>>>>>>> project.getScriptSourceRoots() ) ) );
>>>>>>> }
>>>>>>>
>>>>>>> setModel( ( ModelUtils.cloneModel( project.getModel() ) ) );
>>>>>>> @@ -298,25 +301,25 @@
>>>>>>> setDynamicBuild( ModelUtils.cloneBuild(
>>>>>>> project.getDynamicBuild() ) );
>>>>>>> setOriginalInterpolatedBuild( ModelUtils.cloneBuild(
>>>>>>> project.getOriginalInterpolatedBuild() ) );
>>>>>>>
>>>>>>> - List dynamicRoots =
>>>>>>> project.getDynamicCompileSourceRoots();
>>>>>>> + List<String> dynamicRoots =
>>>>>>> project.getDynamicCompileSourceRoots();
>>>>>>> if ( dynamicRoots != null )
>>>>>>> {
>>>>>>> - setDynamicCompileSourceRoots( new ArrayList(
>>>>>>> dynamicRoots ) );
>>>>>>> - setOriginalInterpolatedCompileSourceRoots( new
>>>>>>> ArrayList( project.getOriginalInterpolatedCompileSourceRoots() ) );
>>>>>>> + setDynamicCompileSourceRoots( new
>>>>>>> ArrayList<String>(
>>>>>>> dynamicRoots ) );
>>>>>>> + setOriginalInterpolatedCompileSourceRoots( new
>>>>>>> ArrayList<String>(
>>>>>>> project.getOriginalInterpolatedCompileSourceRoots() ) );
>>>>>>> }
>>>>>>>
>>>>>>> dynamicRoots = project.getDynamicTestCompileSourceRoots();
>>>>>>> if ( dynamicRoots != null )
>>>>>>> {
>>>>>>> - setDynamicTestCompileSourceRoots( new ArrayList(
>>>>>>> dynamicRoots ) );
>>>>>>> - setOriginalInterpolatedTestCompileSourceRoots( new
>>>>>>> ArrayList(
>>>>>>> project.getOriginalInterpolatedTestCompileSourceRoots() ) );
>>>>>>> + setDynamicTestCompileSourceRoots( new
>>>>>>> ArrayList<String>(
>>>>>>> dynamicRoots ) );
>>>>>>> + setOriginalInterpolatedTestCompileSourceRoots( new
>>>>>>> ArrayList<String>(
>>>>>>> project.getOriginalInterpolatedTestCompileSourceRoots() )
>>>>>>> );
>>>>>>> }
>>>>>>>
>>>>>>> dynamicRoots = project.getDynamicScriptSourceRoots();
>>>>>>> if ( dynamicRoots != null )
>>>>>>> {
>>>>>>> - setDynamicScriptSourceRoots( new ArrayList(
>>>>>>> dynamicRoots
>>>>>>> ) );
>>>>>>> - setOriginalInterpolatedScriptSourceRoots( new
>>>>>>> ArrayList(
>>>>>>> project.getOriginalInterpolatedScriptSourceRoots() ) );
>>>>>>> + setDynamicScriptSourceRoots( new ArrayList<String>(
>>>>>>> dynamicRoots ) );
>>>>>>> + setOriginalInterpolatedScriptSourceRoots( new
>>>>>>> ArrayList<String>(
>>>>>>> project.getOriginalInterpolatedScriptSourceRoots() ) );
>>>>>>> }
>>>>>>> }
>>>>>>>
>>>>>>> @@ -347,12 +350,12 @@
>>>>>>>
>>>>>>> if ( moduleAdjustments == null )
>>>>>>> {
>>>>>>> - moduleAdjustments = new HashMap();
>>>>>>> + moduleAdjustments = new HashMap<String, String>();
>>>>>>>
>>>>>>> - List modules = getModules();
>>>>>>> + List<String> modules = getModules();
>>>>>>> if ( modules != null )
>>>>>>> {
>>>>>>> - for ( Iterator it = modules.iterator();
>>>>>>> it.hasNext(); )
>>>>>>> + for ( Iterator<String> it = modules.iterator();
>>>>>>> it.hasNext(); )
>>>>>>> {
>>>>>>> String modulePath = (String) it.next();
>>>>>>> String moduleName = modulePath;
>>>>>>> @@ -415,12 +418,12 @@
>>>>>>> this.parent = parent;
>>>>>>> }
>>>>>>>
>>>>>>> - public void setRemoteArtifactRepositories( List
>>>>>>> remoteArtifactRepositories )
>>>>>>> + public void setRemoteArtifactRepositories(
>>>>>>> List<ArtifactRepository>
>>>>>>> remoteArtifactRepositories )
>>>>>>> {
>>>>>>> this.remoteArtifactRepositories = remoteArtifactRepositories;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getRemoteArtifactRepositories()
>>>>>>> + public List<ArtifactRepository> getRemoteArtifactRepositories()
>>>>>>> {
>>>>>>> return remoteArtifactRepositories;
>>>>>>> }
>>>>>>> @@ -460,12 +463,12 @@
>>>>>>> return basedir;
>>>>>>> }
>>>>>>>
>>>>>>> - public void setDependencies( List dependencies )
>>>>>>> + public void setDependencies( List<Dependency> dependencies )
>>>>>>> {
>>>>>>> getModel().setDependencies( dependencies );
>>>>>>> }
>>>>>>>
>>>>>>> - public List getDependencies()
>>>>>>> + public List<Dependency> getDependencies()
>>>>>>> {
>>>>>>> return getModel().getDependencies();
>>>>>>> }
>>>>>>> @@ -524,29 +527,29 @@
>>>>>>> }
>>>>>>> }
>>>>>>>
>>>>>>> - public List getCompileSourceRoots()
>>>>>>> + public List<String> getCompileSourceRoots()
>>>>>>> {
>>>>>>> return compileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getScriptSourceRoots()
>>>>>>> + public List<String> getScriptSourceRoots()
>>>>>>> {
>>>>>>> return scriptSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getTestCompileSourceRoots()
>>>>>>> + public List<String> getTestCompileSourceRoots()
>>>>>>> {
>>>>>>> return testCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getCompileClasspathElements()
>>>>>>> + public List<String> getCompileClasspathElements()
>>>>>>> throws DependencyResolutionRequiredException
>>>>>>> {
>>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>>> + List<String> list = new ArrayList<String>(
>>>>>>> getArtifacts().size()
>>>>>>> );
>>>>>>>
>>>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>>>
>>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> + for ( Iterator<?> i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -563,11 +566,11 @@
>>>>>>> return list;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getCompileArtifacts()
>>>>>>> + public List<Artifact> getCompileArtifacts()
>>>>>>> {
>>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>>>> getArtifacts().size() );
>>>>>>>
>>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -585,18 +588,18 @@
>>>>>>> return list;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getCompileDependencies()
>>>>>>> + public List<Dependency> getCompileDependencies()
>>>>>>> {
>>>>>>> - Set artifacts = getArtifacts();
>>>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>>>
>>>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>>>> {
>>>>>>> - return Collections.EMPTY_LIST;
>>>>>>> + return Collections.<Dependency> emptyList();
>>>>>>> }
>>>>>>>
>>>>>>> - List list = new ArrayList( artifacts.size() );
>>>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>>>> artifacts.size() );
>>>>>>>
>>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -619,16 +622,16 @@
>>>>>>> return list;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getTestClasspathElements()
>>>>>>> + public List<String> getTestClasspathElements()
>>>>>>> throws DependencyResolutionRequiredException
>>>>>>> {
>>>>>>> - List list = new ArrayList( getArtifacts().size() + 1 );
>>>>>>> + List<String> list = new ArrayList<String>(
>>>>>>> getArtifacts().size()
>>>>>>> + 1 );
>>>>>>>
>>>>>>> list.add( getBuild().getTestOutputDirectory() );
>>>>>>>
>>>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>>>
>>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -652,11 +655,11 @@
>>>>>>> return list;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getTestArtifacts()
>>>>>>> + public List<Artifact> getTestArtifacts()
>>>>>>> {
>>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>>>> getArtifacts().size() );
>>>>>>>
>>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -678,18 +681,18 @@
>>>>>>> return list;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getTestDependencies()
>>>>>>> + public List<Dependency> getTestDependencies()
>>>>>>> {
>>>>>>> - Set artifacts = getArtifacts();
>>>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>>>
>>>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>>>> {
>>>>>>> - return Collections.EMPTY_LIST;
>>>>>>> + return Collections.emptyList();
>>>>>>> }
>>>>>>>
>>>>>>> - List list = new ArrayList( artifacts.size() );
>>>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>>>> artifacts.size() );
>>>>>>>
>>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -715,14 +718,14 @@
>>>>>>> return list;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getRuntimeClasspathElements()
>>>>>>> + public List<String> getRuntimeClasspathElements()
>>>>>>> throws DependencyResolutionRequiredException
>>>>>>> {
>>>>>>> - List list = new ArrayList( getArtifacts().size() + 1 );
>>>>>>> + List<String> list = new ArrayList<String>(
>>>>>>> getArtifacts().size()
>>>>>>> + 1 );
>>>>>>>
>>>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>>>
>>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -743,11 +746,11 @@
>>>>>>> return list;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getRuntimeArtifacts()
>>>>>>> + public List<Artifact> getRuntimeArtifacts()
>>>>>>> {
>>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>>>> getArtifacts().size() );
>>>>>>>
>>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -764,18 +767,18 @@
>>>>>>> return list;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getRuntimeDependencies()
>>>>>>> + public List<Dependency> getRuntimeDependencies()
>>>>>>> {
>>>>>>> - Set artifacts = getArtifacts();
>>>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>>>
>>>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>>>> {
>>>>>>> - return Collections.EMPTY_LIST;
>>>>>>> + return Collections.<Dependency> emptyList();
>>>>>>> }
>>>>>>>
>>>>>>> - List list = new ArrayList( artifacts.size() );
>>>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>>>> artifacts.size() );
>>>>>>>
>>>>>>> - for ( Iterator i = artifacts.iterator(); i.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> i = artifacts.iterator();
>>>>>>> i.hasNext();
>>>>>>> )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -797,14 +800,14 @@
>>>>>>> return list;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getSystemClasspathElements()
>>>>>>> + public List<String> getSystemClasspathElements()
>>>>>>> throws DependencyResolutionRequiredException
>>>>>>> {
>>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>>> + List<String> list = new ArrayList<String>(
>>>>>>> getArtifacts().size()
>>>>>>> );
>>>>>>>
>>>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>>>
>>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -820,11 +823,11 @@
>>>>>>> return list;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getSystemArtifacts()
>>>>>>> + public List<Artifact> getSystemArtifacts()
>>>>>>> {
>>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>>>> getArtifacts().size() );
>>>>>>>
>>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -841,18 +844,18 @@
>>>>>>> return list;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getSystemDependencies()
>>>>>>> + public List<Dependency> getSystemDependencies()
>>>>>>> {
>>>>>>> - Set artifacts = getArtifacts();
>>>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>>>
>>>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>>>> {
>>>>>>> - return Collections.EMPTY_LIST;
>>>>>>> + return Collections.<Dependency> emptyList();
>>>>>>> }
>>>>>>>
>>>>>>> - List list = new ArrayList( artifacts.size() );
>>>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>>>> artifacts.size() );
>>>>>>>
>>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>>> i.hasNext(); )
>>>>>>> {
>>>>>>> Artifact a = (Artifact) i.next();
>>>>>>>
>>>>>>> @@ -1050,12 +1053,12 @@
>>>>>>> return getModel().getScm();
>>>>>>> }
>>>>>>>
>>>>>>> - public void setMailingLists( List mailingLists )
>>>>>>> + public void setMailingLists( List<MailingList> mailingLists )
>>>>>>> {
>>>>>>> getModel().setMailingLists( mailingLists );
>>>>>>> }
>>>>>>>
>>>>>>> - public List getMailingLists()
>>>>>>> + public List<?> getMailingLists()
>>>>>>> {
>>>>>>> return getModel().getMailingLists();
>>>>>>> }
>>>>>>> @@ -1065,12 +1068,12 @@
>>>>>>> getModel().addMailingList( mailingList );
>>>>>>> }
>>>>>>>
>>>>>>> - public void setDevelopers( List developers )
>>>>>>> + public void setDevelopers( List<Developer> developers )
>>>>>>> {
>>>>>>> getModel().setDevelopers( developers );
>>>>>>> }
>>>>>>>
>>>>>>> - public List getDevelopers()
>>>>>>> + public List<Developer> getDevelopers()
>>>>>>> {
>>>>>>> return getModel().getDevelopers();
>>>>>>> }
>>>>>>> @@ -1080,12 +1083,12 @@
>>>>>>> getModel().addDeveloper( developer );
>>>>>>> }
>>>>>>>
>>>>>>> - public void setContributors( List contributors )
>>>>>>> + public void setContributors( List<Contributor> contributors )
>>>>>>> {
>>>>>>> getModel().setContributors( contributors );
>>>>>>> }
>>>>>>>
>>>>>>> - public List getContributors()
>>>>>>> + public List<Contributor> getContributors()
>>>>>>> {
>>>>>>> return getModel().getContributors();
>>>>>>> }
>>>>>>> @@ -1105,12 +1108,12 @@
>>>>>>> return getModelBuild();
>>>>>>> }
>>>>>>>
>>>>>>> - public List getResources()
>>>>>>> + public List<Resource> getResources()
>>>>>>> {
>>>>>>> return getBuild().getResources();
>>>>>>> }
>>>>>>>
>>>>>>> - public List getTestResources()
>>>>>>> + public List<Resource> getTestResources()
>>>>>>> {
>>>>>>> return getBuild().getTestResources();
>>>>>>> }
>>>>>>> @@ -1135,12 +1138,12 @@
>>>>>>> return getModel().getReporting();
>>>>>>> }
>>>>>>>
>>>>>>> - public void setLicenses( List licenses )
>>>>>>> + public void setLicenses( List<License> licenses )
>>>>>>> {
>>>>>>> getModel().setLicenses( licenses );
>>>>>>> }
>>>>>>>
>>>>>>> - public List getLicenses()
>>>>>>> + public List<License> getLicenses()
>>>>>>> {
>>>>>>> return getModel().getLicenses();
>>>>>>> }
>>>>>>> @@ -1150,7 +1153,7 @@
>>>>>>> getModel().addLicense( license );
>>>>>>> }
>>>>>>>
>>>>>>> - public void setArtifacts( Set artifacts )
>>>>>>> + public void setArtifacts( Set<Artifact> artifacts )
>>>>>>> {
>>>>>>> this.artifacts = artifacts;
>>>>>>>
>>>>>>> @@ -1165,12 +1168,12 @@
>>>>>>> * @return {@link Set} < {@link Artifact} >
>>>>>>> * @see #getDependencyArtifacts() to get only direct dependencies
>>>>>>> */
>>>>>>> - public Set getArtifacts()
>>>>>>> + public Set<Artifact> getArtifacts()
>>>>>>> {
>>>>>>> - return artifacts == null ? Collections.EMPTY_SET :
>>>>>>> artifacts;
>>>>>>> + return artifacts == null ? Collections.<Artifact>
>>>>>>> emptySet() :
>>>>>>> artifacts;
>>>>>>> }
>>>>>>>
>>>>>>> - public Map getArtifactMap()
>>>>>>> + public Map<String, Artifact> getArtifactMap()
>>>>>>> {
>>>>>>> if ( artifactMap == null )
>>>>>>> {
>>>>>>> @@ -1180,19 +1183,19 @@
>>>>>>> return artifactMap;
>>>>>>> }
>>>>>>>
>>>>>>> - public void setPluginArtifacts( Set pluginArtifacts )
>>>>>>> + public void setPluginArtifacts( Set<Artifact> pluginArtifacts )
>>>>>>> {
>>>>>>> this.pluginArtifacts = pluginArtifacts;
>>>>>>>
>>>>>>> this.pluginArtifactMap = null;
>>>>>>> }
>>>>>>>
>>>>>>> - public Set getPluginArtifacts()
>>>>>>> + public Set<Artifact> getPluginArtifacts()
>>>>>>> {
>>>>>>> return pluginArtifacts;
>>>>>>> }
>>>>>>>
>>>>>>> - public Map getPluginArtifactMap()
>>>>>>> + public Map<String, Artifact> getPluginArtifactMap()
>>>>>>> {
>>>>>>> if ( pluginArtifactMap == null )
>>>>>>> {
>>>>>>> @@ -1202,19 +1205,19 @@
>>>>>>> return pluginArtifactMap;
>>>>>>> }
>>>>>>>
>>>>>>> - public void setReportArtifacts( Set reportArtifacts )
>>>>>>> + public void setReportArtifacts( Set<Artifact> reportArtifacts )
>>>>>>> {
>>>>>>> this.reportArtifacts = reportArtifacts;
>>>>>>>
>>>>>>> this.reportArtifactMap = null;
>>>>>>> }
>>>>>>>
>>>>>>> - public Set getReportArtifacts()
>>>>>>> + public Set<Artifact> getReportArtifacts()
>>>>>>> {
>>>>>>> return reportArtifacts;
>>>>>>> }
>>>>>>>
>>>>>>> - public Map getReportArtifactMap()
>>>>>>> + public Map<String, Artifact> getReportArtifactMap()
>>>>>>> {
>>>>>>> if ( reportArtifactMap == null )
>>>>>>> {
>>>>>>> @@ -1224,19 +1227,19 @@
>>>>>>> return reportArtifactMap;
>>>>>>> }
>>>>>>>
>>>>>>> - public void setExtensionArtifacts( Set extensionArtifacts )
>>>>>>> + public void setExtensionArtifacts( Set<Artifact>
>>>>>>> extensionArtifacts
>>>>>>> )
>>>>>>> {
>>>>>>> this.extensionArtifacts = extensionArtifacts;
>>>>>>>
>>>>>>> this.extensionArtifactMap = null;
>>>>>>> }
>>>>>>>
>>>>>>> - public Set getExtensionArtifacts()
>>>>>>> + public Set<Artifact> getExtensionArtifacts()
>>>>>>> {
>>>>>>> return this.extensionArtifacts;
>>>>>>> }
>>>>>>>
>>>>>>> - public Map getExtensionArtifactMap()
>>>>>>> + public Map<String, Artifact> getExtensionArtifactMap()
>>>>>>> {
>>>>>>> if ( extensionArtifactMap == null )
>>>>>>> {
>>>>>>> @@ -1256,7 +1259,7 @@
>>>>>>> return parentArtifact;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getRepositories()
>>>>>>> + public List<Repository> getRepositories()
>>>>>>> {
>>>>>>> return getModel().getRepositories();
>>>>>>> }
>>>>>>> @@ -1265,7 +1268,7 @@
>>>>>>> // Plugins
>>>>>>> //
>>>>>>> ----------------------------------------------------------------------
>>>>>>>
>>>>>>>
>>>>>>> - public List getReportPlugins()
>>>>>>> + public List<ReportPlugin> getReportPlugins()
>>>>>>> {
>>>>>>> if ( getModel().getReporting() == null )
>>>>>>> {
>>>>>>> @@ -1275,7 +1278,7 @@
>>>>>>>
>>>>>>> }
>>>>>>>
>>>>>>> - public List getBuildPlugins()
>>>>>>> + public List<Plugin> getBuildPlugins()
>>>>>>> {
>>>>>>> if ( getModel().getBuild() == null )
>>>>>>> {
>>>>>>> @@ -1284,7 +1287,7 @@
>>>>>>> return getModel().getBuild().getPlugins();
>>>>>>> }
>>>>>>>
>>>>>>> - public List getModules()
>>>>>>> + public List<String> getModules()
>>>>>>> {
>>>>>>> return getModel().getModules();
>>>>>>> }
>>>>>>> @@ -1335,7 +1338,7 @@
>>>>>>>
>>>>>>> if ( pm != null )
>>>>>>> {
>>>>>>> - Map pmByKey = pm.getPluginsAsMap();
>>>>>>> + Map<String, Plugin> pmByKey = pm.getPluginsAsMap();
>>>>>>>
>>>>>>> String pluginKey = plugin.getKey();
>>>>>>>
>>>>>>> @@ -1348,17 +1351,17 @@
>>>>>>> }
>>>>>>> }
>>>>>>>
>>>>>>> - public List getCollectedProjects()
>>>>>>> + public List<MavenProject> getCollectedProjects()
>>>>>>> {
>>>>>>> return collectedProjects;
>>>>>>> }
>>>>>>>
>>>>>>> - public void setCollectedProjects( List collectedProjects )
>>>>>>> + public void setCollectedProjects( List<MavenProject>
>>>>>>> collectedProjects )
>>>>>>> {
>>>>>>> this.collectedProjects = collectedProjects;
>>>>>>> }
>>>>>>>
>>>>>>> - public void setPluginArtifactRepositories( List
>>>>>>> pluginArtifactRepositories )
>>>>>>> + public void setPluginArtifactRepositories(
>>>>>>> List<ArtifactRepository>
>>>>>>> pluginArtifactRepositories )
>>>>>>> {
>>>>>>> this.pluginArtifactRepositories = pluginArtifactRepositories;
>>>>>>> }
>>>>>>> @@ -1367,7 +1370,7 @@
>>>>>>> * @return a list of ArtifactRepository objects constructed
>>>>>>> * from the Repository objects returned by
>>>>>>> getPluginRepositories.
>>>>>>> */
>>>>>>> - public List getPluginArtifactRepositories()
>>>>>>> + public List<ArtifactRepository> getPluginArtifactRepositories()
>>>>>>> {
>>>>>>> return pluginArtifactRepositories;
>>>>>>> }
>>>>>>> @@ -1378,17 +1381,17 @@
>>>>>>> : getReleaseArtifactRepository();
>>>>>>> }
>>>>>>>
>>>>>>> - public List getPluginRepositories()
>>>>>>> + public List<Repository> getPluginRepositories()
>>>>>>> {
>>>>>>> return getModel().getPluginRepositories();
>>>>>>> }
>>>>>>>
>>>>>>> - public void setActiveProfiles( List activeProfiles )
>>>>>>> + public void setActiveProfiles( List<String> activeProfiles )
>>>>>>> {
>>>>>>> this.activeProfiles.addAll( activeProfiles );
>>>>>>> }
>>>>>>>
>>>>>>> - public List getActiveProfiles()
>>>>>>> + public List<String> getActiveProfiles()
>>>>>>> {
>>>>>>> return activeProfiles;
>>>>>>> }
>>>>>>> @@ -1398,11 +1401,11 @@
>>>>>>> getAttachedArtifacts().add( artifact );
>>>>>>> }
>>>>>>>
>>>>>>> - public List getAttachedArtifacts()
>>>>>>> + public List<Artifact> getAttachedArtifacts()
>>>>>>> {
>>>>>>> if ( attachedArtifacts == null )
>>>>>>> {
>>>>>>> - attachedArtifacts = new ArrayList();
>>>>>>> + attachedArtifacts = new ArrayList<Artifact>();
>>>>>>> }
>>>>>>> return attachedArtifacts;
>>>>>>> }
>>>>>>> @@ -1420,7 +1423,7 @@
>>>>>>>
>>>>>>> if ( getBuildPlugins() != null )
>>>>>>> {
>>>>>>> - for ( Iterator iterator = getBuildPlugins().iterator();
>>>>>>> iterator.hasNext(); )
>>>>>>> + for ( Iterator<Plugin> iterator =
>>>>>>> getBuildPlugins().iterator(); iterator.hasNext(); )
>>>>>>> {
>>>>>>> Plugin plugin = (Plugin) iterator.next();
>>>>>>>
>>>>>>> @@ -1498,7 +1501,7 @@
>>>>>>>
>>>>>>> if ( getReportPlugins() != null )
>>>>>>> {
>>>>>>> - for ( Iterator iterator =
>>>>>>> getReportPlugins().iterator();
>>>>>>> iterator.hasNext(); )
>>>>>>> + for ( Iterator<ReportPlugin> iterator =
>>>>>>> getReportPlugins().iterator(); iterator.hasNext(); )
>>>>>>> {
>>>>>>> ReportPlugin plugin = (ReportPlugin) iterator.next();
>>>>>>>
>>>>>>> @@ -1564,12 +1567,12 @@
>>>>>>> * @return {@link Set} < {@link Artifact} >
>>>>>>> * @see #getArtifacts() to get all transitive dependencies
>>>>>>> */
>>>>>>> - public Set getDependencyArtifacts()
>>>>>>> + public Set<Artifact> getDependencyArtifacts()
>>>>>>> {
>>>>>>> return dependencyArtifacts;
>>>>>>> }
>>>>>>>
>>>>>>> - public void setDependencyArtifacts( Set dependencyArtifacts )
>>>>>>> + public void setDependencyArtifacts( Set<Artifact>
>>>>>>> dependencyArtifacts )
>>>>>>> {
>>>>>>> this.dependencyArtifacts = dependencyArtifacts;
>>>>>>> }
>>>>>>> @@ -1594,12 +1597,12 @@
>>>>>>> return originalModel;
>>>>>>> }
>>>>>>>
>>>>>>> - public void setManagedVersionMap( Map map )
>>>>>>> + public void setManagedVersionMap( Map<String,
>>>>>>> ArtifactVersion> map )
>>>>>>> {
>>>>>>> this.managedVersionMap = map;
>>>>>>> }
>>>>>>>
>>>>>>> - public Map getManagedVersionMap()
>>>>>>> + public Map<String, ArtifactVersion> getManagedVersionMap()
>>>>>>> {
>>>>>>> return this.managedVersionMap;
>>>>>>> }
>>>>>>> @@ -1627,12 +1630,12 @@
>>>>>>> return getId().hashCode();
>>>>>>> }
>>>>>>>
>>>>>>> - public List getBuildExtensions()
>>>>>>> + public List<Extension> getBuildExtensions()
>>>>>>> {
>>>>>>> Build build = getBuild();
>>>>>>> if ( build == null || build.getExtensions() == null )
>>>>>>> {
>>>>>>> - return Collections.EMPTY_LIST;
>>>>>>> + return Collections.<Extension> emptyList();
>>>>>>> }
>>>>>>> else
>>>>>>> {
>>>>>>> @@ -1644,7 +1647,7 @@
>>>>>>> * @todo the lazy initialisation of this makes me uneasy.
>>>>>>> * @return {@link Set} < {@link Artifact} >
>>>>>>> */
>>>>>>> - public Set createArtifacts( ArtifactFactory artifactFactory,
>>>>>>> String
>>>>>>> inheritedScope,
>>>>>>> + public Set<Artifact> createArtifacts( ArtifactFactory
>>>>>>> artifactFactory, String inheritedScope,
>>>>>>> ArtifactFilter dependencyFilter )
>>>>>>> throws InvalidDependencyVersionException
>>>>>>> {
>>>>>>> @@ -1674,12 +1677,12 @@
>>>>>>> return getModel().getProperties();
>>>>>>> }
>>>>>>>
>>>>>>> - public List getFilters()
>>>>>>> + public List<String> getFilters()
>>>>>>> {
>>>>>>> return getBuild().getFilters();
>>>>>>> }
>>>>>>>
>>>>>>> - public Map getProjectReferences()
>>>>>>> + public Map<String, MavenProject> getProjectReferences()
>>>>>>> {
>>>>>>> return projectReferences;
>>>>>>> }
>>>>>>> @@ -1705,22 +1708,22 @@
>>>>>>> this.model = model;
>>>>>>> }
>>>>>>>
>>>>>>> - protected void setAttachedArtifacts( List attachedArtifacts )
>>>>>>> + protected void setAttachedArtifacts( List<Artifact>
>>>>>>> attachedArtifacts )
>>>>>>> {
>>>>>>> this.attachedArtifacts = attachedArtifacts;
>>>>>>> }
>>>>>>>
>>>>>>> - protected void setCompileSourceRoots( List compileSourceRoots )
>>>>>>> + protected void setCompileSourceRoots( List<String>
>>>>>>> compileSourceRoots )
>>>>>>> {
>>>>>>> this.compileSourceRoots = compileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - protected void setTestCompileSourceRoots( List
>>>>>>> testCompileSourceRoots )
>>>>>>> + protected void setTestCompileSourceRoots( List<String>
>>>>>>> testCompileSourceRoots )
>>>>>>> {
>>>>>>> this.testCompileSourceRoots = testCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - protected void setScriptSourceRoots( List scriptSourceRoots )
>>>>>>> + protected void setScriptSourceRoots( List<String>
>>>>>>> scriptSourceRoots
>>>>>>> )
>>>>>>> {
>>>>>>> this.scriptSourceRoots = scriptSourceRoots;
>>>>>>> }
>>>>>>> @@ -1737,16 +1740,16 @@
>>>>>>>
>>>>>>> public void resolveActiveArtifacts()
>>>>>>> {
>>>>>>> - Set depArtifacts = getDependencyArtifacts();
>>>>>>> + Set<Artifact> depArtifacts = getDependencyArtifacts();
>>>>>>> if ( depArtifacts == null )
>>>>>>> {
>>>>>>> return;
>>>>>>> }
>>>>>>>
>>>>>>> - Set updated = new LinkedHashSet( depArtifacts.size() );
>>>>>>> + Set<Artifact> updated = new LinkedHashSet<Artifact>(
>>>>>>> depArtifacts.size() );
>>>>>>> int updatedCount = 0;
>>>>>>>
>>>>>>> - for ( Iterator it = depArtifacts.iterator();
>>>>>>> it.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> it = depArtifacts.iterator();
>>>>>>> it.hasNext(); )
>>>>>>> {
>>>>>>> Artifact depArtifact = (Artifact) it.next();
>>>>>>> Artifact replaced = replaceWithActiveArtifact(
>>>>>>> depArtifact );
>>>>>>> @@ -1815,13 +1818,13 @@
>>>>>>> * @param requestedArtifact The artifact to resolve, must not be
>>>>>>> <code>null</code>.
>>>>>>> * @return The matching artifact or <code>null</code> if not found.
>>>>>>> */
>>>>>>> - private Artifact findMatchingArtifact( List artifacts, Artifact
>>>>>>> requestedArtifact )
>>>>>>> + private Artifact findMatchingArtifact( List<Artifact>
>>>>>>> artifacts,
>>>>>>> Artifact requestedArtifact )
>>>>>>> {
>>>>>>> if ( artifacts != null && !artifacts.isEmpty() )
>>>>>>> {
>>>>>>> // first try matching by dependency conflict id
>>>>>>> String requestedId =
>>>>>>> requestedArtifact.getDependencyConflictId();
>>>>>>> - for ( Iterator it = artifacts.iterator();
>>>>>>> it.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>>>>> it.hasNext(); )
>>>>>>> {
>>>>>>> Artifact artifact = (Artifact) it.next();
>>>>>>> if ( requestedId.equals(
>>>>>>> artifact.getDependencyConflictId() ) )
>>>>>>> @@ -1832,7 +1835,7 @@
>>>>>>>
>>>>>>> // next try matching by repository conflict id
>>>>>>> requestedId = getRepositoryConflictId( requestedArtifact );
>>>>>>> - for ( Iterator it = artifacts.iterator();
>>>>>>> it.hasNext(); )
>>>>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>>>>> it.hasNext(); )
>>>>>>> {
>>>>>>> Artifact artifact = (Artifact) it.next();
>>>>>>> if ( requestedId.equals( getRepositoryConflictId(
>>>>>>> artifact
>>>>>>> ) ) )
>>>>>>> @@ -1898,7 +1901,7 @@
>>>>>>> }
>>>>>>> }
>>>>>>>
>>>>>>> - private void addArtifactPath(Artifact a, List list) throws
>>>>>>> DependencyResolutionRequiredException
>>>>>>> + private void addArtifactPath(Artifact a, List<String> list)
>>>>>>> throws
>>>>>>> DependencyResolutionRequiredException
>>>>>>> {
>>>>>>> File file = a.getFile();
>>>>>>> if ( file == null )
>>>>>>> @@ -1954,21 +1957,21 @@
>>>>>>>
>>>>>>> private Build originalInterpolatedBuild;
>>>>>>>
>>>>>>> - private List dynamicCompileSourceRoots;
>>>>>>> + private List<String> dynamicCompileSourceRoots;
>>>>>>>
>>>>>>> - private List originalInterpolatedCompileSourceRoots;
>>>>>>> + private List<String> originalInterpolatedCompileSourceRoots;
>>>>>>>
>>>>>>> - private List dynamicTestCompileSourceRoots;
>>>>>>> + private List<String> dynamicTestCompileSourceRoots;
>>>>>>>
>>>>>>> - private List originalInterpolatedTestCompileSourceRoots;
>>>>>>> + private List<String>
>>>>>>> originalInterpolatedTestCompileSourceRoots;
>>>>>>>
>>>>>>> - private List dynamicScriptSourceRoots;
>>>>>>> + private List<String> dynamicScriptSourceRoots;
>>>>>>>
>>>>>>> - private List originalInterpolatedScriptSourceRoots;
>>>>>>> + private List<String> originalInterpolatedScriptSourceRoots;
>>>>>>>
>>>>>>> private boolean isConcrete = false;
>>>>>>>
>>>>>>> - public boolean isConcrete()
>>>>>>> + public boolean isConcrete()
>>>>>>> {
>>>>>>> return isConcrete;
>>>>>>> }
>>>>>>> @@ -1988,32 +1991,32 @@
>>>>>>> return originalInterpolatedBuild;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getDynamicCompileSourceRoots()
>>>>>>> + public List<String> getDynamicCompileSourceRoots()
>>>>>>> {
>>>>>>> return dynamicCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getOriginalInterpolatedCompileSourceRoots()
>>>>>>> + public List<String> getOriginalInterpolatedCompileSourceRoots()
>>>>>>> {
>>>>>>> return originalInterpolatedCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getDynamicTestCompileSourceRoots()
>>>>>>> + public List<String> getDynamicTestCompileSourceRoots()
>>>>>>> {
>>>>>>> return dynamicTestCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getOriginalInterpolatedTestCompileSourceRoots()
>>>>>>> + public List<String>
>>>>>>> getOriginalInterpolatedTestCompileSourceRoots()
>>>>>>> {
>>>>>>> return originalInterpolatedTestCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getDynamicScriptSourceRoots()
>>>>>>> + public List<String> getDynamicScriptSourceRoots()
>>>>>>> {
>>>>>>> return dynamicScriptSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - public List getOriginalInterpolatedScriptSourceRoots()
>>>>>>> + public List<String> getOriginalInterpolatedScriptSourceRoots()
>>>>>>> {
>>>>>>> return originalInterpolatedScriptSourceRoots;
>>>>>>> }
>>>>>>> @@ -2034,19 +2037,19 @@
>>>>>>> originalInterpolatedBuild = null;
>>>>>>> }
>>>>>>>
>>>>>>> - public void preserveCompileSourceRoots( List
>>>>>>> originalInterpolatedCompileSourceRoots )
>>>>>>> + public void preserveCompileSourceRoots( List<String>
>>>>>>> originalInterpolatedCompileSourceRoots )
>>>>>>> {
>>>>>>> dynamicCompileSourceRoots = getCompileSourceRoots();
>>>>>>> this.originalInterpolatedCompileSourceRoots =
>>>>>>> originalInterpolatedCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - public void preserveTestCompileSourceRoots( List
>>>>>>> originalInterpolatedTestCompileSourceRoots )
>>>>>>> + public void preserveTestCompileSourceRoots( List<String>
>>>>>>> originalInterpolatedTestCompileSourceRoots )
>>>>>>> {
>>>>>>> dynamicTestCompileSourceRoots = getTestCompileSourceRoots();
>>>>>>> this.originalInterpolatedTestCompileSourceRoots =
>>>>>>> originalInterpolatedTestCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - public void preserveScriptSourceRoots( List
>>>>>>> originalInterpolatedScriptSourceRoots )
>>>>>>> + public void preserveScriptSourceRoots( List<String>
>>>>>>> originalInterpolatedScriptSourceRoots )
>>>>>>> {
>>>>>>> dynamicScriptSourceRoots = getScriptSourceRoots();
>>>>>>> this.originalInterpolatedScriptSourceRoots =
>>>>>>> originalInterpolatedScriptSourceRoots;
>>>>>>> @@ -2071,32 +2074,32 @@
>>>>>>> this.originalInterpolatedBuild = originalInterpolatedBuild;
>>>>>>> }
>>>>>>>
>>>>>>> - protected void setDynamicCompileSourceRoots( List
>>>>>>> dynamicCompileSourceRoots )
>>>>>>> + protected void setDynamicCompileSourceRoots( List<String>
>>>>>>> dynamicCompileSourceRoots )
>>>>>>> {
>>>>>>> this.dynamicCompileSourceRoots = dynamicCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - protected void setOriginalInterpolatedCompileSourceRoots( List
>>>>>>> originalInterpolatedCompileSourceRoots )
>>>>>>> + protected void setOriginalInterpolatedCompileSourceRoots(
>>>>>>> List<String> originalInterpolatedCompileSourceRoots )
>>>>>>> {
>>>>>>> this.originalInterpolatedCompileSourceRoots =
>>>>>>> originalInterpolatedCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - protected void setDynamicTestCompileSourceRoots( List
>>>>>>> dynamicTestCompileSourceRoots )
>>>>>>> + protected void setDynamicTestCompileSourceRoots( List<String>
>>>>>>> dynamicTestCompileSourceRoots )
>>>>>>> {
>>>>>>> this.dynamicTestCompileSourceRoots =
>>>>>>> dynamicTestCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - protected void
>>>>>>> setOriginalInterpolatedTestCompileSourceRoots( List
>>>>>>> originalInterpolatedTestCompileSourceRoots )
>>>>>>> + protected void setOriginalInterpolatedTestCompileSourceRoots(
>>>>>>> List<String> originalInterpolatedTestCompileSourceRoots )
>>>>>>> {
>>>>>>> this.originalInterpolatedTestCompileSourceRoots =
>>>>>>> originalInterpolatedTestCompileSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - protected void setDynamicScriptSourceRoots( List
>>>>>>> dynamicScriptSourceRoots )
>>>>>>> + protected void setDynamicScriptSourceRoots( List<String>
>>>>>>> dynamicScriptSourceRoots )
>>>>>>> {
>>>>>>> this.dynamicScriptSourceRoots = dynamicScriptSourceRoots;
>>>>>>> }
>>>>>>>
>>>>>>> - protected void setOriginalInterpolatedScriptSourceRoots( List
>>>>>>> originalInterpolatedScriptSourceRoots )
>>>>>>> + protected void setOriginalInterpolatedScriptSourceRoots(
>>>>>>> List<String> originalInterpolatedScriptSourceRoots )
>>>>>>> {
>>>>>>> this.originalInterpolatedScriptSourceRoots =
>>>>>>> originalInterpolatedScriptSourceRoots;
>>>>>>> }
>>>>>>> @@ -2114,7 +2117,7 @@
>>>>>>> if ( p != null )
>>>>>>> {
>>>>>>> preservedProperties = new Properties();
>>>>>>> - for( Enumeration e = p.propertyNames();
>>>>>>> e.hasMoreElements();
>>>>>>> )
>>>>>>> + for( Enumeration<?> e = p.propertyNames();
>>>>>>> e.hasMoreElements(); )
>>>>>>> {
>>>>>>> String key = (String) e.nextElement();
>>>>>>> preservedProperties.setProperty( key, p.getProperty(
>>>>>>> key )
>>>>>>> );
>>>>>>>
>>>>>>> Modified:
>>>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
>>>>>>>
>>>>>>> URL:
>>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>>>
>>>>>>> ==============================================================================
>>>>>>>
>>>>>>>
>>>>>>> ---
>>>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
>>>>>>>
>>>>>>> (original)
>>>>>>> +++
>>>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
>>>>>>> Fri
>>>>>>> May 1 08:04:48 2009
>>>>>>> @@ -47,6 +47,7 @@
>>>>>>> <models>
>>>>>>> <model>src/main/mdo/metadata.mdo</model>
>>>>>>> </models>
>>>>>>> + <useJava5>true</useJava5>
>>>>>>> </configuration>
>>>>>>> </plugin>
>>>>>>> </plugins>
>>>>>>>
>>>>>>> Modified:
>>>>>>> maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>>>>> URL:
>>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-settings/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>>>
>>>>>>> ==============================================================================
>>>>>>>
>>>>>>>
>>>>>>> --- maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>>>>> (original)
>>>>>>> +++ maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>>>>> Fri May
>>>>>>> 1 08:04:48 2009
>>>>>>> @@ -58,6 +58,7 @@
>>>>>>> <models>
>>>>>>> <model>src/main/mdo/settings.mdo</model>
>>>>>>> </models>
>>>>>>> + <useJava5>true</useJava5>
>>>>>>> </configuration>
>>>>>>> </plugin>
>>>>>>> </plugins>
>>>>>>>
>>>>>>> Modified:
>>>>>>> maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>>>>> URL:
>>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>>>
>>>>>>> ==============================================================================
>>>>>>>
>>>>>>>
>>>>>>> --- maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>>>>> (original)
>>>>>>> +++ maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>>>>> Fri May
>>>>>>> 1 08:04:48 2009
>>>>>>> @@ -66,6 +66,7 @@
>>>>>>> <models>
>>>>>>> <model>src/main/mdo/toolchains.mdo</model>
>>>>>>> </models>
>>>>>>> + <useJava5>true</useJava5>
>>>>>>> </configuration>
>>>>>>> </plugin>
>>>>>>> <plugin>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>> Thanks,
>>>>>>
>>>>>> Jason
>>>>>>
>>>>>> ----------------------------------------------------------
>>>>>> Jason van Zyl
>>>>>> Founder, Apache Maven
>>>>>> http://twitter.com/jvanzyl
>>>>>> http://twitter.com/SonatypeNexus
>>>>>> http://twitter.com/SonatypeM2E
>>>>>> ----------------------------------------------------------
>>>>>>
>>>>>> We all have problems. How we deal with them is a measure of our
>>>>>> worth.
>>>>>>
>>>>>> -- Unknown
>>>>>>
>>>>>>
>>>>>> ---------------------------------------------------------------------
>>>>>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>>>>>> For additional commands, e-mail: dev-help@maven.apache.org
>>>>>>
>>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>>>>> For additional commands, e-mail: dev-help@maven.apache.org
>>>>>
>>>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>>> For additional commands, e-mail: dev-help@maven.apache.org
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>> For additional commands, e-mail: dev-help@maven.apache.org
>>
>
> Thanks,
>
> Jason
>
> ----------------------------------------------------------
> Jason van Zyl
> Founder, Apache Maven
> http://twitter.com/jvanzyl
> http://twitter.com/SonatypeNexus
> http://twitter.com/SonatypeM2E
> ----------------------------------------------------------
>
> To do two things at once is to do neither.
>
> -—Publilius Syrus, Roman slave, first century B.C.
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: svn commit: r770570 - in /maven/components/branches/maven-2.2.x: maven-artifact/src/main/java/org/apache/maven/artifact/ maven-model/ maven-profile/ maven-project/src/main/java/org/apache/maven/project/ maven-repository-metadata/ maven-settings/
Posted by Jason van Zyl <jv...@sonatype.com>.
On 2-May-09, at 1:58 PM, John Casey wrote:
> The ITs all looked fine with this commit, so it looks like this
> stuff won't cause too much grief...but hopefully the testing of the
> RC will tell.
>
If it was decided that this should run in only in 1.5 and you don't
have to worry about a 1.4 runtime then it should be fine. I lost track
of whether it was decided a 1.4 runtime should continue to work.
> I've created a 2.2.0-RC branch for subsequent RCs and the final
> release of 2.2.0, so we can continue converting the syntax on the
> 2.2.x branch.
>
> -john
>
> BRIAN FOX wrote:
>> In general, no. I just wanted to draw John's attention to the
>> changes since I know he was preparing to cut an RC. I am suspicious
>> of all changes when a release is drawing near.
>> On May 2, 2009, at 2:44 AM, nicolas de loof wrote:
>>> Do we prefer plugin developers to use List<?> and get
>>> ClassCastExceptions at
>>> runtime or sue type-safe collections to help them create stronger
>>> code ?Anyway,
>>> few plugins allready use Java5. Java 1.4 based one will not be
>>> broken as the
>>> generics signature is only a compile-time check.
>>>
>>>
>>> 2009/5/1 Brian Fox <br...@infinity.nu>
>>>
>>>> I'm not sure if this is in scope of what John is trying to do wrt
>>>> to 2.2.
>>>>
>>>>
>>>> Jason van Zyl wrote:
>>>>
>>>>> I don't believe anyone actually agreed to this yet. Are you sure
>>>>> this is
>>>>> not going to cause problems for users?
>>>>>
>>>>> On 1-May-09, at 1:04 AM, nicolas@apache.org wrote:
>>>>>
>>>>> Author: nicolas
>>>>>> Date: Fri May 1 08:04:48 2009
>>>>>> New Revision: 770570
>>>>>>
>>>>>> URL: http://svn.apache.org/viewvc?rev=770570&view=rev
>>>>>> Log:
>>>>>> use java5 syntax.
>>>>>> -> set modello to generate java5 syntax
>>>>>> -> MavenProject updated to use generics on collections &
>>>>>> methods for
>>>>>> better type safety
>>>>>>
>>>>>> Modified:
>>>>>>
>>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/
>>>>>> java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>>
>>>>>> maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>>>> maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>>>>
>>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/
>>>>>> java/org/apache/maven/project/MavenProject.java
>>>>>>
>>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/
>>>>>> pom.xml
>>>>>> maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>>>> maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>>>>
>>>>>> Modified:
>>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/
>>>>>> java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>>
>>>>>> URL:
>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java?rev=770570&r1=770569&r2=770570&view=diff
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =================================================================
>>>>>>
>>>>>> ---
>>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/
>>>>>> java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>> (original)
>>>>>> +++
>>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/
>>>>>> java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>> Fri May 1 08:04:48 2009
>>>>>> @@ -86,13 +86,13 @@
>>>>>> baseVersion;
>>>>>> }
>>>>>>
>>>>>> - public static Map artifactMapByVersionlessId( Collection
>>>>>> artifacts )
>>>>>> + public static Map<String, Artifact>
>>>>>> artifactMapByVersionlessId(
>>>>>> Collection<Artifact> artifacts )
>>>>>> {
>>>>>> - Map artifactMap = new LinkedHashMap();
>>>>>> + Map<String, Artifact> artifactMap = new
>>>>>> LinkedHashMap<String,
>>>>>> Artifact>();
>>>>>>
>>>>>> if ( artifacts != null )
>>>>>> {
>>>>>> - for ( Iterator it = artifacts.iterator();
>>>>>> it.hasNext(); )
>>>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>>>> it.hasNext(); )
>>>>>> {
>>>>>> Artifact artifact = (Artifact) it.next();
>>>>>>
>>>>>>
>>>>>> Modified: maven/components/branches/maven-2.2.x/maven-model/
>>>>>> pom.xml
>>>>>> URL:
>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-model/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =================================================================
>>>>>>
>>>>>> --- maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>>>> (original)
>>>>>> +++ maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>>>> Fri May 1
>>>>>> 08:04:48 2009
>>>>>> @@ -47,6 +47,7 @@
>>>>>> <models>
>>>>>> <model>src/main/mdo/maven.mdo</model>
>>>>>> </models>
>>>>>> + <useJava5>true</useJava5>
>>>>>> </configuration>
>>>>>> </plugin>
>>>>>> <plugin>
>>>>>>
>>>>>> Modified: maven/components/branches/maven-2.2.x/maven-profile/
>>>>>> pom.xml
>>>>>> URL:
>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-profile/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =================================================================
>>>>>>
>>>>>> --- maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>>>> (original)
>>>>>> +++ maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>>>> Fri May
>>>>>> 1 08:04:48 2009
>>>>>> @@ -54,6 +54,7 @@
>>>>>> <models>
>>>>>> <model>src/main/mdo/profiles.mdo</model>
>>>>>> </models>
>>>>>> + <useJava5>true</useJava5>
>>>>>> </configuration>
>>>>>> </plugin>
>>>>>> </plugins>
>>>>>>
>>>>>> Modified:
>>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/
>>>>>> java/org/apache/maven/project/MavenProject.java
>>>>>>
>>>>>> URL:
>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java?rev=770570&r1=770569&r2=770570&view=diff
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =================================================================
>>>>>>
>>>>>> ---
>>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/
>>>>>> java/org/apache/maven/project/MavenProject.java
>>>>>> (original)
>>>>>> +++
>>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/
>>>>>> java/org/apache/maven/project/MavenProject.java
>>>>>> Fri May 1 08:04:48 2009
>>>>>> @@ -36,10 +36,11 @@
>>>>>> import org.apache.maven.artifact.Artifact;
>>>>>> import org.apache.maven.artifact.ArtifactUtils;
>>>>>> import
>>>>>> org.apache.maven.artifact.DependencyResolutionRequiredException;
>>>>>> -import org.apache.maven.artifact.versioning.ManagedVersionMap;
>>>>>> import org.apache.maven.artifact.factory.ArtifactFactory;
>>>>>> import org.apache.maven.artifact.repository.ArtifactRepository;
>>>>>> import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
>>>>>> +import org.apache.maven.artifact.versioning.ArtifactVersion;
>>>>>> +import org.apache.maven.artifact.versioning.ManagedVersionMap;
>>>>>> import org.apache.maven.model.Build;
>>>>>> import org.apache.maven.model.CiManagement;
>>>>>> import org.apache.maven.model.Contributor;
>>>>>> @@ -47,6 +48,7 @@
>>>>>> import org.apache.maven.model.DependencyManagement;
>>>>>> import org.apache.maven.model.Developer;
>>>>>> import org.apache.maven.model.DistributionManagement;
>>>>>> +import org.apache.maven.model.Extension;
>>>>>> import org.apache.maven.model.IssueManagement;
>>>>>> import org.apache.maven.model.License;
>>>>>> import org.apache.maven.model.MailingList;
>>>>>> @@ -59,6 +61,7 @@
>>>>>> import org.apache.maven.model.ReportPlugin;
>>>>>> import org.apache.maven.model.ReportSet;
>>>>>> import org.apache.maven.model.Reporting;
>>>>>> +import org.apache.maven.model.Repository;
>>>>>> import org.apache.maven.model.Resource;
>>>>>> import org.apache.maven.model.Scm;
>>>>>> import org.apache.maven.model.io.xpp3.MavenXpp3Writer;
>>>>>> @@ -96,60 +99,60 @@
>>>>>>
>>>>>> private File file;
>>>>>>
>>>>>> - private Set artifacts;
>>>>>> + private Set<Artifact> artifacts;
>>>>>>
>>>>>> private Artifact parentArtifact;
>>>>>>
>>>>>> - private Set pluginArtifacts;
>>>>>> + private Set<Artifact> pluginArtifacts;
>>>>>>
>>>>>> - private List remoteArtifactRepositories;
>>>>>> + private List<ArtifactRepository> remoteArtifactRepositories;
>>>>>>
>>>>>> - private List collectedProjects = Collections.EMPTY_LIST;
>>>>>> + private List<MavenProject> collectedProjects =
>>>>>> Collections.<MavenProject> emptyList();
>>>>>>
>>>>>> - private List attachedArtifacts;
>>>>>> + private List<Artifact> attachedArtifacts;
>>>>>>
>>>>>> private MavenProject executionProject;
>>>>>>
>>>>>> - private List compileSourceRoots = new ArrayList();
>>>>>> + private List<String> compileSourceRoots = new
>>>>>> ArrayList<String>();
>>>>>>
>>>>>> - private List testCompileSourceRoots = new ArrayList();
>>>>>> + private List<String> testCompileSourceRoots = new
>>>>>> ArrayList<String>();
>>>>>>
>>>>>> - private List scriptSourceRoots = new ArrayList();
>>>>>> + private List<String> scriptSourceRoots = new
>>>>>> ArrayList<String>();
>>>>>>
>>>>>> - private List pluginArtifactRepositories;
>>>>>> + private List<ArtifactRepository> pluginArtifactRepositories;
>>>>>>
>>>>>> private ArtifactRepository releaseArtifactRepository;
>>>>>>
>>>>>> private ArtifactRepository snapshotArtifactRepository;
>>>>>>
>>>>>> - private List activeProfiles = new ArrayList();
>>>>>> + private List<String> activeProfiles = new
>>>>>> ArrayList<String>();
>>>>>>
>>>>>> - private Set dependencyArtifacts;
>>>>>> + private Set<Artifact> dependencyArtifacts;
>>>>>>
>>>>>> private Artifact artifact;
>>>>>>
>>>>>> // calculated.
>>>>>> - private Map artifactMap;
>>>>>> + private Map<String, Artifact> artifactMap;
>>>>>>
>>>>>> private Model originalModel;
>>>>>>
>>>>>> - private Map pluginArtifactMap;
>>>>>> + private Map<String, Artifact> pluginArtifactMap;
>>>>>>
>>>>>> - private Set reportArtifacts;
>>>>>> + private Set<Artifact> reportArtifacts;
>>>>>>
>>>>>> - private Map reportArtifactMap;
>>>>>> + private Map<String, Artifact> reportArtifactMap;
>>>>>>
>>>>>> - private Set extensionArtifacts;
>>>>>> + private Set<Artifact> extensionArtifacts;
>>>>>>
>>>>>> - private Map extensionArtifactMap;
>>>>>> + private Map<String, Artifact> extensionArtifactMap;
>>>>>>
>>>>>> - private Map managedVersionMap;
>>>>>> + private Map<String, ArtifactVersion> managedVersionMap;
>>>>>>
>>>>>> - private Map projectReferences = new HashMap();
>>>>>> + private Map<String, MavenProject> projectReferences = new
>>>>>> HashMap<String, MavenProject>();
>>>>>>
>>>>>> private boolean executionRoot;
>>>>>>
>>>>>> - private Map moduleAdjustments;
>>>>>> + private Map<String, String> moduleAdjustments;
>>>>>>
>>>>>> private File basedir;
>>>>>>
>>>>>> @@ -245,23 +248,23 @@
>>>>>> if ( project.getAttachedArtifacts() != null )
>>>>>> {
>>>>>> // clone properties modifyable by plugins in a forked
>>>>>> lifecycle
>>>>>> - setAttachedArtifacts( new ArrayList(
>>>>>> project.getAttachedArtifacts() ) );
>>>>>> + setAttachedArtifacts( new ArrayList<Artifact>(
>>>>>> project.getAttachedArtifacts() ) );
>>>>>> }
>>>>>>
>>>>>> if ( project.getCompileSourceRoots() != null )
>>>>>> {
>>>>>> // clone source roots
>>>>>> - setCompileSourceRoots( ( new ArrayList(
>>>>>> project.getCompileSourceRoots() ) ) );
>>>>>> + setCompileSourceRoots( ( new ArrayList<String>(
>>>>>> project.getCompileSourceRoots() ) ) );
>>>>>> }
>>>>>>
>>>>>> if ( project.getTestCompileSourceRoots() != null )
>>>>>> {
>>>>>> - setTestCompileSourceRoots( ( new ArrayList(
>>>>>> project.getTestCompileSourceRoots() ) ) );
>>>>>> + setTestCompileSourceRoots( ( new ArrayList<String>(
>>>>>> project.getTestCompileSourceRoots() ) ) );
>>>>>> }
>>>>>>
>>>>>> if ( project.getScriptSourceRoots() != null )
>>>>>> {
>>>>>> - setScriptSourceRoots( ( new ArrayList(
>>>>>> project.getScriptSourceRoots() ) ) );
>>>>>> + setScriptSourceRoots( ( new ArrayList<String>(
>>>>>> project.getScriptSourceRoots() ) ) );
>>>>>> }
>>>>>>
>>>>>> setModel( ( ModelUtils.cloneModel( project.getModel() ) ) );
>>>>>> @@ -298,25 +301,25 @@
>>>>>> setDynamicBuild( ModelUtils.cloneBuild(
>>>>>> project.getDynamicBuild() ) );
>>>>>> setOriginalInterpolatedBuild( ModelUtils.cloneBuild(
>>>>>> project.getOriginalInterpolatedBuild() ) );
>>>>>>
>>>>>> - List dynamicRoots =
>>>>>> project.getDynamicCompileSourceRoots();
>>>>>> + List<String> dynamicRoots =
>>>>>> project.getDynamicCompileSourceRoots();
>>>>>> if ( dynamicRoots != null )
>>>>>> {
>>>>>> - setDynamicCompileSourceRoots( new ArrayList(
>>>>>> dynamicRoots ) );
>>>>>> - setOriginalInterpolatedCompileSourceRoots( new
>>>>>> ArrayList
>>>>>> ( project.getOriginalInterpolatedCompileSourceRoots() ) );
>>>>>> + setDynamicCompileSourceRoots( new
>>>>>> ArrayList<String>(
>>>>>> dynamicRoots ) );
>>>>>> + setOriginalInterpolatedCompileSourceRoots( new
>>>>>> ArrayList
>>>>>> <
>>>>>> String>( project.getOriginalInterpolatedCompileSourceRoots() ) );
>>>>>> }
>>>>>>
>>>>>> dynamicRoots =
>>>>>> project.getDynamicTestCompileSourceRoots();
>>>>>> if ( dynamicRoots != null )
>>>>>> {
>>>>>> - setDynamicTestCompileSourceRoots( new ArrayList(
>>>>>> dynamicRoots ) );
>>>>>> -
>>>>>> setOriginalInterpolatedTestCompileSourceRoots( new
>>>>>> ArrayList
>>>>>> ( project.getOriginalInterpolatedTestCompileSourceRoots() ) );
>>>>>> + setDynamicTestCompileSourceRoots( new
>>>>>> ArrayList<String>(
>>>>>> dynamicRoots ) );
>>>>>> +
>>>>>> setOriginalInterpolatedTestCompileSourceRoots( new
>>>>>> ArrayList
>>>>>> <
>>>>>> String
>>>>>> >( project.getOriginalInterpolatedTestCompileSourceRoots() )
>>>>>> );
>>>>>> }
>>>>>>
>>>>>> dynamicRoots = project.getDynamicScriptSourceRoots();
>>>>>> if ( dynamicRoots != null )
>>>>>> {
>>>>>> - setDynamicScriptSourceRoots( new
>>>>>> ArrayList( dynamicRoots
>>>>>> ) );
>>>>>> - setOriginalInterpolatedScriptSourceRoots( new
>>>>>> ArrayList(
>>>>>> project.getOriginalInterpolatedScriptSourceRoots() ) );
>>>>>> + setDynamicScriptSourceRoots( new
>>>>>> ArrayList<String>(
>>>>>> dynamicRoots ) );
>>>>>> + setOriginalInterpolatedScriptSourceRoots( new
>>>>>> ArrayList
>>>>>> <String>( project.getOriginalInterpolatedScriptSourceRoots() ) );
>>>>>> }
>>>>>> }
>>>>>>
>>>>>> @@ -347,12 +350,12 @@
>>>>>>
>>>>>> if ( moduleAdjustments == null )
>>>>>> {
>>>>>> - moduleAdjustments = new HashMap();
>>>>>> + moduleAdjustments = new HashMap<String, String>();
>>>>>>
>>>>>> - List modules = getModules();
>>>>>> + List<String> modules = getModules();
>>>>>> if ( modules != null )
>>>>>> {
>>>>>> - for ( Iterator it = modules.iterator();
>>>>>> it.hasNext(); )
>>>>>> + for ( Iterator<String> it = modules.iterator();
>>>>>> it.hasNext(); )
>>>>>> {
>>>>>> String modulePath = (String) it.next();
>>>>>> String moduleName = modulePath;
>>>>>> @@ -415,12 +418,12 @@
>>>>>> this.parent = parent;
>>>>>> }
>>>>>>
>>>>>> - public void setRemoteArtifactRepositories( List
>>>>>> remoteArtifactRepositories )
>>>>>> + public void
>>>>>> setRemoteArtifactRepositories( List<ArtifactRepository>
>>>>>> remoteArtifactRepositories )
>>>>>> {
>>>>>> this.remoteArtifactRepositories =
>>>>>> remoteArtifactRepositories;
>>>>>> }
>>>>>>
>>>>>> - public List getRemoteArtifactRepositories()
>>>>>> + public List<ArtifactRepository>
>>>>>> getRemoteArtifactRepositories()
>>>>>> {
>>>>>> return remoteArtifactRepositories;
>>>>>> }
>>>>>> @@ -460,12 +463,12 @@
>>>>>> return basedir;
>>>>>> }
>>>>>>
>>>>>> - public void setDependencies( List dependencies )
>>>>>> + public void setDependencies( List<Dependency> dependencies )
>>>>>> {
>>>>>> getModel().setDependencies( dependencies );
>>>>>> }
>>>>>>
>>>>>> - public List getDependencies()
>>>>>> + public List<Dependency> getDependencies()
>>>>>> {
>>>>>> return getModel().getDependencies();
>>>>>> }
>>>>>> @@ -524,29 +527,29 @@
>>>>>> }
>>>>>> }
>>>>>>
>>>>>> - public List getCompileSourceRoots()
>>>>>> + public List<String> getCompileSourceRoots()
>>>>>> {
>>>>>> return compileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - public List getScriptSourceRoots()
>>>>>> + public List<String> getScriptSourceRoots()
>>>>>> {
>>>>>> return scriptSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - public List getTestCompileSourceRoots()
>>>>>> + public List<String> getTestCompileSourceRoots()
>>>>>> {
>>>>>> return testCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - public List getCompileClasspathElements()
>>>>>> + public List<String> getCompileClasspathElements()
>>>>>> throws DependencyResolutionRequiredException
>>>>>> {
>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>> + List<String> list = new
>>>>>> ArrayList<String>( getArtifacts().size()
>>>>>> );
>>>>>>
>>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>>
>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> + for ( Iterator<?> i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -563,11 +566,11 @@
>>>>>> return list;
>>>>>> }
>>>>>>
>>>>>> - public List getCompileArtifacts()
>>>>>> + public List<Artifact> getCompileArtifacts()
>>>>>> {
>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>>> getArtifacts().size() );
>>>>>>
>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -585,18 +588,18 @@
>>>>>> return list;
>>>>>> }
>>>>>>
>>>>>> - public List getCompileDependencies()
>>>>>> + public List<Dependency> getCompileDependencies()
>>>>>> {
>>>>>> - Set artifacts = getArtifacts();
>>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>>
>>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>>> {
>>>>>> - return Collections.EMPTY_LIST;
>>>>>> + return Collections.<Dependency> emptyList();
>>>>>> }
>>>>>>
>>>>>> - List list = new ArrayList( artifacts.size() );
>>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>>> artifacts.size() );
>>>>>>
>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -619,16 +622,16 @@
>>>>>> return list;
>>>>>> }
>>>>>>
>>>>>> - public List getTestClasspathElements()
>>>>>> + public List<String> getTestClasspathElements()
>>>>>> throws DependencyResolutionRequiredException
>>>>>> {
>>>>>> - List list = new ArrayList( getArtifacts().size() + 1 );
>>>>>> + List<String> list = new
>>>>>> ArrayList<String>( getArtifacts().size()
>>>>>> + 1 );
>>>>>>
>>>>>> list.add( getBuild().getTestOutputDirectory() );
>>>>>>
>>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>>
>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -652,11 +655,11 @@
>>>>>> return list;
>>>>>> }
>>>>>>
>>>>>> - public List getTestArtifacts()
>>>>>> + public List<Artifact> getTestArtifacts()
>>>>>> {
>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>>> getArtifacts().size() );
>>>>>>
>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -678,18 +681,18 @@
>>>>>> return list;
>>>>>> }
>>>>>>
>>>>>> - public List getTestDependencies()
>>>>>> + public List<Dependency> getTestDependencies()
>>>>>> {
>>>>>> - Set artifacts = getArtifacts();
>>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>>
>>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>>> {
>>>>>> - return Collections.EMPTY_LIST;
>>>>>> + return Collections.emptyList();
>>>>>> }
>>>>>>
>>>>>> - List list = new ArrayList( artifacts.size() );
>>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>>> artifacts.size() );
>>>>>>
>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -715,14 +718,14 @@
>>>>>> return list;
>>>>>> }
>>>>>>
>>>>>> - public List getRuntimeClasspathElements()
>>>>>> + public List<String> getRuntimeClasspathElements()
>>>>>> throws DependencyResolutionRequiredException
>>>>>> {
>>>>>> - List list = new ArrayList( getArtifacts().size() + 1 );
>>>>>> + List<String> list = new
>>>>>> ArrayList<String>( getArtifacts().size()
>>>>>> + 1 );
>>>>>>
>>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>>
>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -743,11 +746,11 @@
>>>>>> return list;
>>>>>> }
>>>>>>
>>>>>> - public List getRuntimeArtifacts()
>>>>>> + public List<Artifact> getRuntimeArtifacts()
>>>>>> {
>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>>> getArtifacts().size() );
>>>>>>
>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -764,18 +767,18 @@
>>>>>> return list;
>>>>>> }
>>>>>>
>>>>>> - public List getRuntimeDependencies()
>>>>>> + public List<Dependency> getRuntimeDependencies()
>>>>>> {
>>>>>> - Set artifacts = getArtifacts();
>>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>>
>>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>>> {
>>>>>> - return Collections.EMPTY_LIST;
>>>>>> + return Collections.<Dependency> emptyList();
>>>>>> }
>>>>>>
>>>>>> - List list = new ArrayList( artifacts.size() );
>>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>>> artifacts.size() );
>>>>>>
>>>>>> - for ( Iterator i = artifacts.iterator(); i.hasNext(); )
>>>>>> + for ( Iterator<Artifact> i = artifacts.iterator();
>>>>>> i.hasNext();
>>>>>> )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -797,14 +800,14 @@
>>>>>> return list;
>>>>>> }
>>>>>>
>>>>>> - public List getSystemClasspathElements()
>>>>>> + public List<String> getSystemClasspathElements()
>>>>>> throws DependencyResolutionRequiredException
>>>>>> {
>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>> + List<String> list = new
>>>>>> ArrayList<String>( getArtifacts().size()
>>>>>> );
>>>>>>
>>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>>
>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -820,11 +823,11 @@
>>>>>> return list;
>>>>>> }
>>>>>>
>>>>>> - public List getSystemArtifacts()
>>>>>> + public List<Artifact> getSystemArtifacts()
>>>>>> {
>>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>>> getArtifacts().size() );
>>>>>>
>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -841,18 +844,18 @@
>>>>>> return list;
>>>>>> }
>>>>>>
>>>>>> - public List getSystemDependencies()
>>>>>> + public List<Dependency> getSystemDependencies()
>>>>>> {
>>>>>> - Set artifacts = getArtifacts();
>>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>>
>>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>>> {
>>>>>> - return Collections.EMPTY_LIST;
>>>>>> + return Collections.<Dependency> emptyList();
>>>>>> }
>>>>>>
>>>>>> - List list = new ArrayList( artifacts.size() );
>>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>>> artifacts.size() );
>>>>>>
>>>>>> - for ( Iterator i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>>> i.hasNext(); )
>>>>>> {
>>>>>> Artifact a = (Artifact) i.next();
>>>>>>
>>>>>> @@ -1050,12 +1053,12 @@
>>>>>> return getModel().getScm();
>>>>>> }
>>>>>>
>>>>>> - public void setMailingLists( List mailingLists )
>>>>>> + public void setMailingLists( List<MailingList>
>>>>>> mailingLists )
>>>>>> {
>>>>>> getModel().setMailingLists( mailingLists );
>>>>>> }
>>>>>>
>>>>>> - public List getMailingLists()
>>>>>> + public List<?> getMailingLists()
>>>>>> {
>>>>>> return getModel().getMailingLists();
>>>>>> }
>>>>>> @@ -1065,12 +1068,12 @@
>>>>>> getModel().addMailingList( mailingList );
>>>>>> }
>>>>>>
>>>>>> - public void setDevelopers( List developers )
>>>>>> + public void setDevelopers( List<Developer> developers )
>>>>>> {
>>>>>> getModel().setDevelopers( developers );
>>>>>> }
>>>>>>
>>>>>> - public List getDevelopers()
>>>>>> + public List<Developer> getDevelopers()
>>>>>> {
>>>>>> return getModel().getDevelopers();
>>>>>> }
>>>>>> @@ -1080,12 +1083,12 @@
>>>>>> getModel().addDeveloper( developer );
>>>>>> }
>>>>>>
>>>>>> - public void setContributors( List contributors )
>>>>>> + public void setContributors( List<Contributor>
>>>>>> contributors )
>>>>>> {
>>>>>> getModel().setContributors( contributors );
>>>>>> }
>>>>>>
>>>>>> - public List getContributors()
>>>>>> + public List<Contributor> getContributors()
>>>>>> {
>>>>>> return getModel().getContributors();
>>>>>> }
>>>>>> @@ -1105,12 +1108,12 @@
>>>>>> return getModelBuild();
>>>>>> }
>>>>>>
>>>>>> - public List getResources()
>>>>>> + public List<Resource> getResources()
>>>>>> {
>>>>>> return getBuild().getResources();
>>>>>> }
>>>>>>
>>>>>> - public List getTestResources()
>>>>>> + public List<Resource> getTestResources()
>>>>>> {
>>>>>> return getBuild().getTestResources();
>>>>>> }
>>>>>> @@ -1135,12 +1138,12 @@
>>>>>> return getModel().getReporting();
>>>>>> }
>>>>>>
>>>>>> - public void setLicenses( List licenses )
>>>>>> + public void setLicenses( List<License> licenses )
>>>>>> {
>>>>>> getModel().setLicenses( licenses );
>>>>>> }
>>>>>>
>>>>>> - public List getLicenses()
>>>>>> + public List<License> getLicenses()
>>>>>> {
>>>>>> return getModel().getLicenses();
>>>>>> }
>>>>>> @@ -1150,7 +1153,7 @@
>>>>>> getModel().addLicense( license );
>>>>>> }
>>>>>>
>>>>>> - public void setArtifacts( Set artifacts )
>>>>>> + public void setArtifacts( Set<Artifact> artifacts )
>>>>>> {
>>>>>> this.artifacts = artifacts;
>>>>>>
>>>>>> @@ -1165,12 +1168,12 @@
>>>>>> * @return {@link Set} < {@link Artifact} >
>>>>>> * @see #getDependencyArtifacts() to get only direct
>>>>>> dependencies
>>>>>> */
>>>>>> - public Set getArtifacts()
>>>>>> + public Set<Artifact> getArtifacts()
>>>>>> {
>>>>>> - return artifacts == null ? Collections.EMPTY_SET :
>>>>>> artifacts;
>>>>>> + return artifacts == null ? Collections.<Artifact>
>>>>>> emptySet() :
>>>>>> artifacts;
>>>>>> }
>>>>>>
>>>>>> - public Map getArtifactMap()
>>>>>> + public Map<String, Artifact> getArtifactMap()
>>>>>> {
>>>>>> if ( artifactMap == null )
>>>>>> {
>>>>>> @@ -1180,19 +1183,19 @@
>>>>>> return artifactMap;
>>>>>> }
>>>>>>
>>>>>> - public void setPluginArtifacts( Set pluginArtifacts )
>>>>>> + public void setPluginArtifacts( Set<Artifact>
>>>>>> pluginArtifacts )
>>>>>> {
>>>>>> this.pluginArtifacts = pluginArtifacts;
>>>>>>
>>>>>> this.pluginArtifactMap = null;
>>>>>> }
>>>>>>
>>>>>> - public Set getPluginArtifacts()
>>>>>> + public Set<Artifact> getPluginArtifacts()
>>>>>> {
>>>>>> return pluginArtifacts;
>>>>>> }
>>>>>>
>>>>>> - public Map getPluginArtifactMap()
>>>>>> + public Map<String, Artifact> getPluginArtifactMap()
>>>>>> {
>>>>>> if ( pluginArtifactMap == null )
>>>>>> {
>>>>>> @@ -1202,19 +1205,19 @@
>>>>>> return pluginArtifactMap;
>>>>>> }
>>>>>>
>>>>>> - public void setReportArtifacts( Set reportArtifacts )
>>>>>> + public void setReportArtifacts( Set<Artifact>
>>>>>> reportArtifacts )
>>>>>> {
>>>>>> this.reportArtifacts = reportArtifacts;
>>>>>>
>>>>>> this.reportArtifactMap = null;
>>>>>> }
>>>>>>
>>>>>> - public Set getReportArtifacts()
>>>>>> + public Set<Artifact> getReportArtifacts()
>>>>>> {
>>>>>> return reportArtifacts;
>>>>>> }
>>>>>>
>>>>>> - public Map getReportArtifactMap()
>>>>>> + public Map<String, Artifact> getReportArtifactMap()
>>>>>> {
>>>>>> if ( reportArtifactMap == null )
>>>>>> {
>>>>>> @@ -1224,19 +1227,19 @@
>>>>>> return reportArtifactMap;
>>>>>> }
>>>>>>
>>>>>> - public void setExtensionArtifacts( Set extensionArtifacts )
>>>>>> + public void setExtensionArtifacts( Set<Artifact>
>>>>>> extensionArtifacts
>>>>>> )
>>>>>> {
>>>>>> this.extensionArtifacts = extensionArtifacts;
>>>>>>
>>>>>> this.extensionArtifactMap = null;
>>>>>> }
>>>>>>
>>>>>> - public Set getExtensionArtifacts()
>>>>>> + public Set<Artifact> getExtensionArtifacts()
>>>>>> {
>>>>>> return this.extensionArtifacts;
>>>>>> }
>>>>>>
>>>>>> - public Map getExtensionArtifactMap()
>>>>>> + public Map<String, Artifact> getExtensionArtifactMap()
>>>>>> {
>>>>>> if ( extensionArtifactMap == null )
>>>>>> {
>>>>>> @@ -1256,7 +1259,7 @@
>>>>>> return parentArtifact;
>>>>>> }
>>>>>>
>>>>>> - public List getRepositories()
>>>>>> + public List<Repository> getRepositories()
>>>>>> {
>>>>>> return getModel().getRepositories();
>>>>>> }
>>>>>> @@ -1265,7 +1268,7 @@
>>>>>> // Plugins
>>>>>> //
>>>>>> ----------------------------------------------------------------------
>>>>>>
>>>>>> - public List getReportPlugins()
>>>>>> + public List<ReportPlugin> getReportPlugins()
>>>>>> {
>>>>>> if ( getModel().getReporting() == null )
>>>>>> {
>>>>>> @@ -1275,7 +1278,7 @@
>>>>>>
>>>>>> }
>>>>>>
>>>>>> - public List getBuildPlugins()
>>>>>> + public List<Plugin> getBuildPlugins()
>>>>>> {
>>>>>> if ( getModel().getBuild() == null )
>>>>>> {
>>>>>> @@ -1284,7 +1287,7 @@
>>>>>> return getModel().getBuild().getPlugins();
>>>>>> }
>>>>>>
>>>>>> - public List getModules()
>>>>>> + public List<String> getModules()
>>>>>> {
>>>>>> return getModel().getModules();
>>>>>> }
>>>>>> @@ -1335,7 +1338,7 @@
>>>>>>
>>>>>> if ( pm != null )
>>>>>> {
>>>>>> - Map pmByKey = pm.getPluginsAsMap();
>>>>>> + Map<String, Plugin> pmByKey = pm.getPluginsAsMap();
>>>>>>
>>>>>> String pluginKey = plugin.getKey();
>>>>>>
>>>>>> @@ -1348,17 +1351,17 @@
>>>>>> }
>>>>>> }
>>>>>>
>>>>>> - public List getCollectedProjects()
>>>>>> + public List<MavenProject> getCollectedProjects()
>>>>>> {
>>>>>> return collectedProjects;
>>>>>> }
>>>>>>
>>>>>> - public void setCollectedProjects( List collectedProjects )
>>>>>> + public void setCollectedProjects( List<MavenProject>
>>>>>> collectedProjects )
>>>>>> {
>>>>>> this.collectedProjects = collectedProjects;
>>>>>> }
>>>>>>
>>>>>> - public void setPluginArtifactRepositories( List
>>>>>> pluginArtifactRepositories )
>>>>>> + public void
>>>>>> setPluginArtifactRepositories( List<ArtifactRepository>
>>>>>> pluginArtifactRepositories )
>>>>>> {
>>>>>> this.pluginArtifactRepositories =
>>>>>> pluginArtifactRepositories;
>>>>>> }
>>>>>> @@ -1367,7 +1370,7 @@
>>>>>> * @return a list of ArtifactRepository objects constructed
>>>>>> * from the Repository objects returned by
>>>>>> getPluginRepositories.
>>>>>> */
>>>>>> - public List getPluginArtifactRepositories()
>>>>>> + public List<ArtifactRepository>
>>>>>> getPluginArtifactRepositories()
>>>>>> {
>>>>>> return pluginArtifactRepositories;
>>>>>> }
>>>>>> @@ -1378,17 +1381,17 @@
>>>>>> : getReleaseArtifactRepository();
>>>>>> }
>>>>>>
>>>>>> - public List getPluginRepositories()
>>>>>> + public List<Repository> getPluginRepositories()
>>>>>> {
>>>>>> return getModel().getPluginRepositories();
>>>>>> }
>>>>>>
>>>>>> - public void setActiveProfiles( List activeProfiles )
>>>>>> + public void setActiveProfiles( List<String> activeProfiles )
>>>>>> {
>>>>>> this.activeProfiles.addAll( activeProfiles );
>>>>>> }
>>>>>>
>>>>>> - public List getActiveProfiles()
>>>>>> + public List<String> getActiveProfiles()
>>>>>> {
>>>>>> return activeProfiles;
>>>>>> }
>>>>>> @@ -1398,11 +1401,11 @@
>>>>>> getAttachedArtifacts().add( artifact );
>>>>>> }
>>>>>>
>>>>>> - public List getAttachedArtifacts()
>>>>>> + public List<Artifact> getAttachedArtifacts()
>>>>>> {
>>>>>> if ( attachedArtifacts == null )
>>>>>> {
>>>>>> - attachedArtifacts = new ArrayList();
>>>>>> + attachedArtifacts = new ArrayList<Artifact>();
>>>>>> }
>>>>>> return attachedArtifacts;
>>>>>> }
>>>>>> @@ -1420,7 +1423,7 @@
>>>>>>
>>>>>> if ( getBuildPlugins() != null )
>>>>>> {
>>>>>> - for ( Iterator iterator =
>>>>>> getBuildPlugins().iterator();
>>>>>> iterator.hasNext(); )
>>>>>> + for ( Iterator<Plugin> iterator =
>>>>>> getBuildPlugins().iterator(); iterator.hasNext(); )
>>>>>> {
>>>>>> Plugin plugin = (Plugin) iterator.next();
>>>>>>
>>>>>> @@ -1498,7 +1501,7 @@
>>>>>>
>>>>>> if ( getReportPlugins() != null )
>>>>>> {
>>>>>> - for ( Iterator iterator =
>>>>>> getReportPlugins().iterator();
>>>>>> iterator.hasNext(); )
>>>>>> + for ( Iterator<ReportPlugin> iterator =
>>>>>> getReportPlugins().iterator(); iterator.hasNext(); )
>>>>>> {
>>>>>> ReportPlugin plugin = (ReportPlugin)
>>>>>> iterator.next();
>>>>>>
>>>>>> @@ -1564,12 +1567,12 @@
>>>>>> * @return {@link Set} < {@link Artifact} >
>>>>>> * @see #getArtifacts() to get all transitive dependencies
>>>>>> */
>>>>>> - public Set getDependencyArtifacts()
>>>>>> + public Set<Artifact> getDependencyArtifacts()
>>>>>> {
>>>>>> return dependencyArtifacts;
>>>>>> }
>>>>>>
>>>>>> - public void setDependencyArtifacts( Set
>>>>>> dependencyArtifacts )
>>>>>> + public void setDependencyArtifacts( Set<Artifact>
>>>>>> dependencyArtifacts )
>>>>>> {
>>>>>> this.dependencyArtifacts = dependencyArtifacts;
>>>>>> }
>>>>>> @@ -1594,12 +1597,12 @@
>>>>>> return originalModel;
>>>>>> }
>>>>>>
>>>>>> - public void setManagedVersionMap( Map map )
>>>>>> + public void setManagedVersionMap( Map<String,
>>>>>> ArtifactVersion> map )
>>>>>> {
>>>>>> this.managedVersionMap = map;
>>>>>> }
>>>>>>
>>>>>> - public Map getManagedVersionMap()
>>>>>> + public Map<String, ArtifactVersion> getManagedVersionMap()
>>>>>> {
>>>>>> return this.managedVersionMap;
>>>>>> }
>>>>>> @@ -1627,12 +1630,12 @@
>>>>>> return getId().hashCode();
>>>>>> }
>>>>>>
>>>>>> - public List getBuildExtensions()
>>>>>> + public List<Extension> getBuildExtensions()
>>>>>> {
>>>>>> Build build = getBuild();
>>>>>> if ( build == null || build.getExtensions() == null )
>>>>>> {
>>>>>> - return Collections.EMPTY_LIST;
>>>>>> + return Collections.<Extension> emptyList();
>>>>>> }
>>>>>> else
>>>>>> {
>>>>>> @@ -1644,7 +1647,7 @@
>>>>>> * @todo the lazy initialisation of this makes me uneasy.
>>>>>> * @return {@link Set} < {@link Artifact} >
>>>>>> */
>>>>>> - public Set createArtifacts( ArtifactFactory
>>>>>> artifactFactory, String
>>>>>> inheritedScope,
>>>>>> + public Set<Artifact> createArtifacts( ArtifactFactory
>>>>>> artifactFactory, String inheritedScope,
>>>>>> ArtifactFilter dependencyFilter )
>>>>>> throws InvalidDependencyVersionException
>>>>>> {
>>>>>> @@ -1674,12 +1677,12 @@
>>>>>> return getModel().getProperties();
>>>>>> }
>>>>>>
>>>>>> - public List getFilters()
>>>>>> + public List<String> getFilters()
>>>>>> {
>>>>>> return getBuild().getFilters();
>>>>>> }
>>>>>>
>>>>>> - public Map getProjectReferences()
>>>>>> + public Map<String, MavenProject> getProjectReferences()
>>>>>> {
>>>>>> return projectReferences;
>>>>>> }
>>>>>> @@ -1705,22 +1708,22 @@
>>>>>> this.model = model;
>>>>>> }
>>>>>>
>>>>>> - protected void setAttachedArtifacts( List
>>>>>> attachedArtifacts )
>>>>>> + protected void setAttachedArtifacts( List<Artifact>
>>>>>> attachedArtifacts )
>>>>>> {
>>>>>> this.attachedArtifacts = attachedArtifacts;
>>>>>> }
>>>>>>
>>>>>> - protected void setCompileSourceRoots( List
>>>>>> compileSourceRoots )
>>>>>> + protected void setCompileSourceRoots( List<String>
>>>>>> compileSourceRoots )
>>>>>> {
>>>>>> this.compileSourceRoots = compileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - protected void setTestCompileSourceRoots( List
>>>>>> testCompileSourceRoots )
>>>>>> + protected void setTestCompileSourceRoots( List<String>
>>>>>> testCompileSourceRoots )
>>>>>> {
>>>>>> this.testCompileSourceRoots = testCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - protected void setScriptSourceRoots( List
>>>>>> scriptSourceRoots )
>>>>>> + protected void setScriptSourceRoots( List<String>
>>>>>> scriptSourceRoots
>>>>>> )
>>>>>> {
>>>>>> this.scriptSourceRoots = scriptSourceRoots;
>>>>>> }
>>>>>> @@ -1737,16 +1740,16 @@
>>>>>>
>>>>>> public void resolveActiveArtifacts()
>>>>>> {
>>>>>> - Set depArtifacts = getDependencyArtifacts();
>>>>>> + Set<Artifact> depArtifacts = getDependencyArtifacts();
>>>>>> if ( depArtifacts == null )
>>>>>> {
>>>>>> return;
>>>>>> }
>>>>>>
>>>>>> - Set updated = new LinkedHashSet( depArtifacts.size() );
>>>>>> + Set<Artifact> updated = new LinkedHashSet<Artifact>(
>>>>>> depArtifacts.size() );
>>>>>> int updatedCount = 0;
>>>>>>
>>>>>> - for ( Iterator it = depArtifacts.iterator();
>>>>>> it.hasNext(); )
>>>>>> + for ( Iterator<Artifact> it = depArtifacts.iterator();
>>>>>> it.hasNext(); )
>>>>>> {
>>>>>> Artifact depArtifact = (Artifact) it.next();
>>>>>> Artifact replaced =
>>>>>> replaceWithActiveArtifact( depArtifact );
>>>>>> @@ -1815,13 +1818,13 @@
>>>>>> * @param requestedArtifact The artifact to resolve, must not be
>>>>>> <code>null</code>.
>>>>>> * @return The matching artifact or <code>null</code> if not
>>>>>> found.
>>>>>> */
>>>>>> - private Artifact findMatchingArtifact( List artifacts,
>>>>>> Artifact
>>>>>> requestedArtifact )
>>>>>> + private Artifact findMatchingArtifact( List<Artifact>
>>>>>> artifacts,
>>>>>> Artifact requestedArtifact )
>>>>>> {
>>>>>> if ( artifacts != null && !artifacts.isEmpty() )
>>>>>> {
>>>>>> // first try matching by dependency conflict id
>>>>>> String requestedId =
>>>>>> requestedArtifact.getDependencyConflictId();
>>>>>> - for ( Iterator it = artifacts.iterator();
>>>>>> it.hasNext(); )
>>>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>>>> it.hasNext(); )
>>>>>> {
>>>>>> Artifact artifact = (Artifact) it.next();
>>>>>> if ( requestedId.equals(
>>>>>> artifact.getDependencyConflictId() ) )
>>>>>> @@ -1832,7 +1835,7 @@
>>>>>>
>>>>>> // next try matching by repository conflict id
>>>>>> requestedId =
>>>>>> getRepositoryConflictId( requestedArtifact );
>>>>>> - for ( Iterator it = artifacts.iterator();
>>>>>> it.hasNext(); )
>>>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>>>> it.hasNext(); )
>>>>>> {
>>>>>> Artifact artifact = (Artifact) it.next();
>>>>>> if
>>>>>> ( requestedId.equals( getRepositoryConflictId( artifact
>>>>>> ) ) )
>>>>>> @@ -1898,7 +1901,7 @@
>>>>>> }
>>>>>> }
>>>>>>
>>>>>> - private void addArtifactPath(Artifact a, List list) throws
>>>>>> DependencyResolutionRequiredException
>>>>>> + private void addArtifactPath(Artifact a, List<String>
>>>>>> list) throws
>>>>>> DependencyResolutionRequiredException
>>>>>> {
>>>>>> File file = a.getFile();
>>>>>> if ( file == null )
>>>>>> @@ -1954,21 +1957,21 @@
>>>>>>
>>>>>> private Build originalInterpolatedBuild;
>>>>>>
>>>>>> - private List dynamicCompileSourceRoots;
>>>>>> + private List<String> dynamicCompileSourceRoots;
>>>>>>
>>>>>> - private List originalInterpolatedCompileSourceRoots;
>>>>>> + private List<String> originalInterpolatedCompileSourceRoots;
>>>>>>
>>>>>> - private List dynamicTestCompileSourceRoots;
>>>>>> + private List<String> dynamicTestCompileSourceRoots;
>>>>>>
>>>>>> - private List originalInterpolatedTestCompileSourceRoots;
>>>>>> + private List<String>
>>>>>> originalInterpolatedTestCompileSourceRoots;
>>>>>>
>>>>>> - private List dynamicScriptSourceRoots;
>>>>>> + private List<String> dynamicScriptSourceRoots;
>>>>>>
>>>>>> - private List originalInterpolatedScriptSourceRoots;
>>>>>> + private List<String> originalInterpolatedScriptSourceRoots;
>>>>>>
>>>>>> private boolean isConcrete = false;
>>>>>>
>>>>>> - public boolean isConcrete()
>>>>>> + public boolean isConcrete()
>>>>>> {
>>>>>> return isConcrete;
>>>>>> }
>>>>>> @@ -1988,32 +1991,32 @@
>>>>>> return originalInterpolatedBuild;
>>>>>> }
>>>>>>
>>>>>> - public List getDynamicCompileSourceRoots()
>>>>>> + public List<String> getDynamicCompileSourceRoots()
>>>>>> {
>>>>>> return dynamicCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - public List getOriginalInterpolatedCompileSourceRoots()
>>>>>> + public List<String>
>>>>>> getOriginalInterpolatedCompileSourceRoots()
>>>>>> {
>>>>>> return originalInterpolatedCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - public List getDynamicTestCompileSourceRoots()
>>>>>> + public List<String> getDynamicTestCompileSourceRoots()
>>>>>> {
>>>>>> return dynamicTestCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - public List getOriginalInterpolatedTestCompileSourceRoots()
>>>>>> + public List<String>
>>>>>> getOriginalInterpolatedTestCompileSourceRoots()
>>>>>> {
>>>>>> return originalInterpolatedTestCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - public List getDynamicScriptSourceRoots()
>>>>>> + public List<String> getDynamicScriptSourceRoots()
>>>>>> {
>>>>>> return dynamicScriptSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - public List getOriginalInterpolatedScriptSourceRoots()
>>>>>> + public List<String>
>>>>>> getOriginalInterpolatedScriptSourceRoots()
>>>>>> {
>>>>>> return originalInterpolatedScriptSourceRoots;
>>>>>> }
>>>>>> @@ -2034,19 +2037,19 @@
>>>>>> originalInterpolatedBuild = null;
>>>>>> }
>>>>>>
>>>>>> - public void preserveCompileSourceRoots( List
>>>>>> originalInterpolatedCompileSourceRoots )
>>>>>> + public void preserveCompileSourceRoots( List<String>
>>>>>> originalInterpolatedCompileSourceRoots )
>>>>>> {
>>>>>> dynamicCompileSourceRoots = getCompileSourceRoots();
>>>>>> this.originalInterpolatedCompileSourceRoots =
>>>>>> originalInterpolatedCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - public void preserveTestCompileSourceRoots( List
>>>>>> originalInterpolatedTestCompileSourceRoots )
>>>>>> + public void preserveTestCompileSourceRoots( List<String>
>>>>>> originalInterpolatedTestCompileSourceRoots )
>>>>>> {
>>>>>> dynamicTestCompileSourceRoots = getTestCompileSourceRoots();
>>>>>> this.originalInterpolatedTestCompileSourceRoots =
>>>>>> originalInterpolatedTestCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - public void preserveScriptSourceRoots( List
>>>>>> originalInterpolatedScriptSourceRoots )
>>>>>> + public void preserveScriptSourceRoots( List<String>
>>>>>> originalInterpolatedScriptSourceRoots )
>>>>>> {
>>>>>> dynamicScriptSourceRoots = getScriptSourceRoots();
>>>>>> this.originalInterpolatedScriptSourceRoots =
>>>>>> originalInterpolatedScriptSourceRoots;
>>>>>> @@ -2071,32 +2074,32 @@
>>>>>> this.originalInterpolatedBuild = originalInterpolatedBuild;
>>>>>> }
>>>>>>
>>>>>> - protected void setDynamicCompileSourceRoots( List
>>>>>> dynamicCompileSourceRoots )
>>>>>> + protected void setDynamicCompileSourceRoots( List<String>
>>>>>> dynamicCompileSourceRoots )
>>>>>> {
>>>>>> this.dynamicCompileSourceRoots = dynamicCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - protected void
>>>>>> setOriginalInterpolatedCompileSourceRoots( List
>>>>>> originalInterpolatedCompileSourceRoots )
>>>>>> + protected void setOriginalInterpolatedCompileSourceRoots(
>>>>>> List<String> originalInterpolatedCompileSourceRoots )
>>>>>> {
>>>>>> this.originalInterpolatedCompileSourceRoots =
>>>>>> originalInterpolatedCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - protected void setDynamicTestCompileSourceRoots( List
>>>>>> dynamicTestCompileSourceRoots )
>>>>>> + protected void
>>>>>> setDynamicTestCompileSourceRoots( List<String>
>>>>>> dynamicTestCompileSourceRoots )
>>>>>> {
>>>>>> this.dynamicTestCompileSourceRoots =
>>>>>> dynamicTestCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - protected void
>>>>>> setOriginalInterpolatedTestCompileSourceRoots( List
>>>>>> originalInterpolatedTestCompileSourceRoots )
>>>>>> + protected void
>>>>>> setOriginalInterpolatedTestCompileSourceRoots(
>>>>>> List<String> originalInterpolatedTestCompileSourceRoots )
>>>>>> {
>>>>>> this.originalInterpolatedTestCompileSourceRoots =
>>>>>> originalInterpolatedTestCompileSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - protected void setDynamicScriptSourceRoots( List
>>>>>> dynamicScriptSourceRoots )
>>>>>> + protected void setDynamicScriptSourceRoots( List<String>
>>>>>> dynamicScriptSourceRoots )
>>>>>> {
>>>>>> this.dynamicScriptSourceRoots = dynamicScriptSourceRoots;
>>>>>> }
>>>>>>
>>>>>> - protected void
>>>>>> setOriginalInterpolatedScriptSourceRoots( List
>>>>>> originalInterpolatedScriptSourceRoots )
>>>>>> + protected void setOriginalInterpolatedScriptSourceRoots(
>>>>>> List<String> originalInterpolatedScriptSourceRoots )
>>>>>> {
>>>>>> this.originalInterpolatedScriptSourceRoots =
>>>>>> originalInterpolatedScriptSourceRoots;
>>>>>> }
>>>>>> @@ -2114,7 +2117,7 @@
>>>>>> if ( p != null )
>>>>>> {
>>>>>> preservedProperties = new Properties();
>>>>>> - for( Enumeration e = p.propertyNames();
>>>>>> e.hasMoreElements();
>>>>>> )
>>>>>> + for( Enumeration<?> e = p.propertyNames();
>>>>>> e.hasMoreElements(); )
>>>>>> {
>>>>>> String key = (String) e.nextElement();
>>>>>> preservedProperties.setProperty( key,
>>>>>> p.getProperty( key )
>>>>>> );
>>>>>>
>>>>>> Modified:
>>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/
>>>>>> pom.xml
>>>>>> URL:
>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =================================================================
>>>>>>
>>>>>> ---
>>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/
>>>>>> pom.xml
>>>>>> (original)
>>>>>> +++
>>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/
>>>>>> pom.xml Fri
>>>>>> May 1 08:04:48 2009
>>>>>> @@ -47,6 +47,7 @@
>>>>>> <models>
>>>>>> <model>src/main/mdo/metadata.mdo</model>
>>>>>> </models>
>>>>>> + <useJava5>true</useJava5>
>>>>>> </configuration>
>>>>>> </plugin>
>>>>>> </plugins>
>>>>>>
>>>>>> Modified: maven/components/branches/maven-2.2.x/maven-settings/
>>>>>> pom.xml
>>>>>> URL:
>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-settings/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =================================================================
>>>>>>
>>>>>> --- maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>>>> (original)
>>>>>> +++ maven/components/branches/maven-2.2.x/maven-settings/
>>>>>> pom.xml Fri May
>>>>>> 1 08:04:48 2009
>>>>>> @@ -58,6 +58,7 @@
>>>>>> <models>
>>>>>> <model>src/main/mdo/settings.mdo</model>
>>>>>> </models>
>>>>>> + <useJava5>true</useJava5>
>>>>>> </configuration>
>>>>>> </plugin>
>>>>>> </plugins>
>>>>>>
>>>>>> Modified: maven/components/branches/maven-2.2.x/maven-toolchain/
>>>>>> pom.xml
>>>>>> URL:
>>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =
>>>>>> =================================================================
>>>>>>
>>>>>> --- maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>>>> (original)
>>>>>> +++ maven/components/branches/maven-2.2.x/maven-toolchain/
>>>>>> pom.xml Fri May
>>>>>> 1 08:04:48 2009
>>>>>> @@ -66,6 +66,7 @@
>>>>>> <models>
>>>>>> <model>src/main/mdo/toolchains.mdo</model>
>>>>>> </models>
>>>>>> + <useJava5>true</useJava5>
>>>>>> </configuration>
>>>>>> </plugin>
>>>>>> <plugin>
>>>>>>
>>>>>>
>>>>>>
>>>>> Thanks,
>>>>>
>>>>> Jason
>>>>>
>>>>> ----------------------------------------------------------
>>>>> Jason van Zyl
>>>>> Founder, Apache Maven
>>>>> http://twitter.com/jvanzyl
>>>>> http://twitter.com/SonatypeNexus
>>>>> http://twitter.com/SonatypeM2E
>>>>> ----------------------------------------------------------
>>>>>
>>>>> We all have problems. How we deal with them is a measure of our
>>>>> worth.
>>>>>
>>>>> -- Unknown
>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>>>>> For additional commands, e-mail: dev-help@maven.apache.org
>>>>>
>>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>>>> For additional commands, e-mail: dev-help@maven.apache.org
>>>>
>>>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>> For additional commands, e-mail: dev-help@maven.apache.org
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
Thanks,
Jason
----------------------------------------------------------
Jason van Zyl
Founder, Apache Maven
http://twitter.com/jvanzyl
http://twitter.com/SonatypeNexus
http://twitter.com/SonatypeM2E
----------------------------------------------------------
To do two things at once is to do neither.
-—Publilius Syrus, Roman slave, first century B.C.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: svn commit: r770570 - in /maven/components/branches/maven-2.2.x: maven-artifact/src/main/java/org/apache/maven/artifact/ maven-model/ maven-profile/ maven-project/src/main/java/org/apache/maven/project/ maven-repository-metadata/ maven-settings/
Posted by Brett Porter <br...@apache.org>.
On 03/05/2009, at 6:58 AM, John Casey wrote:
> The ITs all looked fine with this commit, so it looks like this
> stuff won't cause too much grief...but hopefully the testing of the
> RC will tell.
>
> I've created a 2.2.0-RC branch for subsequent RCs and the final
> release of 2.2.0, so we can continue converting the syntax on the
> 2.2.x branch.
Makes sense to me...
>>>>>
>>>>> On 1-May-09, at 1:04 AM, nicolas@apache.org wrote:
>>>>>
>>>>> Author: nicolas
>>>>>> Date: Fri May 1 08:04:48 2009
>>>>>> New Revision: 770570
>>>>>>
>>>>>> URL: http://svn.apache.org/viewvc?rev=770570&view=rev
>>>>>> Log:
>>>>>> use java5 syntax.
>>>>>> -> set modello to generate java5 syntax
>>>>>> -> MavenProject updated to use generics on collections &
>>>>>> methods for
>>>>>> better type safety
Nicolas, can you make sure this has a JIRA issue for the release and
adjust the commit log.
Thanks,
Brett
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: svn commit: r770570 - in /maven/components/branches/maven-2.2.x:
maven-artifact/src/main/java/org/apache/maven/artifact/ maven-model/ maven-profile/
maven-project/src/main/java/org/apache/maven/project/ maven-repository-metadata/
maven-settings/
Posted by John Casey <jd...@commonjava.org>.
The ITs all looked fine with this commit, so it looks like this stuff
won't cause too much grief...but hopefully the testing of the RC will tell.
I've created a 2.2.0-RC branch for subsequent RCs and the final release
of 2.2.0, so we can continue converting the syntax on the 2.2.x branch.
-john
BRIAN FOX wrote:
> In general, no. I just wanted to draw John's attention to the changes
> since I know he was preparing to cut an RC. I am suspicious of all
> changes when a release is drawing near.
>
> On May 2, 2009, at 2:44 AM, nicolas de loof wrote:
>
>> Do we prefer plugin developers to use List<?> and get
>> ClassCastExceptions at
>> runtime or sue type-safe collections to help them create stronger code
>> ?Anyway,
>> few plugins allready use Java5. Java 1.4 based one will not be broken
>> as the
>> generics signature is only a compile-time check.
>>
>>
>> 2009/5/1 Brian Fox <br...@infinity.nu>
>>
>>> I'm not sure if this is in scope of what John is trying to do wrt to
>>> 2.2.
>>>
>>>
>>> Jason van Zyl wrote:
>>>
>>>> I don't believe anyone actually agreed to this yet. Are you sure
>>>> this is
>>>> not going to cause problems for users?
>>>>
>>>> On 1-May-09, at 1:04 AM, nicolas@apache.org wrote:
>>>>
>>>> Author: nicolas
>>>>> Date: Fri May 1 08:04:48 2009
>>>>> New Revision: 770570
>>>>>
>>>>> URL: http://svn.apache.org/viewvc?rev=770570&view=rev
>>>>> Log:
>>>>> use java5 syntax.
>>>>> -> set modello to generate java5 syntax
>>>>> -> MavenProject updated to use generics on collections & methods for
>>>>> better type safety
>>>>>
>>>>> Modified:
>>>>>
>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>
>>>>>
>>>>> maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>>> maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>>>
>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>>>>
>>>>>
>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
>>>>>
>>>>> maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>>> maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>>>
>>>>> Modified:
>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>
>>>>>
>>>>> URL:
>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java?rev=770570&r1=770569&r2=770570&view=diff
>>>>>
>>>>> ==============================================================================
>>>>>
>>>>>
>>>>> ---
>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>
>>>>> (original)
>>>>> +++
>>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java
>>>>>
>>>>> Fri May 1 08:04:48 2009
>>>>> @@ -86,13 +86,13 @@
>>>>> baseVersion;
>>>>> }
>>>>>
>>>>> - public static Map artifactMapByVersionlessId( Collection
>>>>> artifacts )
>>>>> + public static Map<String, Artifact> artifactMapByVersionlessId(
>>>>> Collection<Artifact> artifacts )
>>>>> {
>>>>> - Map artifactMap = new LinkedHashMap();
>>>>> + Map<String, Artifact> artifactMap = new LinkedHashMap<String,
>>>>> Artifact>();
>>>>>
>>>>> if ( artifacts != null )
>>>>> {
>>>>> - for ( Iterator it = artifacts.iterator(); it.hasNext(); )
>>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>>> it.hasNext(); )
>>>>> {
>>>>> Artifact artifact = (Artifact) it.next();
>>>>>
>>>>>
>>>>> Modified: maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>>> URL:
>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-model/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>
>>>>> ==============================================================================
>>>>>
>>>>>
>>>>> --- maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>>> (original)
>>>>> +++ maven/components/branches/maven-2.2.x/maven-model/pom.xml Fri
>>>>> May 1
>>>>> 08:04:48 2009
>>>>> @@ -47,6 +47,7 @@
>>>>> <models>
>>>>> <model>src/main/mdo/maven.mdo</model>
>>>>> </models>
>>>>> + <useJava5>true</useJava5>
>>>>> </configuration>
>>>>> </plugin>
>>>>> <plugin>
>>>>>
>>>>> Modified: maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>>> URL:
>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-profile/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>
>>>>> ==============================================================================
>>>>>
>>>>>
>>>>> --- maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>>> (original)
>>>>> +++ maven/components/branches/maven-2.2.x/maven-profile/pom.xml Fri
>>>>> May
>>>>> 1 08:04:48 2009
>>>>> @@ -54,6 +54,7 @@
>>>>> <models>
>>>>> <model>src/main/mdo/profiles.mdo</model>
>>>>> </models>
>>>>> + <useJava5>true</useJava5>
>>>>> </configuration>
>>>>> </plugin>
>>>>> </plugins>
>>>>>
>>>>> Modified:
>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>>>>
>>>>>
>>>>> URL:
>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java?rev=770570&r1=770569&r2=770570&view=diff
>>>>>
>>>>> ==============================================================================
>>>>>
>>>>>
>>>>> ---
>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>>>>
>>>>> (original)
>>>>> +++
>>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java
>>>>>
>>>>> Fri May 1 08:04:48 2009
>>>>> @@ -36,10 +36,11 @@
>>>>> import org.apache.maven.artifact.Artifact;
>>>>> import org.apache.maven.artifact.ArtifactUtils;
>>>>> import
>>>>> org.apache.maven.artifact.DependencyResolutionRequiredException;
>>>>> -import org.apache.maven.artifact.versioning.ManagedVersionMap;
>>>>> import org.apache.maven.artifact.factory.ArtifactFactory;
>>>>> import org.apache.maven.artifact.repository.ArtifactRepository;
>>>>> import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
>>>>> +import org.apache.maven.artifact.versioning.ArtifactVersion;
>>>>> +import org.apache.maven.artifact.versioning.ManagedVersionMap;
>>>>> import org.apache.maven.model.Build;
>>>>> import org.apache.maven.model.CiManagement;
>>>>> import org.apache.maven.model.Contributor;
>>>>> @@ -47,6 +48,7 @@
>>>>> import org.apache.maven.model.DependencyManagement;
>>>>> import org.apache.maven.model.Developer;
>>>>> import org.apache.maven.model.DistributionManagement;
>>>>> +import org.apache.maven.model.Extension;
>>>>> import org.apache.maven.model.IssueManagement;
>>>>> import org.apache.maven.model.License;
>>>>> import org.apache.maven.model.MailingList;
>>>>> @@ -59,6 +61,7 @@
>>>>> import org.apache.maven.model.ReportPlugin;
>>>>> import org.apache.maven.model.ReportSet;
>>>>> import org.apache.maven.model.Reporting;
>>>>> +import org.apache.maven.model.Repository;
>>>>> import org.apache.maven.model.Resource;
>>>>> import org.apache.maven.model.Scm;
>>>>> import org.apache.maven.model.io.xpp3.MavenXpp3Writer;
>>>>> @@ -96,60 +99,60 @@
>>>>>
>>>>> private File file;
>>>>>
>>>>> - private Set artifacts;
>>>>> + private Set<Artifact> artifacts;
>>>>>
>>>>> private Artifact parentArtifact;
>>>>>
>>>>> - private Set pluginArtifacts;
>>>>> + private Set<Artifact> pluginArtifacts;
>>>>>
>>>>> - private List remoteArtifactRepositories;
>>>>> + private List<ArtifactRepository> remoteArtifactRepositories;
>>>>>
>>>>> - private List collectedProjects = Collections.EMPTY_LIST;
>>>>> + private List<MavenProject> collectedProjects =
>>>>> Collections.<MavenProject> emptyList();
>>>>>
>>>>> - private List attachedArtifacts;
>>>>> + private List<Artifact> attachedArtifacts;
>>>>>
>>>>> private MavenProject executionProject;
>>>>>
>>>>> - private List compileSourceRoots = new ArrayList();
>>>>> + private List<String> compileSourceRoots = new
>>>>> ArrayList<String>();
>>>>>
>>>>> - private List testCompileSourceRoots = new ArrayList();
>>>>> + private List<String> testCompileSourceRoots = new
>>>>> ArrayList<String>();
>>>>>
>>>>> - private List scriptSourceRoots = new ArrayList();
>>>>> + private List<String> scriptSourceRoots = new ArrayList<String>();
>>>>>
>>>>> - private List pluginArtifactRepositories;
>>>>> + private List<ArtifactRepository> pluginArtifactRepositories;
>>>>>
>>>>> private ArtifactRepository releaseArtifactRepository;
>>>>>
>>>>> private ArtifactRepository snapshotArtifactRepository;
>>>>>
>>>>> - private List activeProfiles = new ArrayList();
>>>>> + private List<String> activeProfiles = new ArrayList<String>();
>>>>>
>>>>> - private Set dependencyArtifacts;
>>>>> + private Set<Artifact> dependencyArtifacts;
>>>>>
>>>>> private Artifact artifact;
>>>>>
>>>>> // calculated.
>>>>> - private Map artifactMap;
>>>>> + private Map<String, Artifact> artifactMap;
>>>>>
>>>>> private Model originalModel;
>>>>>
>>>>> - private Map pluginArtifactMap;
>>>>> + private Map<String, Artifact> pluginArtifactMap;
>>>>>
>>>>> - private Set reportArtifacts;
>>>>> + private Set<Artifact> reportArtifacts;
>>>>>
>>>>> - private Map reportArtifactMap;
>>>>> + private Map<String, Artifact> reportArtifactMap;
>>>>>
>>>>> - private Set extensionArtifacts;
>>>>> + private Set<Artifact> extensionArtifacts;
>>>>>
>>>>> - private Map extensionArtifactMap;
>>>>> + private Map<String, Artifact> extensionArtifactMap;
>>>>>
>>>>> - private Map managedVersionMap;
>>>>> + private Map<String, ArtifactVersion> managedVersionMap;
>>>>>
>>>>> - private Map projectReferences = new HashMap();
>>>>> + private Map<String, MavenProject> projectReferences = new
>>>>> HashMap<String, MavenProject>();
>>>>>
>>>>> private boolean executionRoot;
>>>>>
>>>>> - private Map moduleAdjustments;
>>>>> + private Map<String, String> moduleAdjustments;
>>>>>
>>>>> private File basedir;
>>>>>
>>>>> @@ -245,23 +248,23 @@
>>>>> if ( project.getAttachedArtifacts() != null )
>>>>> {
>>>>> // clone properties modifyable by plugins in a forked
>>>>> lifecycle
>>>>> - setAttachedArtifacts( new ArrayList(
>>>>> project.getAttachedArtifacts() ) );
>>>>> + setAttachedArtifacts( new ArrayList<Artifact>(
>>>>> project.getAttachedArtifacts() ) );
>>>>> }
>>>>>
>>>>> if ( project.getCompileSourceRoots() != null )
>>>>> {
>>>>> // clone source roots
>>>>> - setCompileSourceRoots( ( new ArrayList(
>>>>> project.getCompileSourceRoots() ) ) );
>>>>> + setCompileSourceRoots( ( new ArrayList<String>(
>>>>> project.getCompileSourceRoots() ) ) );
>>>>> }
>>>>>
>>>>> if ( project.getTestCompileSourceRoots() != null )
>>>>> {
>>>>> - setTestCompileSourceRoots( ( new ArrayList(
>>>>> project.getTestCompileSourceRoots() ) ) );
>>>>> + setTestCompileSourceRoots( ( new ArrayList<String>(
>>>>> project.getTestCompileSourceRoots() ) ) );
>>>>> }
>>>>>
>>>>> if ( project.getScriptSourceRoots() != null )
>>>>> {
>>>>> - setScriptSourceRoots( ( new ArrayList(
>>>>> project.getScriptSourceRoots() ) ) );
>>>>> + setScriptSourceRoots( ( new ArrayList<String>(
>>>>> project.getScriptSourceRoots() ) ) );
>>>>> }
>>>>>
>>>>> setModel( ( ModelUtils.cloneModel( project.getModel() ) ) );
>>>>> @@ -298,25 +301,25 @@
>>>>> setDynamicBuild( ModelUtils.cloneBuild(
>>>>> project.getDynamicBuild() ) );
>>>>> setOriginalInterpolatedBuild( ModelUtils.cloneBuild(
>>>>> project.getOriginalInterpolatedBuild() ) );
>>>>>
>>>>> - List dynamicRoots =
>>>>> project.getDynamicCompileSourceRoots();
>>>>> + List<String> dynamicRoots =
>>>>> project.getDynamicCompileSourceRoots();
>>>>> if ( dynamicRoots != null )
>>>>> {
>>>>> - setDynamicCompileSourceRoots( new ArrayList(
>>>>> dynamicRoots ) );
>>>>> - setOriginalInterpolatedCompileSourceRoots( new
>>>>> ArrayList( project.getOriginalInterpolatedCompileSourceRoots() ) );
>>>>> + setDynamicCompileSourceRoots( new ArrayList<String>(
>>>>> dynamicRoots ) );
>>>>> + setOriginalInterpolatedCompileSourceRoots( new
>>>>> ArrayList<String>(
>>>>> project.getOriginalInterpolatedCompileSourceRoots() ) );
>>>>> }
>>>>>
>>>>> dynamicRoots = project.getDynamicTestCompileSourceRoots();
>>>>> if ( dynamicRoots != null )
>>>>> {
>>>>> - setDynamicTestCompileSourceRoots( new ArrayList(
>>>>> dynamicRoots ) );
>>>>> - setOriginalInterpolatedTestCompileSourceRoots( new
>>>>> ArrayList( project.getOriginalInterpolatedTestCompileSourceRoots()
>>>>> ) );
>>>>> + setDynamicTestCompileSourceRoots( new
>>>>> ArrayList<String>(
>>>>> dynamicRoots ) );
>>>>> + setOriginalInterpolatedTestCompileSourceRoots( new
>>>>> ArrayList<String>(
>>>>> project.getOriginalInterpolatedTestCompileSourceRoots() )
>>>>> );
>>>>> }
>>>>>
>>>>> dynamicRoots = project.getDynamicScriptSourceRoots();
>>>>> if ( dynamicRoots != null )
>>>>> {
>>>>> - setDynamicScriptSourceRoots( new ArrayList(
>>>>> dynamicRoots
>>>>> ) );
>>>>> - setOriginalInterpolatedScriptSourceRoots( new
>>>>> ArrayList(
>>>>> project.getOriginalInterpolatedScriptSourceRoots() ) );
>>>>> + setDynamicScriptSourceRoots( new ArrayList<String>(
>>>>> dynamicRoots ) );
>>>>> + setOriginalInterpolatedScriptSourceRoots( new
>>>>> ArrayList<String>(
>>>>> project.getOriginalInterpolatedScriptSourceRoots() ) );
>>>>> }
>>>>> }
>>>>>
>>>>> @@ -347,12 +350,12 @@
>>>>>
>>>>> if ( moduleAdjustments == null )
>>>>> {
>>>>> - moduleAdjustments = new HashMap();
>>>>> + moduleAdjustments = new HashMap<String, String>();
>>>>>
>>>>> - List modules = getModules();
>>>>> + List<String> modules = getModules();
>>>>> if ( modules != null )
>>>>> {
>>>>> - for ( Iterator it = modules.iterator();
>>>>> it.hasNext(); )
>>>>> + for ( Iterator<String> it = modules.iterator();
>>>>> it.hasNext(); )
>>>>> {
>>>>> String modulePath = (String) it.next();
>>>>> String moduleName = modulePath;
>>>>> @@ -415,12 +418,12 @@
>>>>> this.parent = parent;
>>>>> }
>>>>>
>>>>> - public void setRemoteArtifactRepositories( List
>>>>> remoteArtifactRepositories )
>>>>> + public void setRemoteArtifactRepositories(
>>>>> List<ArtifactRepository>
>>>>> remoteArtifactRepositories )
>>>>> {
>>>>> this.remoteArtifactRepositories = remoteArtifactRepositories;
>>>>> }
>>>>>
>>>>> - public List getRemoteArtifactRepositories()
>>>>> + public List<ArtifactRepository> getRemoteArtifactRepositories()
>>>>> {
>>>>> return remoteArtifactRepositories;
>>>>> }
>>>>> @@ -460,12 +463,12 @@
>>>>> return basedir;
>>>>> }
>>>>>
>>>>> - public void setDependencies( List dependencies )
>>>>> + public void setDependencies( List<Dependency> dependencies )
>>>>> {
>>>>> getModel().setDependencies( dependencies );
>>>>> }
>>>>>
>>>>> - public List getDependencies()
>>>>> + public List<Dependency> getDependencies()
>>>>> {
>>>>> return getModel().getDependencies();
>>>>> }
>>>>> @@ -524,29 +527,29 @@
>>>>> }
>>>>> }
>>>>>
>>>>> - public List getCompileSourceRoots()
>>>>> + public List<String> getCompileSourceRoots()
>>>>> {
>>>>> return compileSourceRoots;
>>>>> }
>>>>>
>>>>> - public List getScriptSourceRoots()
>>>>> + public List<String> getScriptSourceRoots()
>>>>> {
>>>>> return scriptSourceRoots;
>>>>> }
>>>>>
>>>>> - public List getTestCompileSourceRoots()
>>>>> + public List<String> getTestCompileSourceRoots()
>>>>> {
>>>>> return testCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - public List getCompileClasspathElements()
>>>>> + public List<String> getCompileClasspathElements()
>>>>> throws DependencyResolutionRequiredException
>>>>> {
>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>> + List<String> list = new ArrayList<String>(
>>>>> getArtifacts().size()
>>>>> );
>>>>>
>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>
>>>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>>>> + for ( Iterator<?> i = getArtifacts().iterator();
>>>>> i.hasNext(); )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -563,11 +566,11 @@
>>>>> return list;
>>>>> }
>>>>>
>>>>> - public List getCompileArtifacts()
>>>>> + public List<Artifact> getCompileArtifacts()
>>>>> {
>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>> getArtifacts().size() );
>>>>>
>>>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>> i.hasNext(); )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -585,18 +588,18 @@
>>>>> return list;
>>>>> }
>>>>>
>>>>> - public List getCompileDependencies()
>>>>> + public List<Dependency> getCompileDependencies()
>>>>> {
>>>>> - Set artifacts = getArtifacts();
>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>
>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>> {
>>>>> - return Collections.EMPTY_LIST;
>>>>> + return Collections.<Dependency> emptyList();
>>>>> }
>>>>>
>>>>> - List list = new ArrayList( artifacts.size() );
>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>> artifacts.size() );
>>>>>
>>>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>> i.hasNext(); )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -619,16 +622,16 @@
>>>>> return list;
>>>>> }
>>>>>
>>>>> - public List getTestClasspathElements()
>>>>> + public List<String> getTestClasspathElements()
>>>>> throws DependencyResolutionRequiredException
>>>>> {
>>>>> - List list = new ArrayList( getArtifacts().size() + 1 );
>>>>> + List<String> list = new ArrayList<String>(
>>>>> getArtifacts().size()
>>>>> + 1 );
>>>>>
>>>>> list.add( getBuild().getTestOutputDirectory() );
>>>>>
>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>
>>>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>> i.hasNext(); )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -652,11 +655,11 @@
>>>>> return list;
>>>>> }
>>>>>
>>>>> - public List getTestArtifacts()
>>>>> + public List<Artifact> getTestArtifacts()
>>>>> {
>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>> getArtifacts().size() );
>>>>>
>>>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>> i.hasNext(); )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -678,18 +681,18 @@
>>>>> return list;
>>>>> }
>>>>>
>>>>> - public List getTestDependencies()
>>>>> + public List<Dependency> getTestDependencies()
>>>>> {
>>>>> - Set artifacts = getArtifacts();
>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>
>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>> {
>>>>> - return Collections.EMPTY_LIST;
>>>>> + return Collections.emptyList();
>>>>> }
>>>>>
>>>>> - List list = new ArrayList( artifacts.size() );
>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>> artifacts.size() );
>>>>>
>>>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>> i.hasNext(); )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -715,14 +718,14 @@
>>>>> return list;
>>>>> }
>>>>>
>>>>> - public List getRuntimeClasspathElements()
>>>>> + public List<String> getRuntimeClasspathElements()
>>>>> throws DependencyResolutionRequiredException
>>>>> {
>>>>> - List list = new ArrayList( getArtifacts().size() + 1 );
>>>>> + List<String> list = new ArrayList<String>(
>>>>> getArtifacts().size()
>>>>> + 1 );
>>>>>
>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>
>>>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>> i.hasNext(); )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -743,11 +746,11 @@
>>>>> return list;
>>>>> }
>>>>>
>>>>> - public List getRuntimeArtifacts()
>>>>> + public List<Artifact> getRuntimeArtifacts()
>>>>> {
>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>> getArtifacts().size() );
>>>>>
>>>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>> i.hasNext(); )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -764,18 +767,18 @@
>>>>> return list;
>>>>> }
>>>>>
>>>>> - public List getRuntimeDependencies()
>>>>> + public List<Dependency> getRuntimeDependencies()
>>>>> {
>>>>> - Set artifacts = getArtifacts();
>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>
>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>> {
>>>>> - return Collections.EMPTY_LIST;
>>>>> + return Collections.<Dependency> emptyList();
>>>>> }
>>>>>
>>>>> - List list = new ArrayList( artifacts.size() );
>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>> artifacts.size() );
>>>>>
>>>>> - for ( Iterator i = artifacts.iterator(); i.hasNext(); )
>>>>> + for ( Iterator<Artifact> i = artifacts.iterator();
>>>>> i.hasNext();
>>>>> )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -797,14 +800,14 @@
>>>>> return list;
>>>>> }
>>>>>
>>>>> - public List getSystemClasspathElements()
>>>>> + public List<String> getSystemClasspathElements()
>>>>> throws DependencyResolutionRequiredException
>>>>> {
>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>> + List<String> list = new ArrayList<String>(
>>>>> getArtifacts().size()
>>>>> );
>>>>>
>>>>> list.add( getBuild().getOutputDirectory() );
>>>>>
>>>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>> i.hasNext(); )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -820,11 +823,11 @@
>>>>> return list;
>>>>> }
>>>>>
>>>>> - public List getSystemArtifacts()
>>>>> + public List<Artifact> getSystemArtifacts()
>>>>> {
>>>>> - List list = new ArrayList( getArtifacts().size() );
>>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>>> getArtifacts().size() );
>>>>>
>>>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>> i.hasNext(); )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -841,18 +844,18 @@
>>>>> return list;
>>>>> }
>>>>>
>>>>> - public List getSystemDependencies()
>>>>> + public List<Dependency> getSystemDependencies()
>>>>> {
>>>>> - Set artifacts = getArtifacts();
>>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>>
>>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>>> {
>>>>> - return Collections.EMPTY_LIST;
>>>>> + return Collections.<Dependency> emptyList();
>>>>> }
>>>>>
>>>>> - List list = new ArrayList( artifacts.size() );
>>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>>> artifacts.size() );
>>>>>
>>>>> - for ( Iterator i = getArtifacts().iterator(); i.hasNext(); )
>>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>>> i.hasNext(); )
>>>>> {
>>>>> Artifact a = (Artifact) i.next();
>>>>>
>>>>> @@ -1050,12 +1053,12 @@
>>>>> return getModel().getScm();
>>>>> }
>>>>>
>>>>> - public void setMailingLists( List mailingLists )
>>>>> + public void setMailingLists( List<MailingList> mailingLists )
>>>>> {
>>>>> getModel().setMailingLists( mailingLists );
>>>>> }
>>>>>
>>>>> - public List getMailingLists()
>>>>> + public List<?> getMailingLists()
>>>>> {
>>>>> return getModel().getMailingLists();
>>>>> }
>>>>> @@ -1065,12 +1068,12 @@
>>>>> getModel().addMailingList( mailingList );
>>>>> }
>>>>>
>>>>> - public void setDevelopers( List developers )
>>>>> + public void setDevelopers( List<Developer> developers )
>>>>> {
>>>>> getModel().setDevelopers( developers );
>>>>> }
>>>>>
>>>>> - public List getDevelopers()
>>>>> + public List<Developer> getDevelopers()
>>>>> {
>>>>> return getModel().getDevelopers();
>>>>> }
>>>>> @@ -1080,12 +1083,12 @@
>>>>> getModel().addDeveloper( developer );
>>>>> }
>>>>>
>>>>> - public void setContributors( List contributors )
>>>>> + public void setContributors( List<Contributor> contributors )
>>>>> {
>>>>> getModel().setContributors( contributors );
>>>>> }
>>>>>
>>>>> - public List getContributors()
>>>>> + public List<Contributor> getContributors()
>>>>> {
>>>>> return getModel().getContributors();
>>>>> }
>>>>> @@ -1105,12 +1108,12 @@
>>>>> return getModelBuild();
>>>>> }
>>>>>
>>>>> - public List getResources()
>>>>> + public List<Resource> getResources()
>>>>> {
>>>>> return getBuild().getResources();
>>>>> }
>>>>>
>>>>> - public List getTestResources()
>>>>> + public List<Resource> getTestResources()
>>>>> {
>>>>> return getBuild().getTestResources();
>>>>> }
>>>>> @@ -1135,12 +1138,12 @@
>>>>> return getModel().getReporting();
>>>>> }
>>>>>
>>>>> - public void setLicenses( List licenses )
>>>>> + public void setLicenses( List<License> licenses )
>>>>> {
>>>>> getModel().setLicenses( licenses );
>>>>> }
>>>>>
>>>>> - public List getLicenses()
>>>>> + public List<License> getLicenses()
>>>>> {
>>>>> return getModel().getLicenses();
>>>>> }
>>>>> @@ -1150,7 +1153,7 @@
>>>>> getModel().addLicense( license );
>>>>> }
>>>>>
>>>>> - public void setArtifacts( Set artifacts )
>>>>> + public void setArtifacts( Set<Artifact> artifacts )
>>>>> {
>>>>> this.artifacts = artifacts;
>>>>>
>>>>> @@ -1165,12 +1168,12 @@
>>>>> * @return {@link Set} < {@link Artifact} >
>>>>> * @see #getDependencyArtifacts() to get only direct dependencies
>>>>> */
>>>>> - public Set getArtifacts()
>>>>> + public Set<Artifact> getArtifacts()
>>>>> {
>>>>> - return artifacts == null ? Collections.EMPTY_SET : artifacts;
>>>>> + return artifacts == null ? Collections.<Artifact>
>>>>> emptySet() :
>>>>> artifacts;
>>>>> }
>>>>>
>>>>> - public Map getArtifactMap()
>>>>> + public Map<String, Artifact> getArtifactMap()
>>>>> {
>>>>> if ( artifactMap == null )
>>>>> {
>>>>> @@ -1180,19 +1183,19 @@
>>>>> return artifactMap;
>>>>> }
>>>>>
>>>>> - public void setPluginArtifacts( Set pluginArtifacts )
>>>>> + public void setPluginArtifacts( Set<Artifact> pluginArtifacts )
>>>>> {
>>>>> this.pluginArtifacts = pluginArtifacts;
>>>>>
>>>>> this.pluginArtifactMap = null;
>>>>> }
>>>>>
>>>>> - public Set getPluginArtifacts()
>>>>> + public Set<Artifact> getPluginArtifacts()
>>>>> {
>>>>> return pluginArtifacts;
>>>>> }
>>>>>
>>>>> - public Map getPluginArtifactMap()
>>>>> + public Map<String, Artifact> getPluginArtifactMap()
>>>>> {
>>>>> if ( pluginArtifactMap == null )
>>>>> {
>>>>> @@ -1202,19 +1205,19 @@
>>>>> return pluginArtifactMap;
>>>>> }
>>>>>
>>>>> - public void setReportArtifacts( Set reportArtifacts )
>>>>> + public void setReportArtifacts( Set<Artifact> reportArtifacts )
>>>>> {
>>>>> this.reportArtifacts = reportArtifacts;
>>>>>
>>>>> this.reportArtifactMap = null;
>>>>> }
>>>>>
>>>>> - public Set getReportArtifacts()
>>>>> + public Set<Artifact> getReportArtifacts()
>>>>> {
>>>>> return reportArtifacts;
>>>>> }
>>>>>
>>>>> - public Map getReportArtifactMap()
>>>>> + public Map<String, Artifact> getReportArtifactMap()
>>>>> {
>>>>> if ( reportArtifactMap == null )
>>>>> {
>>>>> @@ -1224,19 +1227,19 @@
>>>>> return reportArtifactMap;
>>>>> }
>>>>>
>>>>> - public void setExtensionArtifacts( Set extensionArtifacts )
>>>>> + public void setExtensionArtifacts( Set<Artifact>
>>>>> extensionArtifacts
>>>>> )
>>>>> {
>>>>> this.extensionArtifacts = extensionArtifacts;
>>>>>
>>>>> this.extensionArtifactMap = null;
>>>>> }
>>>>>
>>>>> - public Set getExtensionArtifacts()
>>>>> + public Set<Artifact> getExtensionArtifacts()
>>>>> {
>>>>> return this.extensionArtifacts;
>>>>> }
>>>>>
>>>>> - public Map getExtensionArtifactMap()
>>>>> + public Map<String, Artifact> getExtensionArtifactMap()
>>>>> {
>>>>> if ( extensionArtifactMap == null )
>>>>> {
>>>>> @@ -1256,7 +1259,7 @@
>>>>> return parentArtifact;
>>>>> }
>>>>>
>>>>> - public List getRepositories()
>>>>> + public List<Repository> getRepositories()
>>>>> {
>>>>> return getModel().getRepositories();
>>>>> }
>>>>> @@ -1265,7 +1268,7 @@
>>>>> // Plugins
>>>>> //
>>>>> ----------------------------------------------------------------------
>>>>>
>>>>> - public List getReportPlugins()
>>>>> + public List<ReportPlugin> getReportPlugins()
>>>>> {
>>>>> if ( getModel().getReporting() == null )
>>>>> {
>>>>> @@ -1275,7 +1278,7 @@
>>>>>
>>>>> }
>>>>>
>>>>> - public List getBuildPlugins()
>>>>> + public List<Plugin> getBuildPlugins()
>>>>> {
>>>>> if ( getModel().getBuild() == null )
>>>>> {
>>>>> @@ -1284,7 +1287,7 @@
>>>>> return getModel().getBuild().getPlugins();
>>>>> }
>>>>>
>>>>> - public List getModules()
>>>>> + public List<String> getModules()
>>>>> {
>>>>> return getModel().getModules();
>>>>> }
>>>>> @@ -1335,7 +1338,7 @@
>>>>>
>>>>> if ( pm != null )
>>>>> {
>>>>> - Map pmByKey = pm.getPluginsAsMap();
>>>>> + Map<String, Plugin> pmByKey = pm.getPluginsAsMap();
>>>>>
>>>>> String pluginKey = plugin.getKey();
>>>>>
>>>>> @@ -1348,17 +1351,17 @@
>>>>> }
>>>>> }
>>>>>
>>>>> - public List getCollectedProjects()
>>>>> + public List<MavenProject> getCollectedProjects()
>>>>> {
>>>>> return collectedProjects;
>>>>> }
>>>>>
>>>>> - public void setCollectedProjects( List collectedProjects )
>>>>> + public void setCollectedProjects( List<MavenProject>
>>>>> collectedProjects )
>>>>> {
>>>>> this.collectedProjects = collectedProjects;
>>>>> }
>>>>>
>>>>> - public void setPluginArtifactRepositories( List
>>>>> pluginArtifactRepositories )
>>>>> + public void setPluginArtifactRepositories(
>>>>> List<ArtifactRepository>
>>>>> pluginArtifactRepositories )
>>>>> {
>>>>> this.pluginArtifactRepositories = pluginArtifactRepositories;
>>>>> }
>>>>> @@ -1367,7 +1370,7 @@
>>>>> * @return a list of ArtifactRepository objects constructed
>>>>> * from the Repository objects returned by
>>>>> getPluginRepositories.
>>>>> */
>>>>> - public List getPluginArtifactRepositories()
>>>>> + public List<ArtifactRepository> getPluginArtifactRepositories()
>>>>> {
>>>>> return pluginArtifactRepositories;
>>>>> }
>>>>> @@ -1378,17 +1381,17 @@
>>>>> : getReleaseArtifactRepository();
>>>>> }
>>>>>
>>>>> - public List getPluginRepositories()
>>>>> + public List<Repository> getPluginRepositories()
>>>>> {
>>>>> return getModel().getPluginRepositories();
>>>>> }
>>>>>
>>>>> - public void setActiveProfiles( List activeProfiles )
>>>>> + public void setActiveProfiles( List<String> activeProfiles )
>>>>> {
>>>>> this.activeProfiles.addAll( activeProfiles );
>>>>> }
>>>>>
>>>>> - public List getActiveProfiles()
>>>>> + public List<String> getActiveProfiles()
>>>>> {
>>>>> return activeProfiles;
>>>>> }
>>>>> @@ -1398,11 +1401,11 @@
>>>>> getAttachedArtifacts().add( artifact );
>>>>> }
>>>>>
>>>>> - public List getAttachedArtifacts()
>>>>> + public List<Artifact> getAttachedArtifacts()
>>>>> {
>>>>> if ( attachedArtifacts == null )
>>>>> {
>>>>> - attachedArtifacts = new ArrayList();
>>>>> + attachedArtifacts = new ArrayList<Artifact>();
>>>>> }
>>>>> return attachedArtifacts;
>>>>> }
>>>>> @@ -1420,7 +1423,7 @@
>>>>>
>>>>> if ( getBuildPlugins() != null )
>>>>> {
>>>>> - for ( Iterator iterator = getBuildPlugins().iterator();
>>>>> iterator.hasNext(); )
>>>>> + for ( Iterator<Plugin> iterator =
>>>>> getBuildPlugins().iterator(); iterator.hasNext(); )
>>>>> {
>>>>> Plugin plugin = (Plugin) iterator.next();
>>>>>
>>>>> @@ -1498,7 +1501,7 @@
>>>>>
>>>>> if ( getReportPlugins() != null )
>>>>> {
>>>>> - for ( Iterator iterator = getReportPlugins().iterator();
>>>>> iterator.hasNext(); )
>>>>> + for ( Iterator<ReportPlugin> iterator =
>>>>> getReportPlugins().iterator(); iterator.hasNext(); )
>>>>> {
>>>>> ReportPlugin plugin = (ReportPlugin) iterator.next();
>>>>>
>>>>> @@ -1564,12 +1567,12 @@
>>>>> * @return {@link Set} < {@link Artifact} >
>>>>> * @see #getArtifacts() to get all transitive dependencies
>>>>> */
>>>>> - public Set getDependencyArtifacts()
>>>>> + public Set<Artifact> getDependencyArtifacts()
>>>>> {
>>>>> return dependencyArtifacts;
>>>>> }
>>>>>
>>>>> - public void setDependencyArtifacts( Set dependencyArtifacts )
>>>>> + public void setDependencyArtifacts( Set<Artifact>
>>>>> dependencyArtifacts )
>>>>> {
>>>>> this.dependencyArtifacts = dependencyArtifacts;
>>>>> }
>>>>> @@ -1594,12 +1597,12 @@
>>>>> return originalModel;
>>>>> }
>>>>>
>>>>> - public void setManagedVersionMap( Map map )
>>>>> + public void setManagedVersionMap( Map<String, ArtifactVersion>
>>>>> map )
>>>>> {
>>>>> this.managedVersionMap = map;
>>>>> }
>>>>>
>>>>> - public Map getManagedVersionMap()
>>>>> + public Map<String, ArtifactVersion> getManagedVersionMap()
>>>>> {
>>>>> return this.managedVersionMap;
>>>>> }
>>>>> @@ -1627,12 +1630,12 @@
>>>>> return getId().hashCode();
>>>>> }
>>>>>
>>>>> - public List getBuildExtensions()
>>>>> + public List<Extension> getBuildExtensions()
>>>>> {
>>>>> Build build = getBuild();
>>>>> if ( build == null || build.getExtensions() == null )
>>>>> {
>>>>> - return Collections.EMPTY_LIST;
>>>>> + return Collections.<Extension> emptyList();
>>>>> }
>>>>> else
>>>>> {
>>>>> @@ -1644,7 +1647,7 @@
>>>>> * @todo the lazy initialisation of this makes me uneasy.
>>>>> * @return {@link Set} < {@link Artifact} >
>>>>> */
>>>>> - public Set createArtifacts( ArtifactFactory artifactFactory,
>>>>> String
>>>>> inheritedScope,
>>>>> + public Set<Artifact> createArtifacts( ArtifactFactory
>>>>> artifactFactory, String inheritedScope,
>>>>> ArtifactFilter dependencyFilter )
>>>>> throws InvalidDependencyVersionException
>>>>> {
>>>>> @@ -1674,12 +1677,12 @@
>>>>> return getModel().getProperties();
>>>>> }
>>>>>
>>>>> - public List getFilters()
>>>>> + public List<String> getFilters()
>>>>> {
>>>>> return getBuild().getFilters();
>>>>> }
>>>>>
>>>>> - public Map getProjectReferences()
>>>>> + public Map<String, MavenProject> getProjectReferences()
>>>>> {
>>>>> return projectReferences;
>>>>> }
>>>>> @@ -1705,22 +1708,22 @@
>>>>> this.model = model;
>>>>> }
>>>>>
>>>>> - protected void setAttachedArtifacts( List attachedArtifacts )
>>>>> + protected void setAttachedArtifacts( List<Artifact>
>>>>> attachedArtifacts )
>>>>> {
>>>>> this.attachedArtifacts = attachedArtifacts;
>>>>> }
>>>>>
>>>>> - protected void setCompileSourceRoots( List compileSourceRoots )
>>>>> + protected void setCompileSourceRoots( List<String>
>>>>> compileSourceRoots )
>>>>> {
>>>>> this.compileSourceRoots = compileSourceRoots;
>>>>> }
>>>>>
>>>>> - protected void setTestCompileSourceRoots( List
>>>>> testCompileSourceRoots )
>>>>> + protected void setTestCompileSourceRoots( List<String>
>>>>> testCompileSourceRoots )
>>>>> {
>>>>> this.testCompileSourceRoots = testCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - protected void setScriptSourceRoots( List scriptSourceRoots )
>>>>> + protected void setScriptSourceRoots( List<String>
>>>>> scriptSourceRoots
>>>>> )
>>>>> {
>>>>> this.scriptSourceRoots = scriptSourceRoots;
>>>>> }
>>>>> @@ -1737,16 +1740,16 @@
>>>>>
>>>>> public void resolveActiveArtifacts()
>>>>> {
>>>>> - Set depArtifacts = getDependencyArtifacts();
>>>>> + Set<Artifact> depArtifacts = getDependencyArtifacts();
>>>>> if ( depArtifacts == null )
>>>>> {
>>>>> return;
>>>>> }
>>>>>
>>>>> - Set updated = new LinkedHashSet( depArtifacts.size() );
>>>>> + Set<Artifact> updated = new LinkedHashSet<Artifact>(
>>>>> depArtifacts.size() );
>>>>> int updatedCount = 0;
>>>>>
>>>>> - for ( Iterator it = depArtifacts.iterator(); it.hasNext(); )
>>>>> + for ( Iterator<Artifact> it = depArtifacts.iterator();
>>>>> it.hasNext(); )
>>>>> {
>>>>> Artifact depArtifact = (Artifact) it.next();
>>>>> Artifact replaced = replaceWithActiveArtifact(
>>>>> depArtifact );
>>>>> @@ -1815,13 +1818,13 @@
>>>>> * @param requestedArtifact The artifact to resolve, must not be
>>>>> <code>null</code>.
>>>>> * @return The matching artifact or <code>null</code> if not found.
>>>>> */
>>>>> - private Artifact findMatchingArtifact( List artifacts, Artifact
>>>>> requestedArtifact )
>>>>> + private Artifact findMatchingArtifact( List<Artifact> artifacts,
>>>>> Artifact requestedArtifact )
>>>>> {
>>>>> if ( artifacts != null && !artifacts.isEmpty() )
>>>>> {
>>>>> // first try matching by dependency conflict id
>>>>> String requestedId =
>>>>> requestedArtifact.getDependencyConflictId();
>>>>> - for ( Iterator it = artifacts.iterator(); it.hasNext(); )
>>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>>> it.hasNext(); )
>>>>> {
>>>>> Artifact artifact = (Artifact) it.next();
>>>>> if ( requestedId.equals(
>>>>> artifact.getDependencyConflictId() ) )
>>>>> @@ -1832,7 +1835,7 @@
>>>>>
>>>>> // next try matching by repository conflict id
>>>>> requestedId = getRepositoryConflictId( requestedArtifact );
>>>>> - for ( Iterator it = artifacts.iterator(); it.hasNext(); )
>>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>>> it.hasNext(); )
>>>>> {
>>>>> Artifact artifact = (Artifact) it.next();
>>>>> if ( requestedId.equals( getRepositoryConflictId(
>>>>> artifact
>>>>> ) ) )
>>>>> @@ -1898,7 +1901,7 @@
>>>>> }
>>>>> }
>>>>>
>>>>> - private void addArtifactPath(Artifact a, List list) throws
>>>>> DependencyResolutionRequiredException
>>>>> + private void addArtifactPath(Artifact a, List<String> list)
>>>>> throws
>>>>> DependencyResolutionRequiredException
>>>>> {
>>>>> File file = a.getFile();
>>>>> if ( file == null )
>>>>> @@ -1954,21 +1957,21 @@
>>>>>
>>>>> private Build originalInterpolatedBuild;
>>>>>
>>>>> - private List dynamicCompileSourceRoots;
>>>>> + private List<String> dynamicCompileSourceRoots;
>>>>>
>>>>> - private List originalInterpolatedCompileSourceRoots;
>>>>> + private List<String> originalInterpolatedCompileSourceRoots;
>>>>>
>>>>> - private List dynamicTestCompileSourceRoots;
>>>>> + private List<String> dynamicTestCompileSourceRoots;
>>>>>
>>>>> - private List originalInterpolatedTestCompileSourceRoots;
>>>>> + private List<String> originalInterpolatedTestCompileSourceRoots;
>>>>>
>>>>> - private List dynamicScriptSourceRoots;
>>>>> + private List<String> dynamicScriptSourceRoots;
>>>>>
>>>>> - private List originalInterpolatedScriptSourceRoots;
>>>>> + private List<String> originalInterpolatedScriptSourceRoots;
>>>>>
>>>>> private boolean isConcrete = false;
>>>>>
>>>>> - public boolean isConcrete()
>>>>> + public boolean isConcrete()
>>>>> {
>>>>> return isConcrete;
>>>>> }
>>>>> @@ -1988,32 +1991,32 @@
>>>>> return originalInterpolatedBuild;
>>>>> }
>>>>>
>>>>> - public List getDynamicCompileSourceRoots()
>>>>> + public List<String> getDynamicCompileSourceRoots()
>>>>> {
>>>>> return dynamicCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - public List getOriginalInterpolatedCompileSourceRoots()
>>>>> + public List<String> getOriginalInterpolatedCompileSourceRoots()
>>>>> {
>>>>> return originalInterpolatedCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - public List getDynamicTestCompileSourceRoots()
>>>>> + public List<String> getDynamicTestCompileSourceRoots()
>>>>> {
>>>>> return dynamicTestCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - public List getOriginalInterpolatedTestCompileSourceRoots()
>>>>> + public List<String>
>>>>> getOriginalInterpolatedTestCompileSourceRoots()
>>>>> {
>>>>> return originalInterpolatedTestCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - public List getDynamicScriptSourceRoots()
>>>>> + public List<String> getDynamicScriptSourceRoots()
>>>>> {
>>>>> return dynamicScriptSourceRoots;
>>>>> }
>>>>>
>>>>> - public List getOriginalInterpolatedScriptSourceRoots()
>>>>> + public List<String> getOriginalInterpolatedScriptSourceRoots()
>>>>> {
>>>>> return originalInterpolatedScriptSourceRoots;
>>>>> }
>>>>> @@ -2034,19 +2037,19 @@
>>>>> originalInterpolatedBuild = null;
>>>>> }
>>>>>
>>>>> - public void preserveCompileSourceRoots( List
>>>>> originalInterpolatedCompileSourceRoots )
>>>>> + public void preserveCompileSourceRoots( List<String>
>>>>> originalInterpolatedCompileSourceRoots )
>>>>> {
>>>>> dynamicCompileSourceRoots = getCompileSourceRoots();
>>>>> this.originalInterpolatedCompileSourceRoots =
>>>>> originalInterpolatedCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - public void preserveTestCompileSourceRoots( List
>>>>> originalInterpolatedTestCompileSourceRoots )
>>>>> + public void preserveTestCompileSourceRoots( List<String>
>>>>> originalInterpolatedTestCompileSourceRoots )
>>>>> {
>>>>> dynamicTestCompileSourceRoots = getTestCompileSourceRoots();
>>>>> this.originalInterpolatedTestCompileSourceRoots =
>>>>> originalInterpolatedTestCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - public void preserveScriptSourceRoots( List
>>>>> originalInterpolatedScriptSourceRoots )
>>>>> + public void preserveScriptSourceRoots( List<String>
>>>>> originalInterpolatedScriptSourceRoots )
>>>>> {
>>>>> dynamicScriptSourceRoots = getScriptSourceRoots();
>>>>> this.originalInterpolatedScriptSourceRoots =
>>>>> originalInterpolatedScriptSourceRoots;
>>>>> @@ -2071,32 +2074,32 @@
>>>>> this.originalInterpolatedBuild = originalInterpolatedBuild;
>>>>> }
>>>>>
>>>>> - protected void setDynamicCompileSourceRoots( List
>>>>> dynamicCompileSourceRoots )
>>>>> + protected void setDynamicCompileSourceRoots( List<String>
>>>>> dynamicCompileSourceRoots )
>>>>> {
>>>>> this.dynamicCompileSourceRoots = dynamicCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - protected void setOriginalInterpolatedCompileSourceRoots( List
>>>>> originalInterpolatedCompileSourceRoots )
>>>>> + protected void setOriginalInterpolatedCompileSourceRoots(
>>>>> List<String> originalInterpolatedCompileSourceRoots )
>>>>> {
>>>>> this.originalInterpolatedCompileSourceRoots =
>>>>> originalInterpolatedCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - protected void setDynamicTestCompileSourceRoots( List
>>>>> dynamicTestCompileSourceRoots )
>>>>> + protected void setDynamicTestCompileSourceRoots( List<String>
>>>>> dynamicTestCompileSourceRoots )
>>>>> {
>>>>> this.dynamicTestCompileSourceRoots =
>>>>> dynamicTestCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - protected void setOriginalInterpolatedTestCompileSourceRoots(
>>>>> List
>>>>> originalInterpolatedTestCompileSourceRoots )
>>>>> + protected void setOriginalInterpolatedTestCompileSourceRoots(
>>>>> List<String> originalInterpolatedTestCompileSourceRoots )
>>>>> {
>>>>> this.originalInterpolatedTestCompileSourceRoots =
>>>>> originalInterpolatedTestCompileSourceRoots;
>>>>> }
>>>>>
>>>>> - protected void setDynamicScriptSourceRoots( List
>>>>> dynamicScriptSourceRoots )
>>>>> + protected void setDynamicScriptSourceRoots( List<String>
>>>>> dynamicScriptSourceRoots )
>>>>> {
>>>>> this.dynamicScriptSourceRoots = dynamicScriptSourceRoots;
>>>>> }
>>>>>
>>>>> - protected void setOriginalInterpolatedScriptSourceRoots( List
>>>>> originalInterpolatedScriptSourceRoots )
>>>>> + protected void setOriginalInterpolatedScriptSourceRoots(
>>>>> List<String> originalInterpolatedScriptSourceRoots )
>>>>> {
>>>>> this.originalInterpolatedScriptSourceRoots =
>>>>> originalInterpolatedScriptSourceRoots;
>>>>> }
>>>>> @@ -2114,7 +2117,7 @@
>>>>> if ( p != null )
>>>>> {
>>>>> preservedProperties = new Properties();
>>>>> - for( Enumeration e = p.propertyNames();
>>>>> e.hasMoreElements();
>>>>> )
>>>>> + for( Enumeration<?> e = p.propertyNames();
>>>>> e.hasMoreElements(); )
>>>>> {
>>>>> String key = (String) e.nextElement();
>>>>> preservedProperties.setProperty( key, p.getProperty(
>>>>> key )
>>>>> );
>>>>>
>>>>> Modified:
>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
>>>>>
>>>>> URL:
>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>
>>>>> ==============================================================================
>>>>>
>>>>>
>>>>> ---
>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
>>>>>
>>>>> (original)
>>>>> +++
>>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml
>>>>> Fri
>>>>> May 1 08:04:48 2009
>>>>> @@ -47,6 +47,7 @@
>>>>> <models>
>>>>> <model>src/main/mdo/metadata.mdo</model>
>>>>> </models>
>>>>> + <useJava5>true</useJava5>
>>>>> </configuration>
>>>>> </plugin>
>>>>> </plugins>
>>>>>
>>>>> Modified: maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>>> URL:
>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-settings/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>
>>>>> ==============================================================================
>>>>>
>>>>>
>>>>> --- maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>>> (original)
>>>>> +++ maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>>> Fri May
>>>>> 1 08:04:48 2009
>>>>> @@ -58,6 +58,7 @@
>>>>> <models>
>>>>> <model>src/main/mdo/settings.mdo</model>
>>>>> </models>
>>>>> + <useJava5>true</useJava5>
>>>>> </configuration>
>>>>> </plugin>
>>>>> </plugins>
>>>>>
>>>>> Modified:
>>>>> maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>>> URL:
>>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>>>
>>>>> ==============================================================================
>>>>>
>>>>>
>>>>> --- maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>>> (original)
>>>>> +++ maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>>> Fri May
>>>>> 1 08:04:48 2009
>>>>> @@ -66,6 +66,7 @@
>>>>> <models>
>>>>> <model>src/main/mdo/toolchains.mdo</model>
>>>>> </models>
>>>>> + <useJava5>true</useJava5>
>>>>> </configuration>
>>>>> </plugin>
>>>>> <plugin>
>>>>>
>>>>>
>>>>>
>>>> Thanks,
>>>>
>>>> Jason
>>>>
>>>> ----------------------------------------------------------
>>>> Jason van Zyl
>>>> Founder, Apache Maven
>>>> http://twitter.com/jvanzyl
>>>> http://twitter.com/SonatypeNexus
>>>> http://twitter.com/SonatypeM2E
>>>> ----------------------------------------------------------
>>>>
>>>> We all have problems. How we deal with them is a measure of our worth.
>>>>
>>>> -- Unknown
>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>>>> For additional commands, e-mail: dev-help@maven.apache.org
>>>>
>>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>>> For additional commands, e-mail: dev-help@maven.apache.org
>>>
>>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: svn commit: r770570 - in /maven/components/branches/maven-2.2.x: maven-artifact/src/main/java/org/apache/maven/artifact/ maven-model/ maven-profile/ maven-project/src/main/java/org/apache/maven/project/ maven-repository-metadata/ maven-settings/
Posted by BRIAN FOX <br...@infinity.nu>.
In general, no. I just wanted to draw John's attention to the changes
since I know he was preparing to cut an RC. I am suspicious of all
changes when a release is drawing near.
On May 2, 2009, at 2:44 AM, nicolas de loof wrote:
> Do we prefer plugin developers to use List<?> and get
> ClassCastExceptions at
> runtime or sue type-safe collections to help them create stronger
> code ?Anyway,
> few plugins allready use Java5. Java 1.4 based one will not be
> broken as the
> generics signature is only a compile-time check.
>
>
> 2009/5/1 Brian Fox <br...@infinity.nu>
>
>> I'm not sure if this is in scope of what John is trying to do wrt
>> to 2.2.
>>
>>
>> Jason van Zyl wrote:
>>
>>> I don't believe anyone actually agreed to this yet. Are you sure
>>> this is
>>> not going to cause problems for users?
>>>
>>> On 1-May-09, at 1:04 AM, nicolas@apache.org wrote:
>>>
>>> Author: nicolas
>>>> Date: Fri May 1 08:04:48 2009
>>>> New Revision: 770570
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=770570&view=rev
>>>> Log:
>>>> use java5 syntax.
>>>> -> set modello to generate java5 syntax
>>>> -> MavenProject updated to use generics on collections & methods
>>>> for
>>>> better type safety
>>>>
>>>> Modified:
>>>>
>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/
>>>> java/org/apache/maven/artifact/ArtifactUtils.java
>>>>
>>>> maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>> maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>>
>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/
>>>> org/apache/maven/project/MavenProject.java
>>>>
>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/
>>>> pom.xml
>>>> maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>> maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>>
>>>> Modified:
>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/
>>>> java/org/apache/maven/artifact/ArtifactUtils.java
>>>>
>>>> URL:
>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-artifact/src/main/java/org/apache/maven/artifact/ArtifactUtils.java?rev=770570&r1=770569&r2=770570&view=diff
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> ===================================================================
>>>>
>>>> ---
>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/
>>>> java/org/apache/maven/artifact/ArtifactUtils.java
>>>> (original)
>>>> +++
>>>> maven/components/branches/maven-2.2.x/maven-artifact/src/main/
>>>> java/org/apache/maven/artifact/ArtifactUtils.java
>>>> Fri May 1 08:04:48 2009
>>>> @@ -86,13 +86,13 @@
>>>> baseVersion;
>>>> }
>>>>
>>>> - public static Map artifactMapByVersionlessId( Collection
>>>> artifacts )
>>>> + public static Map<String, Artifact>
>>>> artifactMapByVersionlessId(
>>>> Collection<Artifact> artifacts )
>>>> {
>>>> - Map artifactMap = new LinkedHashMap();
>>>> + Map<String, Artifact> artifactMap = new
>>>> LinkedHashMap<String,
>>>> Artifact>();
>>>>
>>>> if ( artifacts != null )
>>>> {
>>>> - for ( Iterator it = artifacts.iterator();
>>>> it.hasNext(); )
>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>> it.hasNext(); )
>>>> {
>>>> Artifact artifact = (Artifact) it.next();
>>>>
>>>>
>>>> Modified: maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>> URL:
>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-model/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> ===================================================================
>>>>
>>>> --- maven/components/branches/maven-2.2.x/maven-model/pom.xml
>>>> (original)
>>>> +++ maven/components/branches/maven-2.2.x/maven-model/pom.xml Fri
>>>> May 1
>>>> 08:04:48 2009
>>>> @@ -47,6 +47,7 @@
>>>> <models>
>>>> <model>src/main/mdo/maven.mdo</model>
>>>> </models>
>>>> + <useJava5>true</useJava5>
>>>> </configuration>
>>>> </plugin>
>>>> <plugin>
>>>>
>>>> Modified: maven/components/branches/maven-2.2.x/maven-profile/
>>>> pom.xml
>>>> URL:
>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-profile/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> ===================================================================
>>>>
>>>> --- maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>> (original)
>>>> +++ maven/components/branches/maven-2.2.x/maven-profile/pom.xml
>>>> Fri May
>>>> 1 08:04:48 2009
>>>> @@ -54,6 +54,7 @@
>>>> <models>
>>>> <model>src/main/mdo/profiles.mdo</model>
>>>> </models>
>>>> + <useJava5>true</useJava5>
>>>> </configuration>
>>>> </plugin>
>>>> </plugins>
>>>>
>>>> Modified:
>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/
>>>> org/apache/maven/project/MavenProject.java
>>>>
>>>> URL:
>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-project/src/main/java/org/apache/maven/project/MavenProject.java?rev=770570&r1=770569&r2=770570&view=diff
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> ===================================================================
>>>>
>>>> ---
>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/
>>>> org/apache/maven/project/MavenProject.java
>>>> (original)
>>>> +++
>>>> maven/components/branches/maven-2.2.x/maven-project/src/main/java/
>>>> org/apache/maven/project/MavenProject.java
>>>> Fri May 1 08:04:48 2009
>>>> @@ -36,10 +36,11 @@
>>>> import org.apache.maven.artifact.Artifact;
>>>> import org.apache.maven.artifact.ArtifactUtils;
>>>> import
>>>> org.apache.maven.artifact.DependencyResolutionRequiredException;
>>>> -import org.apache.maven.artifact.versioning.ManagedVersionMap;
>>>> import org.apache.maven.artifact.factory.ArtifactFactory;
>>>> import org.apache.maven.artifact.repository.ArtifactRepository;
>>>> import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
>>>> +import org.apache.maven.artifact.versioning.ArtifactVersion;
>>>> +import org.apache.maven.artifact.versioning.ManagedVersionMap;
>>>> import org.apache.maven.model.Build;
>>>> import org.apache.maven.model.CiManagement;
>>>> import org.apache.maven.model.Contributor;
>>>> @@ -47,6 +48,7 @@
>>>> import org.apache.maven.model.DependencyManagement;
>>>> import org.apache.maven.model.Developer;
>>>> import org.apache.maven.model.DistributionManagement;
>>>> +import org.apache.maven.model.Extension;
>>>> import org.apache.maven.model.IssueManagement;
>>>> import org.apache.maven.model.License;
>>>> import org.apache.maven.model.MailingList;
>>>> @@ -59,6 +61,7 @@
>>>> import org.apache.maven.model.ReportPlugin;
>>>> import org.apache.maven.model.ReportSet;
>>>> import org.apache.maven.model.Reporting;
>>>> +import org.apache.maven.model.Repository;
>>>> import org.apache.maven.model.Resource;
>>>> import org.apache.maven.model.Scm;
>>>> import org.apache.maven.model.io.xpp3.MavenXpp3Writer;
>>>> @@ -96,60 +99,60 @@
>>>>
>>>> private File file;
>>>>
>>>> - private Set artifacts;
>>>> + private Set<Artifact> artifacts;
>>>>
>>>> private Artifact parentArtifact;
>>>>
>>>> - private Set pluginArtifacts;
>>>> + private Set<Artifact> pluginArtifacts;
>>>>
>>>> - private List remoteArtifactRepositories;
>>>> + private List<ArtifactRepository> remoteArtifactRepositories;
>>>>
>>>> - private List collectedProjects = Collections.EMPTY_LIST;
>>>> + private List<MavenProject> collectedProjects =
>>>> Collections.<MavenProject> emptyList();
>>>>
>>>> - private List attachedArtifacts;
>>>> + private List<Artifact> attachedArtifacts;
>>>>
>>>> private MavenProject executionProject;
>>>>
>>>> - private List compileSourceRoots = new ArrayList();
>>>> + private List<String> compileSourceRoots = new
>>>> ArrayList<String>();
>>>>
>>>> - private List testCompileSourceRoots = new ArrayList();
>>>> + private List<String> testCompileSourceRoots = new
>>>> ArrayList<String>();
>>>>
>>>> - private List scriptSourceRoots = new ArrayList();
>>>> + private List<String> scriptSourceRoots = new
>>>> ArrayList<String>();
>>>>
>>>> - private List pluginArtifactRepositories;
>>>> + private List<ArtifactRepository> pluginArtifactRepositories;
>>>>
>>>> private ArtifactRepository releaseArtifactRepository;
>>>>
>>>> private ArtifactRepository snapshotArtifactRepository;
>>>>
>>>> - private List activeProfiles = new ArrayList();
>>>> + private List<String> activeProfiles = new ArrayList<String>();
>>>>
>>>> - private Set dependencyArtifacts;
>>>> + private Set<Artifact> dependencyArtifacts;
>>>>
>>>> private Artifact artifact;
>>>>
>>>> // calculated.
>>>> - private Map artifactMap;
>>>> + private Map<String, Artifact> artifactMap;
>>>>
>>>> private Model originalModel;
>>>>
>>>> - private Map pluginArtifactMap;
>>>> + private Map<String, Artifact> pluginArtifactMap;
>>>>
>>>> - private Set reportArtifacts;
>>>> + private Set<Artifact> reportArtifacts;
>>>>
>>>> - private Map reportArtifactMap;
>>>> + private Map<String, Artifact> reportArtifactMap;
>>>>
>>>> - private Set extensionArtifacts;
>>>> + private Set<Artifact> extensionArtifacts;
>>>>
>>>> - private Map extensionArtifactMap;
>>>> + private Map<String, Artifact> extensionArtifactMap;
>>>>
>>>> - private Map managedVersionMap;
>>>> + private Map<String, ArtifactVersion> managedVersionMap;
>>>>
>>>> - private Map projectReferences = new HashMap();
>>>> + private Map<String, MavenProject> projectReferences = new
>>>> HashMap<String, MavenProject>();
>>>>
>>>> private boolean executionRoot;
>>>>
>>>> - private Map moduleAdjustments;
>>>> + private Map<String, String> moduleAdjustments;
>>>>
>>>> private File basedir;
>>>>
>>>> @@ -245,23 +248,23 @@
>>>> if ( project.getAttachedArtifacts() != null )
>>>> {
>>>> // clone properties modifyable by plugins in a forked
>>>> lifecycle
>>>> - setAttachedArtifacts( new ArrayList(
>>>> project.getAttachedArtifacts() ) );
>>>> + setAttachedArtifacts( new ArrayList<Artifact>(
>>>> project.getAttachedArtifacts() ) );
>>>> }
>>>>
>>>> if ( project.getCompileSourceRoots() != null )
>>>> {
>>>> // clone source roots
>>>> - setCompileSourceRoots( ( new ArrayList(
>>>> project.getCompileSourceRoots() ) ) );
>>>> + setCompileSourceRoots( ( new ArrayList<String>(
>>>> project.getCompileSourceRoots() ) ) );
>>>> }
>>>>
>>>> if ( project.getTestCompileSourceRoots() != null )
>>>> {
>>>> - setTestCompileSourceRoots( ( new ArrayList(
>>>> project.getTestCompileSourceRoots() ) ) );
>>>> + setTestCompileSourceRoots( ( new ArrayList<String>(
>>>> project.getTestCompileSourceRoots() ) ) );
>>>> }
>>>>
>>>> if ( project.getScriptSourceRoots() != null )
>>>> {
>>>> - setScriptSourceRoots( ( new ArrayList(
>>>> project.getScriptSourceRoots() ) ) );
>>>> + setScriptSourceRoots( ( new ArrayList<String>(
>>>> project.getScriptSourceRoots() ) ) );
>>>> }
>>>>
>>>> setModel( ( ModelUtils.cloneModel( project.getModel() ) ) );
>>>> @@ -298,25 +301,25 @@
>>>> setDynamicBuild( ModelUtils.cloneBuild(
>>>> project.getDynamicBuild() ) );
>>>> setOriginalInterpolatedBuild( ModelUtils.cloneBuild(
>>>> project.getOriginalInterpolatedBuild() ) );
>>>>
>>>> - List dynamicRoots =
>>>> project.getDynamicCompileSourceRoots();
>>>> + List<String> dynamicRoots =
>>>> project.getDynamicCompileSourceRoots();
>>>> if ( dynamicRoots != null )
>>>> {
>>>> - setDynamicCompileSourceRoots( new ArrayList(
>>>> dynamicRoots ) );
>>>> - setOriginalInterpolatedCompileSourceRoots( new
>>>> ArrayList( project.getOriginalInterpolatedCompileSourceRoots() ) );
>>>> + setDynamicCompileSourceRoots( new
>>>> ArrayList<String>(
>>>> dynamicRoots ) );
>>>> + setOriginalInterpolatedCompileSourceRoots( new
>>>> ArrayList
>>>> <String>( project.getOriginalInterpolatedCompileSourceRoots() ) );
>>>> }
>>>>
>>>> dynamicRoots =
>>>> project.getDynamicTestCompileSourceRoots();
>>>> if ( dynamicRoots != null )
>>>> {
>>>> - setDynamicTestCompileSourceRoots( new ArrayList(
>>>> dynamicRoots ) );
>>>> - setOriginalInterpolatedTestCompileSourceRoots( new
>>>> ArrayList
>>>> ( project.getOriginalInterpolatedTestCompileSourceRoots() ) );
>>>> + setDynamicTestCompileSourceRoots( new
>>>> ArrayList<String>(
>>>> dynamicRoots ) );
>>>> + setOriginalInterpolatedTestCompileSourceRoots( new
>>>> ArrayList
>>>> <String>( project.getOriginalInterpolatedTestCompileSourceRoots() )
>>>> );
>>>> }
>>>>
>>>> dynamicRoots = project.getDynamicScriptSourceRoots();
>>>> if ( dynamicRoots != null )
>>>> {
>>>> - setDynamicScriptSourceRoots( new
>>>> ArrayList( dynamicRoots
>>>> ) );
>>>> - setOriginalInterpolatedScriptSourceRoots( new
>>>> ArrayList(
>>>> project.getOriginalInterpolatedScriptSourceRoots() ) );
>>>> + setDynamicScriptSourceRoots( new
>>>> ArrayList<String>(
>>>> dynamicRoots ) );
>>>> + setOriginalInterpolatedScriptSourceRoots( new
>>>> ArrayList
>>>> <String>( project.getOriginalInterpolatedScriptSourceRoots() ) );
>>>> }
>>>> }
>>>>
>>>> @@ -347,12 +350,12 @@
>>>>
>>>> if ( moduleAdjustments == null )
>>>> {
>>>> - moduleAdjustments = new HashMap();
>>>> + moduleAdjustments = new HashMap<String, String>();
>>>>
>>>> - List modules = getModules();
>>>> + List<String> modules = getModules();
>>>> if ( modules != null )
>>>> {
>>>> - for ( Iterator it = modules.iterator();
>>>> it.hasNext(); )
>>>> + for ( Iterator<String> it = modules.iterator();
>>>> it.hasNext(); )
>>>> {
>>>> String modulePath = (String) it.next();
>>>> String moduleName = modulePath;
>>>> @@ -415,12 +418,12 @@
>>>> this.parent = parent;
>>>> }
>>>>
>>>> - public void setRemoteArtifactRepositories( List
>>>> remoteArtifactRepositories )
>>>> + public void
>>>> setRemoteArtifactRepositories( List<ArtifactRepository>
>>>> remoteArtifactRepositories )
>>>> {
>>>> this.remoteArtifactRepositories = remoteArtifactRepositories;
>>>> }
>>>>
>>>> - public List getRemoteArtifactRepositories()
>>>> + public List<ArtifactRepository>
>>>> getRemoteArtifactRepositories()
>>>> {
>>>> return remoteArtifactRepositories;
>>>> }
>>>> @@ -460,12 +463,12 @@
>>>> return basedir;
>>>> }
>>>>
>>>> - public void setDependencies( List dependencies )
>>>> + public void setDependencies( List<Dependency> dependencies )
>>>> {
>>>> getModel().setDependencies( dependencies );
>>>> }
>>>>
>>>> - public List getDependencies()
>>>> + public List<Dependency> getDependencies()
>>>> {
>>>> return getModel().getDependencies();
>>>> }
>>>> @@ -524,29 +527,29 @@
>>>> }
>>>> }
>>>>
>>>> - public List getCompileSourceRoots()
>>>> + public List<String> getCompileSourceRoots()
>>>> {
>>>> return compileSourceRoots;
>>>> }
>>>>
>>>> - public List getScriptSourceRoots()
>>>> + public List<String> getScriptSourceRoots()
>>>> {
>>>> return scriptSourceRoots;
>>>> }
>>>>
>>>> - public List getTestCompileSourceRoots()
>>>> + public List<String> getTestCompileSourceRoots()
>>>> {
>>>> return testCompileSourceRoots;
>>>> }
>>>>
>>>> - public List getCompileClasspathElements()
>>>> + public List<String> getCompileClasspathElements()
>>>> throws DependencyResolutionRequiredException
>>>> {
>>>> - List list = new ArrayList( getArtifacts().size() );
>>>> + List<String> list = new
>>>> ArrayList<String>( getArtifacts().size()
>>>> );
>>>>
>>>> list.add( getBuild().getOutputDirectory() );
>>>>
>>>> - for ( Iterator i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> + for ( Iterator<?> i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -563,11 +566,11 @@
>>>> return list;
>>>> }
>>>>
>>>> - public List getCompileArtifacts()
>>>> + public List<Artifact> getCompileArtifacts()
>>>> {
>>>> - List list = new ArrayList( getArtifacts().size() );
>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>> getArtifacts().size() );
>>>>
>>>> - for ( Iterator i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -585,18 +588,18 @@
>>>> return list;
>>>> }
>>>>
>>>> - public List getCompileDependencies()
>>>> + public List<Dependency> getCompileDependencies()
>>>> {
>>>> - Set artifacts = getArtifacts();
>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>
>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>> {
>>>> - return Collections.EMPTY_LIST;
>>>> + return Collections.<Dependency> emptyList();
>>>> }
>>>>
>>>> - List list = new ArrayList( artifacts.size() );
>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>> artifacts.size() );
>>>>
>>>> - for ( Iterator i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -619,16 +622,16 @@
>>>> return list;
>>>> }
>>>>
>>>> - public List getTestClasspathElements()
>>>> + public List<String> getTestClasspathElements()
>>>> throws DependencyResolutionRequiredException
>>>> {
>>>> - List list = new ArrayList( getArtifacts().size() + 1 );
>>>> + List<String> list = new
>>>> ArrayList<String>( getArtifacts().size()
>>>> + 1 );
>>>>
>>>> list.add( getBuild().getTestOutputDirectory() );
>>>>
>>>> list.add( getBuild().getOutputDirectory() );
>>>>
>>>> - for ( Iterator i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -652,11 +655,11 @@
>>>> return list;
>>>> }
>>>>
>>>> - public List getTestArtifacts()
>>>> + public List<Artifact> getTestArtifacts()
>>>> {
>>>> - List list = new ArrayList( getArtifacts().size() );
>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>> getArtifacts().size() );
>>>>
>>>> - for ( Iterator i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -678,18 +681,18 @@
>>>> return list;
>>>> }
>>>>
>>>> - public List getTestDependencies()
>>>> + public List<Dependency> getTestDependencies()
>>>> {
>>>> - Set artifacts = getArtifacts();
>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>
>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>> {
>>>> - return Collections.EMPTY_LIST;
>>>> + return Collections.emptyList();
>>>> }
>>>>
>>>> - List list = new ArrayList( artifacts.size() );
>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>> artifacts.size() );
>>>>
>>>> - for ( Iterator i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -715,14 +718,14 @@
>>>> return list;
>>>> }
>>>>
>>>> - public List getRuntimeClasspathElements()
>>>> + public List<String> getRuntimeClasspathElements()
>>>> throws DependencyResolutionRequiredException
>>>> {
>>>> - List list = new ArrayList( getArtifacts().size() + 1 );
>>>> + List<String> list = new
>>>> ArrayList<String>( getArtifacts().size()
>>>> + 1 );
>>>>
>>>> list.add( getBuild().getOutputDirectory() );
>>>>
>>>> - for ( Iterator i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -743,11 +746,11 @@
>>>> return list;
>>>> }
>>>>
>>>> - public List getRuntimeArtifacts()
>>>> + public List<Artifact> getRuntimeArtifacts()
>>>> {
>>>> - List list = new ArrayList( getArtifacts().size() );
>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>> getArtifacts().size() );
>>>>
>>>> - for ( Iterator i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -764,18 +767,18 @@
>>>> return list;
>>>> }
>>>>
>>>> - public List getRuntimeDependencies()
>>>> + public List<Dependency> getRuntimeDependencies()
>>>> {
>>>> - Set artifacts = getArtifacts();
>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>
>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>> {
>>>> - return Collections.EMPTY_LIST;
>>>> + return Collections.<Dependency> emptyList();
>>>> }
>>>>
>>>> - List list = new ArrayList( artifacts.size() );
>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>> artifacts.size() );
>>>>
>>>> - for ( Iterator i = artifacts.iterator(); i.hasNext(); )
>>>> + for ( Iterator<Artifact> i = artifacts.iterator();
>>>> i.hasNext();
>>>> )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -797,14 +800,14 @@
>>>> return list;
>>>> }
>>>>
>>>> - public List getSystemClasspathElements()
>>>> + public List<String> getSystemClasspathElements()
>>>> throws DependencyResolutionRequiredException
>>>> {
>>>> - List list = new ArrayList( getArtifacts().size() );
>>>> + List<String> list = new
>>>> ArrayList<String>( getArtifacts().size()
>>>> );
>>>>
>>>> list.add( getBuild().getOutputDirectory() );
>>>>
>>>> - for ( Iterator i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -820,11 +823,11 @@
>>>> return list;
>>>> }
>>>>
>>>> - public List getSystemArtifacts()
>>>> + public List<Artifact> getSystemArtifacts()
>>>> {
>>>> - List list = new ArrayList( getArtifacts().size() );
>>>> + List<Artifact> list = new ArrayList<Artifact>(
>>>> getArtifacts().size() );
>>>>
>>>> - for ( Iterator i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -841,18 +844,18 @@
>>>> return list;
>>>> }
>>>>
>>>> - public List getSystemDependencies()
>>>> + public List<Dependency> getSystemDependencies()
>>>> {
>>>> - Set artifacts = getArtifacts();
>>>> + Set<Artifact> artifacts = getArtifacts();
>>>>
>>>> if ( artifacts == null || artifacts.isEmpty() )
>>>> {
>>>> - return Collections.EMPTY_LIST;
>>>> + return Collections.<Dependency> emptyList();
>>>> }
>>>>
>>>> - List list = new ArrayList( artifacts.size() );
>>>> + List<Dependency> list = new ArrayList<Dependency>(
>>>> artifacts.size() );
>>>>
>>>> - for ( Iterator i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> + for ( Iterator<Artifact> i = getArtifacts().iterator();
>>>> i.hasNext(); )
>>>> {
>>>> Artifact a = (Artifact) i.next();
>>>>
>>>> @@ -1050,12 +1053,12 @@
>>>> return getModel().getScm();
>>>> }
>>>>
>>>> - public void setMailingLists( List mailingLists )
>>>> + public void setMailingLists( List<MailingList> mailingLists )
>>>> {
>>>> getModel().setMailingLists( mailingLists );
>>>> }
>>>>
>>>> - public List getMailingLists()
>>>> + public List<?> getMailingLists()
>>>> {
>>>> return getModel().getMailingLists();
>>>> }
>>>> @@ -1065,12 +1068,12 @@
>>>> getModel().addMailingList( mailingList );
>>>> }
>>>>
>>>> - public void setDevelopers( List developers )
>>>> + public void setDevelopers( List<Developer> developers )
>>>> {
>>>> getModel().setDevelopers( developers );
>>>> }
>>>>
>>>> - public List getDevelopers()
>>>> + public List<Developer> getDevelopers()
>>>> {
>>>> return getModel().getDevelopers();
>>>> }
>>>> @@ -1080,12 +1083,12 @@
>>>> getModel().addDeveloper( developer );
>>>> }
>>>>
>>>> - public void setContributors( List contributors )
>>>> + public void setContributors( List<Contributor> contributors )
>>>> {
>>>> getModel().setContributors( contributors );
>>>> }
>>>>
>>>> - public List getContributors()
>>>> + public List<Contributor> getContributors()
>>>> {
>>>> return getModel().getContributors();
>>>> }
>>>> @@ -1105,12 +1108,12 @@
>>>> return getModelBuild();
>>>> }
>>>>
>>>> - public List getResources()
>>>> + public List<Resource> getResources()
>>>> {
>>>> return getBuild().getResources();
>>>> }
>>>>
>>>> - public List getTestResources()
>>>> + public List<Resource> getTestResources()
>>>> {
>>>> return getBuild().getTestResources();
>>>> }
>>>> @@ -1135,12 +1138,12 @@
>>>> return getModel().getReporting();
>>>> }
>>>>
>>>> - public void setLicenses( List licenses )
>>>> + public void setLicenses( List<License> licenses )
>>>> {
>>>> getModel().setLicenses( licenses );
>>>> }
>>>>
>>>> - public List getLicenses()
>>>> + public List<License> getLicenses()
>>>> {
>>>> return getModel().getLicenses();
>>>> }
>>>> @@ -1150,7 +1153,7 @@
>>>> getModel().addLicense( license );
>>>> }
>>>>
>>>> - public void setArtifacts( Set artifacts )
>>>> + public void setArtifacts( Set<Artifact> artifacts )
>>>> {
>>>> this.artifacts = artifacts;
>>>>
>>>> @@ -1165,12 +1168,12 @@
>>>> * @return {@link Set} < {@link Artifact} >
>>>> * @see #getDependencyArtifacts() to get only direct dependencies
>>>> */
>>>> - public Set getArtifacts()
>>>> + public Set<Artifact> getArtifacts()
>>>> {
>>>> - return artifacts == null ? Collections.EMPTY_SET :
>>>> artifacts;
>>>> + return artifacts == null ? Collections.<Artifact>
>>>> emptySet() :
>>>> artifacts;
>>>> }
>>>>
>>>> - public Map getArtifactMap()
>>>> + public Map<String, Artifact> getArtifactMap()
>>>> {
>>>> if ( artifactMap == null )
>>>> {
>>>> @@ -1180,19 +1183,19 @@
>>>> return artifactMap;
>>>> }
>>>>
>>>> - public void setPluginArtifacts( Set pluginArtifacts )
>>>> + public void setPluginArtifacts( Set<Artifact>
>>>> pluginArtifacts )
>>>> {
>>>> this.pluginArtifacts = pluginArtifacts;
>>>>
>>>> this.pluginArtifactMap = null;
>>>> }
>>>>
>>>> - public Set getPluginArtifacts()
>>>> + public Set<Artifact> getPluginArtifacts()
>>>> {
>>>> return pluginArtifacts;
>>>> }
>>>>
>>>> - public Map getPluginArtifactMap()
>>>> + public Map<String, Artifact> getPluginArtifactMap()
>>>> {
>>>> if ( pluginArtifactMap == null )
>>>> {
>>>> @@ -1202,19 +1205,19 @@
>>>> return pluginArtifactMap;
>>>> }
>>>>
>>>> - public void setReportArtifacts( Set reportArtifacts )
>>>> + public void setReportArtifacts( Set<Artifact>
>>>> reportArtifacts )
>>>> {
>>>> this.reportArtifacts = reportArtifacts;
>>>>
>>>> this.reportArtifactMap = null;
>>>> }
>>>>
>>>> - public Set getReportArtifacts()
>>>> + public Set<Artifact> getReportArtifacts()
>>>> {
>>>> return reportArtifacts;
>>>> }
>>>>
>>>> - public Map getReportArtifactMap()
>>>> + public Map<String, Artifact> getReportArtifactMap()
>>>> {
>>>> if ( reportArtifactMap == null )
>>>> {
>>>> @@ -1224,19 +1227,19 @@
>>>> return reportArtifactMap;
>>>> }
>>>>
>>>> - public void setExtensionArtifacts( Set extensionArtifacts )
>>>> + public void setExtensionArtifacts( Set<Artifact>
>>>> extensionArtifacts
>>>> )
>>>> {
>>>> this.extensionArtifacts = extensionArtifacts;
>>>>
>>>> this.extensionArtifactMap = null;
>>>> }
>>>>
>>>> - public Set getExtensionArtifacts()
>>>> + public Set<Artifact> getExtensionArtifacts()
>>>> {
>>>> return this.extensionArtifacts;
>>>> }
>>>>
>>>> - public Map getExtensionArtifactMap()
>>>> + public Map<String, Artifact> getExtensionArtifactMap()
>>>> {
>>>> if ( extensionArtifactMap == null )
>>>> {
>>>> @@ -1256,7 +1259,7 @@
>>>> return parentArtifact;
>>>> }
>>>>
>>>> - public List getRepositories()
>>>> + public List<Repository> getRepositories()
>>>> {
>>>> return getModel().getRepositories();
>>>> }
>>>> @@ -1265,7 +1268,7 @@
>>>> // Plugins
>>>> //
>>>> ----------------------------------------------------------------------
>>>>
>>>> - public List getReportPlugins()
>>>> + public List<ReportPlugin> getReportPlugins()
>>>> {
>>>> if ( getModel().getReporting() == null )
>>>> {
>>>> @@ -1275,7 +1278,7 @@
>>>>
>>>> }
>>>>
>>>> - public List getBuildPlugins()
>>>> + public List<Plugin> getBuildPlugins()
>>>> {
>>>> if ( getModel().getBuild() == null )
>>>> {
>>>> @@ -1284,7 +1287,7 @@
>>>> return getModel().getBuild().getPlugins();
>>>> }
>>>>
>>>> - public List getModules()
>>>> + public List<String> getModules()
>>>> {
>>>> return getModel().getModules();
>>>> }
>>>> @@ -1335,7 +1338,7 @@
>>>>
>>>> if ( pm != null )
>>>> {
>>>> - Map pmByKey = pm.getPluginsAsMap();
>>>> + Map<String, Plugin> pmByKey = pm.getPluginsAsMap();
>>>>
>>>> String pluginKey = plugin.getKey();
>>>>
>>>> @@ -1348,17 +1351,17 @@
>>>> }
>>>> }
>>>>
>>>> - public List getCollectedProjects()
>>>> + public List<MavenProject> getCollectedProjects()
>>>> {
>>>> return collectedProjects;
>>>> }
>>>>
>>>> - public void setCollectedProjects( List collectedProjects )
>>>> + public void setCollectedProjects( List<MavenProject>
>>>> collectedProjects )
>>>> {
>>>> this.collectedProjects = collectedProjects;
>>>> }
>>>>
>>>> - public void setPluginArtifactRepositories( List
>>>> pluginArtifactRepositories )
>>>> + public void
>>>> setPluginArtifactRepositories( List<ArtifactRepository>
>>>> pluginArtifactRepositories )
>>>> {
>>>> this.pluginArtifactRepositories = pluginArtifactRepositories;
>>>> }
>>>> @@ -1367,7 +1370,7 @@
>>>> * @return a list of ArtifactRepository objects constructed
>>>> * from the Repository objects returned by
>>>> getPluginRepositories.
>>>> */
>>>> - public List getPluginArtifactRepositories()
>>>> + public List<ArtifactRepository>
>>>> getPluginArtifactRepositories()
>>>> {
>>>> return pluginArtifactRepositories;
>>>> }
>>>> @@ -1378,17 +1381,17 @@
>>>> : getReleaseArtifactRepository();
>>>> }
>>>>
>>>> - public List getPluginRepositories()
>>>> + public List<Repository> getPluginRepositories()
>>>> {
>>>> return getModel().getPluginRepositories();
>>>> }
>>>>
>>>> - public void setActiveProfiles( List activeProfiles )
>>>> + public void setActiveProfiles( List<String> activeProfiles )
>>>> {
>>>> this.activeProfiles.addAll( activeProfiles );
>>>> }
>>>>
>>>> - public List getActiveProfiles()
>>>> + public List<String> getActiveProfiles()
>>>> {
>>>> return activeProfiles;
>>>> }
>>>> @@ -1398,11 +1401,11 @@
>>>> getAttachedArtifacts().add( artifact );
>>>> }
>>>>
>>>> - public List getAttachedArtifacts()
>>>> + public List<Artifact> getAttachedArtifacts()
>>>> {
>>>> if ( attachedArtifacts == null )
>>>> {
>>>> - attachedArtifacts = new ArrayList();
>>>> + attachedArtifacts = new ArrayList<Artifact>();
>>>> }
>>>> return attachedArtifacts;
>>>> }
>>>> @@ -1420,7 +1423,7 @@
>>>>
>>>> if ( getBuildPlugins() != null )
>>>> {
>>>> - for ( Iterator iterator =
>>>> getBuildPlugins().iterator();
>>>> iterator.hasNext(); )
>>>> + for ( Iterator<Plugin> iterator =
>>>> getBuildPlugins().iterator(); iterator.hasNext(); )
>>>> {
>>>> Plugin plugin = (Plugin) iterator.next();
>>>>
>>>> @@ -1498,7 +1501,7 @@
>>>>
>>>> if ( getReportPlugins() != null )
>>>> {
>>>> - for ( Iterator iterator =
>>>> getReportPlugins().iterator();
>>>> iterator.hasNext(); )
>>>> + for ( Iterator<ReportPlugin> iterator =
>>>> getReportPlugins().iterator(); iterator.hasNext(); )
>>>> {
>>>> ReportPlugin plugin = (ReportPlugin) iterator.next();
>>>>
>>>> @@ -1564,12 +1567,12 @@
>>>> * @return {@link Set} < {@link Artifact} >
>>>> * @see #getArtifacts() to get all transitive dependencies
>>>> */
>>>> - public Set getDependencyArtifacts()
>>>> + public Set<Artifact> getDependencyArtifacts()
>>>> {
>>>> return dependencyArtifacts;
>>>> }
>>>>
>>>> - public void setDependencyArtifacts( Set dependencyArtifacts )
>>>> + public void setDependencyArtifacts( Set<Artifact>
>>>> dependencyArtifacts )
>>>> {
>>>> this.dependencyArtifacts = dependencyArtifacts;
>>>> }
>>>> @@ -1594,12 +1597,12 @@
>>>> return originalModel;
>>>> }
>>>>
>>>> - public void setManagedVersionMap( Map map )
>>>> + public void setManagedVersionMap( Map<String,
>>>> ArtifactVersion> map )
>>>> {
>>>> this.managedVersionMap = map;
>>>> }
>>>>
>>>> - public Map getManagedVersionMap()
>>>> + public Map<String, ArtifactVersion> getManagedVersionMap()
>>>> {
>>>> return this.managedVersionMap;
>>>> }
>>>> @@ -1627,12 +1630,12 @@
>>>> return getId().hashCode();
>>>> }
>>>>
>>>> - public List getBuildExtensions()
>>>> + public List<Extension> getBuildExtensions()
>>>> {
>>>> Build build = getBuild();
>>>> if ( build == null || build.getExtensions() == null )
>>>> {
>>>> - return Collections.EMPTY_LIST;
>>>> + return Collections.<Extension> emptyList();
>>>> }
>>>> else
>>>> {
>>>> @@ -1644,7 +1647,7 @@
>>>> * @todo the lazy initialisation of this makes me uneasy.
>>>> * @return {@link Set} < {@link Artifact} >
>>>> */
>>>> - public Set createArtifacts( ArtifactFactory artifactFactory,
>>>> String
>>>> inheritedScope,
>>>> + public Set<Artifact> createArtifacts( ArtifactFactory
>>>> artifactFactory, String inheritedScope,
>>>> ArtifactFilter dependencyFilter )
>>>> throws InvalidDependencyVersionException
>>>> {
>>>> @@ -1674,12 +1677,12 @@
>>>> return getModel().getProperties();
>>>> }
>>>>
>>>> - public List getFilters()
>>>> + public List<String> getFilters()
>>>> {
>>>> return getBuild().getFilters();
>>>> }
>>>>
>>>> - public Map getProjectReferences()
>>>> + public Map<String, MavenProject> getProjectReferences()
>>>> {
>>>> return projectReferences;
>>>> }
>>>> @@ -1705,22 +1708,22 @@
>>>> this.model = model;
>>>> }
>>>>
>>>> - protected void setAttachedArtifacts( List attachedArtifacts )
>>>> + protected void setAttachedArtifacts( List<Artifact>
>>>> attachedArtifacts )
>>>> {
>>>> this.attachedArtifacts = attachedArtifacts;
>>>> }
>>>>
>>>> - protected void setCompileSourceRoots( List
>>>> compileSourceRoots )
>>>> + protected void setCompileSourceRoots( List<String>
>>>> compileSourceRoots )
>>>> {
>>>> this.compileSourceRoots = compileSourceRoots;
>>>> }
>>>>
>>>> - protected void setTestCompileSourceRoots( List
>>>> testCompileSourceRoots )
>>>> + protected void setTestCompileSourceRoots( List<String>
>>>> testCompileSourceRoots )
>>>> {
>>>> this.testCompileSourceRoots = testCompileSourceRoots;
>>>> }
>>>>
>>>> - protected void setScriptSourceRoots( List scriptSourceRoots )
>>>> + protected void setScriptSourceRoots( List<String>
>>>> scriptSourceRoots
>>>> )
>>>> {
>>>> this.scriptSourceRoots = scriptSourceRoots;
>>>> }
>>>> @@ -1737,16 +1740,16 @@
>>>>
>>>> public void resolveActiveArtifacts()
>>>> {
>>>> - Set depArtifacts = getDependencyArtifacts();
>>>> + Set<Artifact> depArtifacts = getDependencyArtifacts();
>>>> if ( depArtifacts == null )
>>>> {
>>>> return;
>>>> }
>>>>
>>>> - Set updated = new LinkedHashSet( depArtifacts.size() );
>>>> + Set<Artifact> updated = new LinkedHashSet<Artifact>(
>>>> depArtifacts.size() );
>>>> int updatedCount = 0;
>>>>
>>>> - for ( Iterator it = depArtifacts.iterator();
>>>> it.hasNext(); )
>>>> + for ( Iterator<Artifact> it = depArtifacts.iterator();
>>>> it.hasNext(); )
>>>> {
>>>> Artifact depArtifact = (Artifact) it.next();
>>>> Artifact replaced =
>>>> replaceWithActiveArtifact( depArtifact );
>>>> @@ -1815,13 +1818,13 @@
>>>> * @param requestedArtifact The artifact to resolve, must not be
>>>> <code>null</code>.
>>>> * @return The matching artifact or <code>null</code> if not
>>>> found.
>>>> */
>>>> - private Artifact findMatchingArtifact( List artifacts,
>>>> Artifact
>>>> requestedArtifact )
>>>> + private Artifact findMatchingArtifact( List<Artifact>
>>>> artifacts,
>>>> Artifact requestedArtifact )
>>>> {
>>>> if ( artifacts != null && !artifacts.isEmpty() )
>>>> {
>>>> // first try matching by dependency conflict id
>>>> String requestedId =
>>>> requestedArtifact.getDependencyConflictId();
>>>> - for ( Iterator it = artifacts.iterator();
>>>> it.hasNext(); )
>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>> it.hasNext(); )
>>>> {
>>>> Artifact artifact = (Artifact) it.next();
>>>> if ( requestedId.equals(
>>>> artifact.getDependencyConflictId() ) )
>>>> @@ -1832,7 +1835,7 @@
>>>>
>>>> // next try matching by repository conflict id
>>>> requestedId =
>>>> getRepositoryConflictId( requestedArtifact );
>>>> - for ( Iterator it = artifacts.iterator();
>>>> it.hasNext(); )
>>>> + for ( Iterator<Artifact> it = artifacts.iterator();
>>>> it.hasNext(); )
>>>> {
>>>> Artifact artifact = (Artifact) it.next();
>>>> if
>>>> ( requestedId.equals( getRepositoryConflictId( artifact
>>>> ) ) )
>>>> @@ -1898,7 +1901,7 @@
>>>> }
>>>> }
>>>>
>>>> - private void addArtifactPath(Artifact a, List list) throws
>>>> DependencyResolutionRequiredException
>>>> + private void addArtifactPath(Artifact a, List<String> list)
>>>> throws
>>>> DependencyResolutionRequiredException
>>>> {
>>>> File file = a.getFile();
>>>> if ( file == null )
>>>> @@ -1954,21 +1957,21 @@
>>>>
>>>> private Build originalInterpolatedBuild;
>>>>
>>>> - private List dynamicCompileSourceRoots;
>>>> + private List<String> dynamicCompileSourceRoots;
>>>>
>>>> - private List originalInterpolatedCompileSourceRoots;
>>>> + private List<String> originalInterpolatedCompileSourceRoots;
>>>>
>>>> - private List dynamicTestCompileSourceRoots;
>>>> + private List<String> dynamicTestCompileSourceRoots;
>>>>
>>>> - private List originalInterpolatedTestCompileSourceRoots;
>>>> + private List<String>
>>>> originalInterpolatedTestCompileSourceRoots;
>>>>
>>>> - private List dynamicScriptSourceRoots;
>>>> + private List<String> dynamicScriptSourceRoots;
>>>>
>>>> - private List originalInterpolatedScriptSourceRoots;
>>>> + private List<String> originalInterpolatedScriptSourceRoots;
>>>>
>>>> private boolean isConcrete = false;
>>>>
>>>> - public boolean isConcrete()
>>>> + public boolean isConcrete()
>>>> {
>>>> return isConcrete;
>>>> }
>>>> @@ -1988,32 +1991,32 @@
>>>> return originalInterpolatedBuild;
>>>> }
>>>>
>>>> - public List getDynamicCompileSourceRoots()
>>>> + public List<String> getDynamicCompileSourceRoots()
>>>> {
>>>> return dynamicCompileSourceRoots;
>>>> }
>>>>
>>>> - public List getOriginalInterpolatedCompileSourceRoots()
>>>> + public List<String>
>>>> getOriginalInterpolatedCompileSourceRoots()
>>>> {
>>>> return originalInterpolatedCompileSourceRoots;
>>>> }
>>>>
>>>> - public List getDynamicTestCompileSourceRoots()
>>>> + public List<String> getDynamicTestCompileSourceRoots()
>>>> {
>>>> return dynamicTestCompileSourceRoots;
>>>> }
>>>>
>>>> - public List getOriginalInterpolatedTestCompileSourceRoots()
>>>> + public List<String>
>>>> getOriginalInterpolatedTestCompileSourceRoots()
>>>> {
>>>> return originalInterpolatedTestCompileSourceRoots;
>>>> }
>>>>
>>>> - public List getDynamicScriptSourceRoots()
>>>> + public List<String> getDynamicScriptSourceRoots()
>>>> {
>>>> return dynamicScriptSourceRoots;
>>>> }
>>>>
>>>> - public List getOriginalInterpolatedScriptSourceRoots()
>>>> + public List<String> getOriginalInterpolatedScriptSourceRoots()
>>>> {
>>>> return originalInterpolatedScriptSourceRoots;
>>>> }
>>>> @@ -2034,19 +2037,19 @@
>>>> originalInterpolatedBuild = null;
>>>> }
>>>>
>>>> - public void preserveCompileSourceRoots( List
>>>> originalInterpolatedCompileSourceRoots )
>>>> + public void preserveCompileSourceRoots( List<String>
>>>> originalInterpolatedCompileSourceRoots )
>>>> {
>>>> dynamicCompileSourceRoots = getCompileSourceRoots();
>>>> this.originalInterpolatedCompileSourceRoots =
>>>> originalInterpolatedCompileSourceRoots;
>>>> }
>>>>
>>>> - public void preserveTestCompileSourceRoots( List
>>>> originalInterpolatedTestCompileSourceRoots )
>>>> + public void preserveTestCompileSourceRoots( List<String>
>>>> originalInterpolatedTestCompileSourceRoots )
>>>> {
>>>> dynamicTestCompileSourceRoots = getTestCompileSourceRoots();
>>>> this.originalInterpolatedTestCompileSourceRoots =
>>>> originalInterpolatedTestCompileSourceRoots;
>>>> }
>>>>
>>>> - public void preserveScriptSourceRoots( List
>>>> originalInterpolatedScriptSourceRoots )
>>>> + public void preserveScriptSourceRoots( List<String>
>>>> originalInterpolatedScriptSourceRoots )
>>>> {
>>>> dynamicScriptSourceRoots = getScriptSourceRoots();
>>>> this.originalInterpolatedScriptSourceRoots =
>>>> originalInterpolatedScriptSourceRoots;
>>>> @@ -2071,32 +2074,32 @@
>>>> this.originalInterpolatedBuild = originalInterpolatedBuild;
>>>> }
>>>>
>>>> - protected void setDynamicCompileSourceRoots( List
>>>> dynamicCompileSourceRoots )
>>>> + protected void setDynamicCompileSourceRoots( List<String>
>>>> dynamicCompileSourceRoots )
>>>> {
>>>> this.dynamicCompileSourceRoots = dynamicCompileSourceRoots;
>>>> }
>>>>
>>>> - protected void setOriginalInterpolatedCompileSourceRoots( List
>>>> originalInterpolatedCompileSourceRoots )
>>>> + protected void setOriginalInterpolatedCompileSourceRoots(
>>>> List<String> originalInterpolatedCompileSourceRoots )
>>>> {
>>>> this.originalInterpolatedCompileSourceRoots =
>>>> originalInterpolatedCompileSourceRoots;
>>>> }
>>>>
>>>> - protected void setDynamicTestCompileSourceRoots( List
>>>> dynamicTestCompileSourceRoots )
>>>> + protected void setDynamicTestCompileSourceRoots( List<String>
>>>> dynamicTestCompileSourceRoots )
>>>> {
>>>> this.dynamicTestCompileSourceRoots =
>>>> dynamicTestCompileSourceRoots;
>>>> }
>>>>
>>>> - protected void
>>>> setOriginalInterpolatedTestCompileSourceRoots( List
>>>> originalInterpolatedTestCompileSourceRoots )
>>>> + protected void setOriginalInterpolatedTestCompileSourceRoots(
>>>> List<String> originalInterpolatedTestCompileSourceRoots )
>>>> {
>>>> this.originalInterpolatedTestCompileSourceRoots =
>>>> originalInterpolatedTestCompileSourceRoots;
>>>> }
>>>>
>>>> - protected void setDynamicScriptSourceRoots( List
>>>> dynamicScriptSourceRoots )
>>>> + protected void setDynamicScriptSourceRoots( List<String>
>>>> dynamicScriptSourceRoots )
>>>> {
>>>> this.dynamicScriptSourceRoots = dynamicScriptSourceRoots;
>>>> }
>>>>
>>>> - protected void setOriginalInterpolatedScriptSourceRoots( List
>>>> originalInterpolatedScriptSourceRoots )
>>>> + protected void setOriginalInterpolatedScriptSourceRoots(
>>>> List<String> originalInterpolatedScriptSourceRoots )
>>>> {
>>>> this.originalInterpolatedScriptSourceRoots =
>>>> originalInterpolatedScriptSourceRoots;
>>>> }
>>>> @@ -2114,7 +2117,7 @@
>>>> if ( p != null )
>>>> {
>>>> preservedProperties = new Properties();
>>>> - for( Enumeration e = p.propertyNames();
>>>> e.hasMoreElements();
>>>> )
>>>> + for( Enumeration<?> e = p.propertyNames();
>>>> e.hasMoreElements(); )
>>>> {
>>>> String key = (String) e.nextElement();
>>>> preservedProperties.setProperty( key,
>>>> p.getProperty( key )
>>>> );
>>>>
>>>> Modified:
>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/
>>>> pom.xml
>>>> URL:
>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-repository-metadata/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> ===================================================================
>>>>
>>>> ---
>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/
>>>> pom.xml
>>>> (original)
>>>> +++
>>>> maven/components/branches/maven-2.2.x/maven-repository-metadata/
>>>> pom.xml Fri
>>>> May 1 08:04:48 2009
>>>> @@ -47,6 +47,7 @@
>>>> <models>
>>>> <model>src/main/mdo/metadata.mdo</model>
>>>> </models>
>>>> + <useJava5>true</useJava5>
>>>> </configuration>
>>>> </plugin>
>>>> </plugins>
>>>>
>>>> Modified: maven/components/branches/maven-2.2.x/maven-settings/
>>>> pom.xml
>>>> URL:
>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-settings/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> ===================================================================
>>>>
>>>> --- maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>> (original)
>>>> +++ maven/components/branches/maven-2.2.x/maven-settings/pom.xml
>>>> Fri May
>>>> 1 08:04:48 2009
>>>> @@ -58,6 +58,7 @@
>>>> <models>
>>>> <model>src/main/mdo/settings.mdo</model>
>>>> </models>
>>>> + <useJava5>true</useJava5>
>>>> </configuration>
>>>> </plugin>
>>>> </plugins>
>>>>
>>>> Modified: maven/components/branches/maven-2.2.x/maven-toolchain/
>>>> pom.xml
>>>> URL:
>>>> http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml?rev=770570&r1=770569&r2=770570&view=diff
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> ===================================================================
>>>>
>>>> --- maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>> (original)
>>>> +++ maven/components/branches/maven-2.2.x/maven-toolchain/pom.xml
>>>> Fri May
>>>> 1 08:04:48 2009
>>>> @@ -66,6 +66,7 @@
>>>> <models>
>>>> <model>src/main/mdo/toolchains.mdo</model>
>>>> </models>
>>>> + <useJava5>true</useJava5>
>>>> </configuration>
>>>> </plugin>
>>>> <plugin>
>>>>
>>>>
>>>>
>>> Thanks,
>>>
>>> Jason
>>>
>>> ----------------------------------------------------------
>>> Jason van Zyl
>>> Founder, Apache Maven
>>> http://twitter.com/jvanzyl
>>> http://twitter.com/SonatypeNexus
>>> http://twitter.com/SonatypeM2E
>>> ----------------------------------------------------------
>>>
>>> We all have problems. How we deal with them is a measure of our
>>> worth.
>>>
>>> -- Unknown
>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>>> For additional commands, e-mail: dev-help@maven.apache.org
>>>
>>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>> For additional commands, e-mail: dev-help@maven.apache.org
>>
>>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org