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 <[email protected]> > 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, [email protected] 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>${version}</version> > > <type>car</type> > > </dependency> > > - > > + > > + <dependency> > > + <groupId>org.apache.geronimo.framework</groupId> > > + <artifactId>server-security-config</artifactId> > > + <version>${version}</version> > > + <type>car</type> > > + </dependency> > > + > > <dependency> > > <groupId>org.apache.geronimo.modules</groupId> > > <artifactId>geronimo-activemq</artifactId> > > > > > > -- Lei Wang (Rex) rwonly AT apache.org
