Bug 913321 - Do not add transitive dependency to deployments for system dependencies or user dependencies
Summary: Do not add transitive dependency to deployments for system dependencies or us...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: Server
Version: 6.0.1
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ER3
: EAP 6.1.0
Assignee: Brad Maxwell
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-02-21 00:22 UTC by Brad Maxwell
Modified: 2013-07-23 18:46 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker AS7-6580 0 Major Resolved Do not add transitive dependency to deployments for system dependencies or user dependencies 2017-03-07 04:30:05 UTC

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


Note You need to log in before you can comment on or make changes to this bug.