Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 922883

Summary: jboss-as-jsf-injection difference between eap zip and maven repo zip
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Paul Gier <pgier>
Component: Maven RepositoryAssignee: Rostislav Svoboda <rsvoboda>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.1.0CC: rsvoboda, vtunka
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:
Attachments:
Description Flags
Patch to loop through multiple results for the same filename in the eap zip none

Description Paul Gier 2013-03-18 17:44:49 UTC
The smoke test to compare the jars in the eap zip against the jars in the maven repository zip is reporting that there is a difference between the jar jboss-as-jsf-injection-7.2.0.Final-redhat-3.jar in the eap zip compared to the one in the maven repo zip.

This appears to be a bug in the test caused by the fact that this jar is contained in the as zip twice, once under "main" and once under "1.2".

Comment 2 Rostislav Svoboda 2013-03-19 13:07:09 UTC
I think this is related to Multi-JSF support 
https://community.jboss.org/wiki/DesignOfAS7Multi-JSFFeature

Comment 3 Rostislav Svoboda 2013-03-19 13:24:07 UTC
Proposed patch:

-  PATH_IN_EAP=`find $EAP_DIR -name $BASENAME`
+  if [[ $BASENAME == jboss-as-jsf-injection* ]]; then
+    PATH_IN_EAP=`find $EAP_DIR -name $BASENAME | head -1`  ## BZ 922883 - jboss-as-jsf-injection duplicated
+  else 
+      PATH_IN_EAP=`find $EAP_DIR -name $BASENAME`
+  fi

Can I merge it ?  I need confirmation that jboss-as-jsf-injection duplication is intentional.

Comment 4 Vaclav Tunka 2013-03-19 13:25:39 UTC
Hi Rosta,

yes, this is indeed intentional, thank you for fixing the test.

Cheers,
Vaclav

Comment 5 Rostislav Svoboda 2013-03-19 13:44:28 UTC
Proposed patch merged.

Comment 6 Vaclav Tunka 2013-03-19 14:10:04 UTC
Fernado told me duplications like this happen quite a lot, so it's not an isolated case and we can expect issues like this more. We talked with Paul, that this is OK as a workaround this time, but for the future we would like to see more systematic solution. Can you please add this to your todo?

Comment 7 Paul Gier 2013-03-19 14:11:17 UTC
Created attachment 712698 [details]
Patch to loop through multiple results for the same filename in the eap zip

Here is a more general fix that should handle cases where multiple files have the same name in the eap zip.

Comment 8 Rostislav Svoboda 2013-03-19 15:53:09 UTC
Patch merged + added modifications:
 - report generated for each file when having multiple files with the same name
 - listing of duplicated jars

Comment 9 Rostislav Svoboda 2013-03-20 13:46:49 UTC
Duplication of jboss-as-jsf-injection is intentional