Bug 913321

Summary:	Do not add transitive dependency to deployments for system dependencies or user dependencies
Product:	[JBoss] JBoss Enterprise Application Platform 6	Reporter:	Brad Maxwell <bmaxwell>
Component:	Server	Assignee:	Brad Maxwell <bmaxwell>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	6.0.1	CC:	atangrin, brian.stansberry, jason.greene, jpai, jperkins, pkremens
Target Milestone:	ER3
Target Release:	EAP 6.1.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Brad Maxwell 2013-02-21 00:22:48 UTC

Do not add transitive dependency to deployments for system dependencies or user dependencies 

Currently the ModuleSpecProcessor adds transitive dependencies to a deployment if the module spec is specified to add them (looks like only RAR's from what I can see). This includes transitive dependencies that are declared in a jboss-deployment-structure.xml or ones automatically added by the container.

Change to only allow class-path entries to be transitive if the module spec defines they should be. System and user specified dependencies should not be transitive.

Customer reported issue, where war excludes several implicit modules and then depends on another deployment which does not exclude any modules.  The war is getting a transitive dependency back on the modules it excluded: 

https://na7.salesforce.com/500A000000CSLhp

Comment 1 James Perkins 2013-02-21 00:44:57 UTC

Pull request https://github.com/jbossas/jboss-eap/pull/9

Comment 3 Jaikiran Pai 2013-02-21 12:13:47 UTC

Updated PR https://github.com/jbossas/jboss-eap/pull/11

Comment 4 JBoss JIRA Server 2013-02-26 17:54:00 UTC

James Perkins <jperkins> made a comment on jira AS7-6580

Pull request merged.

Comment 5 Brian Stansberry 2013-03-05 20:54:15 UTC

Merged https://github.com/jbossas/jboss-eap/pull/11.

Comment 7 Brian Stansberry 2013-03-06 16:21:08 UTC

This got "unacked" -- perhaps due to a component change??? But it was previously 3-acked and the code is merged in jboss-eap on github.

Comment 12 Petr Kremensky 2013-04-04 08:49:46 UTC

Verified on EAP 6.1.0 ER4

Comment 15 mark yarborough 2013-07-23 18:46:55 UTC

Details inside vpn: https://post-office.corp.redhat.com/mailman/private/eap-pm-list/2013-July/msg00105.html