[prev in list] [next in list] [prev in thread] [next in thread] 

List:       maven-dev
Subject:    Re: Reporting plugin configuration not inheriting
From:       "Nikolas Everett" <nik9000 () gmail ! com>
Date:       2008-10-31 17:28:39
Message-ID: d4e11e980810311028m6e5cf7c5r68a0b6797e39a3bf () mail ! gmail ! com
[Download RAW message or body]


Thanks for pointing me in the right direction.  10-RC2 doesn't fix it, but
this patch on the head of

http://svn.apache.org/repos/asf/maven/components/branches/maven-2.0.x

does:
Index: maven-project/src/main/java/org/apache/maven/project/ModelUtils.java
===================================================================
--- maven-project/src/main/java/org/apache/maven/project/ModelUtils.java
(revision 709447)
+++ maven-project/src/main/java/org/apache/maven/project/ModelUtils.java
(working copy)
@@ -431,7 +431,17 @@
         String parentInherited = parent.getInherited();

         boolean parentIsInherited = ( parentInherited == null ) ||
Boolean.valueOf( parentInherited ).booleanValue();
+
+    if (parentIsInherited)
+    {
+            Xpp3Dom childConfiguration = (Xpp3Dom)
child.getConfiguration();
+            Xpp3Dom parentConfiguration = (Xpp3Dom)
parent.getConfiguration();

+            childConfiguration = Xpp3Dom.mergeXpp3Dom( childConfiguration,
parentConfiguration );
+
+        child.setConfiguration(childConfiguration);
+    }
+
         List parentReportSets = parent.getReportSets();

         if ( ( parentReportSets != null ) && !parentReportSets.isEmpty() )


END PATCH

I'm not sure if it breaks any other behavior, though.  I'll create a bug for
this and submit it.

--Nik

On Fri, Oct 31, 2008 at 11:21 AM, Wendy Smoak <wsmoak@gmail.com> wrote:

> The users list is the right place... give it a bit more time, it
> hasn't even been 24 hours yet. :)
>
> You can also search JIRA to see if this problem has already been
> reported.  Is this it?  http://jira.codehaus.org/browse/MNG-1999
> (Reporting inheritance does not work properly)
>
> If so, it's marked fixed in 2.0.10, so try the release candidate that
> Brian posted recently:
> http://www.nabble.com/-RC%7D-2.0.10-RC2-td20239269.html
>
> --
> Wendy
>
> On Fri, Oct 31, 2008 at 6:10 AM, Nikolas Everett <nik9000@gmail.com>
> wrote:
> > I was having a problem yesterday getting the configuration for reporting
> > plugins to inherit like the configuration for build plugins does.  My
> emails
> > didn't get any traction on the users mailing list so I'm trying here.
>  I'm
> > sorry if this is the wrong place to bring this up.  If it is, please let
> me
> > know where to go.
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>


[prev in list] [next in list] [prev in thread] [next in thread]