Author: kenney Date: Thu Sep 15 17:37:58 2005 New Revision: 289353 URL: http://svn.apache.org/viewcvs?rev=289353&view=rev Log: o EclipsePlugin now does NOT download sources by default. Specify -Declipse.downloadSources=true to do so.
o WarMojo produces an artifact, even if exploded is specified. This makes sure that the install phase doesn't fail because there's no artifact. o DependenciesReport: add caught exception to the new thrown exception for better messages. Modified: maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/main/java/org/apache/maven/plugin/eclipse/EclipsePlugin.java maven/components/trunk/maven-plugins/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/DependenciesReport.java maven/components/trunk/maven-plugins/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/WarMojo.java Modified: maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/main/java/org/apache/maven/plugin/eclipse/EclipsePlugin.java URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/main/java/org/apache/maven/plugin/eclipse/EclipsePlugin.java?rev=289353&r1=289352&r2=289353&view=diff ============================================================================== --- maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/main/java/org/apache/maven/plugin/eclipse/EclipsePlugin.java (original) +++ maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/main/java/org/apache/maven/plugin/eclipse/EclipsePlugin.java Thu Sep 15 17:37:58 2005 @@ -143,7 +143,7 @@ * * @parameter expression="${eclipse.downloadSources}" */ - private boolean downloadSources = true; + private boolean downloadSources = false; /** * Eclipse workspace directory. Modified: maven/components/trunk/maven-plugins/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/DependenciesReport.java URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-plugins/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/DependenciesReport.java?rev=289353&r1=289352&r2=289353&view=diff ============================================================================== --- maven/components/trunk/maven-plugins/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/DependenciesReport.java (original) +++ maven/components/trunk/maven-plugins/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/DependenciesReport.java Thu Sep 15 17:37:58 2005 @@ -279,7 +279,7 @@ throw new IllegalArgumentException( "Can't find a valid Maven project in the repository for the artifact [" + artifact.getGroupId() + ":" + artifact.getArtifactId() + ":" - + artifact.getVersion() + "]." ); + + artifact.getVersion() + "].", e ); } tableRow( new String[]{artifact.getGroupId(), artifact.getArtifactId(), artifact.getVersion(), artifactProject.getDescription(), Modified: maven/components/trunk/maven-plugins/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/WarMojo.java URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-plugins/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/WarMojo.java?rev=289353&r1=289352&r2=289353&view=diff ============================================================================== --- maven/components/trunk/maven-plugins/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/WarMojo.java (original) +++ maven/components/trunk/maven-plugins/maven-war-plugin/src/main/java/org/apache/maven/plugin/war/WarMojo.java Thu Sep 15 17:37:58 2005 @@ -314,7 +314,10 @@ { generateExplodedWebapp(); - if ( !"exploded".equals( mode ) ) + // TODO: make a separate 'exploded' Mojo. For now, + // disable not making an artifact so the install phase + // doesn't fail. + // if ( !"exploded".equals( mode ) ) { //generate war file getLog().info( "Generating war " + warFile.getAbsolutePath() ); --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]