Re: svn commit: r926991 - /geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml

2010-03-24 Thread Rex Wang
Yes, I agree. How about merge server-security-config car into
j2ee-security(actually I think we should rename the car as server-security)?
Will open a jira to track this.

2010/3/24 Kevan Miller 

> Thanks Rex. Makes sense.
>
> I guess that's consistent with our treatment of server-security-config
> dependencies, but I'm not convinced we're handling them correctly. Pretty
> certain that we can build alternate assemblies with the same basic
> problem...
>
> --kevan
>
> On Mar 24, 2010, at 5:40 AM, rwo...@apache.org wrote:
>
> > Author: rwonly
> > Date: Wed Mar 24 09:40:12 2010
> > New Revision: 926991
> >
> > URL: http://svn.apache.org/viewvc?rev=926991&view=rev
> > Log:
> > GERONIMO-5024 var/security/keystores seems to be a required directory,
> but is not being created for custom assemblies
> >
> > Modified:
> >geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml
> >
> > Modified:
> geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml
> > URL:
> http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml?rev=926991&r1=926990&r2=926991&view=diff
> >
> ==
> > --- geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml
> (original)
> > +++ geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml
> Wed Mar 24 09:40:12 2010
> > @@ -51,7 +51,14 @@
> > ${version}
> > car
> > 
> > -
> > +
> > +
> > +org.apache.geronimo.framework
> > +server-security-config
> > +${version}
> > +car
> > +
> > +
> > 
> > org.apache.geronimo.modules
> > geronimo-activemq
> >
> >
>
>


-- 
Lei Wang (Rex)
rwonly AT apache.org


Re: svn commit: r926991 - /geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml

2010-03-24 Thread Kevan Miller
Thanks Rex. Makes sense.

I guess that's consistent with our treatment of server-security-config 
dependencies, but I'm not convinced we're handling them correctly. Pretty 
certain that we can build alternate assemblies with the same basic problem...

--kevan

On Mar 24, 2010, at 5:40 AM, rwo...@apache.org wrote:

> Author: rwonly
> Date: Wed Mar 24 09:40:12 2010
> New Revision: 926991
> 
> URL: http://svn.apache.org/viewvc?rev=926991&view=rev
> Log:
> GERONIMO-5024 var/security/keystores seems to be a required directory, but is 
> not being created for custom assemblies
> 
> Modified:
>geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml
> 
> Modified: 
> geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml?rev=926991&r1=926990&r2=926991&view=diff
> ==
> --- geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml 
> (original)
> +++ geronimo/server/branches/2.2/plugins/activemq/activemq-broker/pom.xml Wed 
> Mar 24 09:40:12 2010
> @@ -51,7 +51,14 @@
> ${version}
> car
> 
> -
> +
> +
> +org.apache.geronimo.framework
> +server-security-config
> +${version}
> +car
> +
> +
> 
> org.apache.geronimo.modules
> geronimo-activemq
> 
>