Return-Path: 
 <commits-return-46657-apmail-maven-commits-archive=maven.apache.org@maven.apache.org>
X-Original-To: apmail-maven-commits-archive@www.apache.org
Delivered-To: apmail-maven-commits-archive@www.apache.org
Received: from mail.apache.org (hermes.apache.org [140.211.11.3])
	by minotaur.apache.org (Postfix) with SMTP id 6995E10908
	for <apmail-maven-commits-archive@www.apache.org>;
 Wed, 10 Apr 2013 01:25:20 +0000 (UTC)
Received: (qmail 16657 invoked by uid 500); 10 Apr 2013 01:25:20 -0000
Delivered-To: apmail-maven-commits-archive@maven.apache.org
Received: (qmail 16526 invoked by uid 500); 10 Apr 2013 01:25:20 -0000
Mailing-List: contact commits-help@maven.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:commits-help@maven.apache.org>
List-Unsubscribe: <mailto:commits-unsubscribe@maven.apache.org>
List-Post: <mailto:commits@maven.apache.org>
List-Id: <commits.maven.apache.org>
Reply-To: dev@maven.apache.org
Delivered-To: mailing list commits@maven.apache.org
Received: (qmail 16519 invoked by uid 99); 10 Apr 2013 01:25:20 -0000
Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230)
    by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 10 Apr 2013 01:25:20 +0000
X-ASF-Spam-Status: No, hits=-2000.0 required=5.0
	tests=ALL_TRUSTED
X-Spam-Check-By: apache.org
Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4)
    by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 10 Apr 2013 01:25:17 +0000
Received: from eris.apache.org (localhost [127.0.0.1])
	by eris.apache.org (Postfix) with ESMTP id CF7B3238899C;
	Wed, 10 Apr 2013 01:24:55 +0000 (UTC)
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r1466309 - in /maven/plugins/trunk/maven-site-plugin/src:
 main/java/org/apache/maven/plugins/site/AbstractSiteRenderingMojo.java
 site/apt/maven-3.apt.vm
Date: Wed, 10 Apr 2013 01:24:55 -0000
To: commits@maven.apache.org
From: hboutemy@apache.org
X-Mailer: svnmailer-1.0.8-patched
Message-Id: <20130410012455.CF7B3238899C@eris.apache.org>
X-Virus-Checked: Checked by ClamAV on apache.org

Author: hboutemy
Date: Wed Apr 10 01:24:55 2013
New Revision: 1466309

URL: http://svn.apache.org/r1466309
Log:
[MSITE-684] mark reportPlugins parameter as read-only, hiding it from generated goal documentation (but not enforcing it won't be used because Maven 3 doesn't check: see MNG-5001)

Modified:
    maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/AbstractSiteRenderingMojo.java
    maven/plugins/trunk/maven-site-plugin/src/site/apt/maven-3.apt.vm

Modified: maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/AbstractSiteRenderingMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/AbstractSiteRenderingMojo.java?rev=1466309&r1=1466308&r2=1466309&view=diff
==============================================================================
--- maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/AbstractSiteRenderingMojo.java (original)
+++ maven/plugins/trunk/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/AbstractSiteRenderingMojo.java Wed Apr 10 01:24:55 2013
@@ -170,7 +170,7 @@ public abstract class AbstractSiteRender
     protected MavenSession mavenSession;
 
     /**
-     * <p>Configuration section used internally by Maven 3.</p>
+     * <p>Configuration section <b>used internally</b> by Maven 3.</p>
      * <p>More details available here:
      * <a href="http://maven.apache.org/plugins/maven-site-plugin/maven-3.html#Configuration_formats" target="_blank">
      * http://maven.apache.org/plugins/maven-site-plugin/maven-3.html#Configuration_formats</a>
@@ -178,9 +178,9 @@ public abstract class AbstractSiteRender
      * <p><b>Note:</b> using this field is not mandatory with Maven 3 as Maven core injects usual
      * <code>&lt;reporting&gt;</code> section into this field.</p>
      *
-     * @since 3.0-beta-1
+     * @since 3.0-beta-1 (and read-only since 3.3)
      */
-    @Parameter
+    @Parameter( readonly = true )
     private ReportPlugin[] reportPlugins;
 
     private PlexusContainer container;

Modified: maven/plugins/trunk/maven-site-plugin/src/site/apt/maven-3.apt.vm
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-site-plugin/src/site/apt/maven-3.apt.vm?rev=1466309&r1=1466308&r2=1466309&view=diff
==============================================================================
--- maven/plugins/trunk/maven-site-plugin/src/site/apt/maven-3.apt.vm (original)
+++ maven/plugins/trunk/maven-site-plugin/src/site/apt/maven-3.apt.vm Wed Apr 10 01:24:55 2013
@@ -289,13 +289,22 @@ Using it with Maven 3
 
 ** New Configuration (Maven 3 only, no reports configuration inheritance)
 
-  Reports can be configured in the configuration of <<<maven-site-plugin>>> as <<<\<reportPlugins\>>>> elements.
+  Starting with <<<maven-site-plugin>>> 3.0, reports could be configured in the configuration of <<<maven-site-plugin>>>
+  as <<<\<reportPlugins\>>>> elements.
   
   <<This new configuration format is not actually ready for end-users: please don't use it for the moment.>>
 
-  <Explanation:> The new format does not actually support report plugins configuration inheritance, which is crucial for
-  usability: see {{{http://jira.codehaus.org/browse/MSITE-484}MSITE-484}}. This format was technically necessary to remove
-  most reporting logic from Maven 3, but a new inheritance mechanism still needs to be added to make it as flexible as the old format.
+  As a consequence, with <<<maven-site-plugin>>> 3.3, the <<<\<reportPlugins\>>>> configuration has simply been removed
+  (more precisely marked private/internal), since it should not be used directly. This documentation has been kept
+  public only to help people who used it in previous versions understand why they need to migrate back to classic configuration.
+
+  <Explanation:> The (former) new format did not support report plugins configuration inheritance, which is crucial for
+  usability: see {{{http://jira.codehaus.org/browse/MSITE-484}MSITE-484}}. This field is technically necessary to remove
+  most reporting logic from Maven 3, but a new inheritance mechanism still needs to be added to make it as flexible as the old format:
+  we made the field public before finding this crucial limitation and required to explain later why end-users must not use it, then
+  cannot use it since <<<maven-site-plugin>>> 3.3.
+
+  Here is how this feature looked like:
 
 +-----+
   <build>
@@ -303,10 +312,10 @@ Using it with Maven 3
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-site-plugin</artifactId>
-        <version>${project.version}</version>
+        <version>3.2</version><!-- last maven-site-plugin version supporting this reportPlugins configuration -->
         <configuration>
           ...
-          <reportPlugins>
+          <reportPlugins><!-- not supported any more... (now kept internal) -->
             <plugin>
               <groupId>org.apache.maven.plugins</groupId>
               <artifactId>maven-project-info-reports-plugin</artifactId>