Bug 1022308 - Build source jar for stilts-stomplet-server-bundle-0.1.26
Summary: Build source jar for stilts-stomplet-server-bundle-0.1.26
Keywords:
Status: CLOSED EOL
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: Build
Version: 6.2.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ER2
: EAP 6.3.0
Assignee: Yong Yang
QA Contact: Petr Kremensky
URL:
Whiteboard:
Depends On:
Blocks: 908741
TreeView+ depends on / blocked
 
Reported: 2013-10-23 02:26 UTC by Paul Gier
Modified: 2016-08-31 08:28 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-08-31 08:28:24 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Paul Gier 2013-10-23 02:26:28 UTC
The prod build of stilts-stomplet-server-bundle-0.1.26 should produce a -sources jar for the Maven repository.
https://brewweb.devel.redhat.com/buildinfo?buildID=229605

Comment 1 Lin Gao 2013-10-24 05:20:51 UTC
The build in question builds using maven-bundle-plugin:2.3.5-redhat-1, which based on upstream version 2.3.5, which is incompatible with maven-source-plugin, see: https://issues.apache.org/jira/browse/FELIX-3058

Comment 2 Lin Gao 2013-10-24 09:22:43 UTC
Mead Build: org.projectodd.stilts-stilts-0.1.26.redhat_3-1 (https://brewweb.devel.redhat.com/buildinfo?buildID=301777)

Brew Build: stilts-0.1.26-9.redhat_3.1.ep6.el6 (https://brewweb.devel.redhat.com/buildinfo?buildID=301787)

Comment 3 Lin Gao 2013-10-24 09:32:41 UTC
Pull Request: https://github.com/jbossas/jboss-eap/pull/606

Comment 4 Lin Gao 2013-10-24 12:26:13 UTC
Although the source jar is generated after this fixing, but the stilts-stomplet-server-bundle.jar contains inner jars, is it an issue?

Comment 5 Vladimir Dosoudil 2013-10-25 13:43:32 UTC
Merged in https://github.com/jbossas/jboss-eap/commit/73d5515

Comment 6 Pavel Jelinek 2013-11-07 16:27:32 UTC
Verified for EAP 6.2.0 ER7.

There is 
jboss-eap-6.2.0.GA-maven-repository/org/projectodd/stilts/stilts-stomplet-server-bundle/0.1.26.redhat-3/stilts-stomplet-server-bundle-0.1.26.redhat-3-sources.jar
in jboss-eap-6.2.0.ER7-maven-repository.zip. It contains 1 source file /org/projectodd/stilts/stomplet/bundle/StompletServerActivator.java. It's ok. 

But it does NOT contain sources for other jar files bundled inside of stilts-stomplet-server-bundle-0.1.26.redhat-3.jar mainly 
stilts-conduit-server-core-0.1.26.redhat-3.jar,
stilts-conduit-server-spi-0.1.26.redhat-3.jar,
stilts-stomp-api-0.1.26.redhat-3.jar,
stilts-stomp-common-0.1.26.redhat-3.jar,
stilts-stomplet-api-0.1.26.redhat-3.jar,
stilts-stomplet-server-core-0.1.26.redhat-3.jar,
stilts-stomp-server-core-0.1.26.redhat-3.jar,
stilts-stomp-server-spi-0.1.26.redhat-3.jar. 
There isn't any source file for these jar files in the jboss-eap-6.2.0.GA-maven-repository.

Comment 7 Lin Gao 2013-11-08 06:15:02 UTC
The mead builds generates the sources already, I don't know why the sources are missing from the mead repository.

see: http://download.eng.bos.redhat.com/brewroot/repos/jb-eap-6-rhel-6-build/latest/maven/org/projectodd/stilts/stilts-stomp-server-core/0.1.26.redhat-3/stilts-stomp-server-core-0.1.26.redhat-3-sources.jar

Comment 8 Pavel Jelinek 2013-11-08 12:07:32 UTC
To be more specific: the problem is that sources for 

stilts-conduit-server-core-0.1.26.redhat-3.jar,
stilts-conduit-server-spi-0.1.26.redhat-3.jar,
stilts-stomp-api-0.1.26.redhat-3.jar,
stilts-stomp-common-0.1.26.redhat-3.jar,
stilts-stomplet-api-0.1.26.redhat-3.jar,
stilts-stomplet-server-core-0.1.26.redhat-3.jar,
stilts-stomp-server-core-0.1.26.redhat-3.jar,
stilts-stomp-server-spi-0.1.26.redhat-3.jar

are not anywhere in the maven repo zip.

Comment 10 Paul Gier 2013-11-11 15:39:14 UTC
Ok, I think we should skip this fix for 6.2, and assign an upstream issue to determine the right fix for this in the community jars for 6.3.  I'm not sure whether it's correct to put the sources from the nested jars directly in the aggregator source jar.

Comment 11 Paul Gier 2013-11-11 15:39:38 UTC
Ok, I think we should skip this fix for 6.2, and assign an upstream issue to determine the right fix for this in the community jars for 6.3.  I'm not sure whether it's correct to put the sources from the nested jars directly in the aggregator source jar.

Comment 12 Yong Yang 2014-04-08 11:33:20 UTC
PR: https://github.com/jbossas/jboss-eap/pull/1198

Comment 13 Pavel Jelinek 2014-05-01 04:45:58 UTC
Unfortunatelly in the jboss-eap-6.3.0.ER2-maven-repository.zip there aren't any sources for bundled jars mentioned in comment 8 https://bugzilla.redhat.com/show_bug.cgi?id=1022308#c8 yet.

Comment 14 Yong Yang 2016-08-31 08:28:24 UTC
I think this issue could be closed now.


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