Bug 745878 (EDG-80) - unneccessary jars in infinispan-server-rest.war
Summary: unneccessary jars in infinispan-server-rest.war
Keywords:
Status: CLOSED NEXTRELEASE
Alias: EDG-80
Product: JBoss Data Grid 5
Classification: JBoss
Component: Infinispan
Version: EAP 5.1.0 EDG TP
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: EAP 5.1.0 EDG TP
Assignee: Default User
QA Contact:
URL: http://jira.jboss.org/jira/browse/EDG-80
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-04-27 12:28 UTC by Michal Linhard
Modified: 2014-03-17 04:02 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-05-13 06:37:35 UTC
Type: Bug


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker EDG-80 0 None None None Never

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 



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