This change (adding @aggregator) is being pointed to as the cause of many problems with version 2.3 of the Javadoc plugin.
See http://jira.codehaus.org/browse/MJAVADOC-137 and related issues. Should all or part of this be reverted? -- Wendy On Wed, Mar 14, 2007 at 1:28 PM, <[EMAIL PROTECTED]> wrote: > Author: carlos > Date: Wed Mar 14 13:28:56 2007 > New Revision: 518309 > > URL: http://svn.apache.org/viewvc?view=rev&rev=518309 > Log: > [MJAVADOC-104] Javadoc of generated sources is not generated when > aggregate=true > Submitted By: Julien Henry > > Modified: > > maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java > > Modified: > maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java > URL: > http://svn.apache.org/viewvc/maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java?view=diff&rev=518309&r1=518308&r2=518309 > > ============================================================================== > --- > maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java > (original) > +++ > maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java > Wed Mar 14 13:28:56 2007 > @@ -9,7 +9,7 @@ > * "License"); you may not use this file except in compliance > * with the License. You may obtain a copy of the License at > * > - * http://www.apache.org/licenses/LICENSE-2.0 > + * http://www.apache.org/licenses/LICENSE-2.0 > * > * Unless required by applicable law or agreed to in writing, > * software distributed under the License is distributed on an > @@ -73,6 +73,7 @@ > * @author <a href="mailto:[EMAIL PROTECTED]">Vincent Siveton</a> > * @requiresDependencyResolution compile > * @execute phase="generate-sources" > + * @aggregator > */ > public abstract class AbstractJavadocMojo > extends AbstractMojo > @@ -1338,6 +1339,12 @@ > MavenProject project = (MavenProject) i.next(); > > List sourceRoots = project.getCompileSourceRoots(); > + > + if ( project.getExecutionProject() != null ) > + { > + sourceRoots.addAll( > project.getExecutionProject().getCompileSourceRoots() ); > + } > + > ArtifactHandler artifactHandler = > project.getArtifact().getArtifactHandler(); > if ( "java".equals( artifactHandler.getLanguage() ) ) > { > @@ -1956,7 +1963,7 @@ > * @param repeatKey repeat or not the key in the command line > * @param splitValue if <code>true</code> given value will be tokenized > by comma > */ > - private void addArgIfNotEmpty( List arguments, String key, String value, > + private void addArgIfNotEmpty( List arguments, String key, String value, > boolean repeatKey, boolean splitValue ) > { > if ( StringUtils.isNotEmpty( value ) ) > @@ -1971,11 +1978,11 @@ > while ( token.hasMoreTokens() ) > { > String current = token.nextToken().trim(); > - > + > if ( StringUtils.isNotEmpty( current ) ) > { > arguments.add( current ); > - > + > if ( token.hasMoreTokens() && repeatKey ) > { > arguments.add( key ); > @@ -1987,7 +1994,7 @@ > } > } > } > - > + > /** > * Convenience method to add an argument to the <code>command line</code> > * if the the value is not null or empty. > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]