Bug 745878 (EDG-80)

Summary: unneccessary jars in infinispan-server-rest.war
Product: [JBoss] JBoss Data Grid 5 Reporter: Michal Linhard <mlinhard>
Component: InfinispanAssignee: Default User <jbpapp-maint>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: high Docs Contact:
Priority: high    
Version: EAP 5.1.0 EDG TPCC: galder.zamarreno, mlinhard, nobody, trustin
Target Milestone: ---   
Target Release: EAP 5.1.0 EDG TP   
Hardware: Unspecified   
OS: Unspecified   
URL: http://jira.jboss.org/jira/browse/EDG-80
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-05-13 06:37:35 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Michal Linhard 2011-04-27 12:28:37 UTC
project_key: EDG

Just noticed some unneccessary jars being added to infinispan-server-rest.war, when creating EDG datagrid profile in
https://anonsvn.jboss.org/repos/jbossas/branches/JBPAPP_5_1_datagrid/cluster/src/installers/datagrid/build.xml

I would extend the last delete command:
{code}
<delete dir="${datagrid.deploy}/datagrid.sar/infinispan-server-rest.war/WEB-INF/lib/" includes="infinispan-core*, log4j*, slf4j*, jgroups*, river*, scala-library*, jboss-*"/>
{code}

to:
{code}
<delete>
	<fileset dir="${datagrid.deploy}/datagrid.sar/infinispan-server-rest.war/WEB-INF/lib">
		<include name="activation*" />
		<include name="javassist*" />
		<include name="marshalling-api*" />
		<include name="netty*" />
		<include name="infinispan-*" />
		<include name="log4j*" />
		<include name="slf4j*" />
		<include name="jgroups*" />
		<include name="river*" />
		<include name="scala-library*" />
		<include name="jboss-*" />
	</fileset>
</delete>
{code}

because the lib contains dependencies that are duplicated in datagrid.sar and EAP itself...
I can do the change myself, just want to gather opinions on this.

WDYT ?

Comment 1 Trustin Lee 2011-04-29 08:04:36 UTC
Sounds OK to me.  Should I check in the change or are you supposed to do it?  I'm just new to EAP process.

Comment 2 Michal Linhard 2011-04-29 08:16:20 UTC
Yeah check it in if you will, please.
I just offered help, because it's quite easy process for me too. I'm quite used to commiting to svn.jboss.org as opposed to infinispan's github :-)

Comment 3 Galder ZamarreƱo 2011-04-29 15:01:43 UTC
Michal, I'm fine wit this too.

Comment 4 Trustin Lee 2011-05-04 06:46:06 UTC
I don't have a write permission of the repository - I've just got the 'forbidden' error.  Could someone add me there?

Comment 5 Trustin Lee 2011-05-11 07:51:54 UTC
No permission yet

Comment 6 Trustin Lee 2011-05-13 06:37:35 UTC
Release Notes Text: Added: Fix checked in as suggested.  Thanks!


Comment 7 Trustin Lee 2011-05-13 11:28:20 UTC
Fix checked in as suggested.  Thanks!
(Previously typed into a wrong field.)

Comment 8 Trustin Lee 2011-05-13 11:28:20 UTC
Release Notes Text: Removed: Fix checked in as suggested.  Thanks! 


Comment 9 Anne-Louise Tangring 2011-10-11 17:06:09 UTC
Docs QE Status: Removed: NEW