Bug 787300

Summary: eclipse-wtp-servertools does not build in koji
Product: [Fedora] Fedora Reporter: Sami Wagiaalla <swagiaal>
Component: eclipse-wtp-servertoolsAssignee: Vladimir Kostadinov <vladimir.kostadinov>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: akurtako, mmcgrath, nitind, overholt, rgrunber, vladimir.kostadinov
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-02-14 09:31:23 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Fix some missing deps none

Description Sami Wagiaalla 2012-02-03 20:44:54 UTC
Created attachment 559363 [details]
Fix some missing deps

The build fails with the following error:

Bundle org.eclipse.wst.server.preview_1.1.100.201202031435 failed to resolve.:
	Missing required plug-in org.mortbay.jetty.server_[6.1.7,6.2.0).
	Missing required plug-in org.mortbay.jetty.util_[6.1.7,6.2.0).
	Missing required plug-in javax.servlet_[2.5.0,2.6.0).

I suspect this is because the jars providing the above bundles were removed from eclipse-platform.

I have fixed other missing deps problems with the attached patch

Comment 1 Roland Grunberg 2012-02-03 21:03:28 UTC
In rawhide, jetty 6 has been updated to jetty 8 and the latest eclipse is using this. The org.mortbay namespace has been replaced by org.eclipse.jetty. Also, javax.servlet in the latest eclipse should be at version 3.0.

Comment 2 Vladimir Kostadinov 2012-02-10 20:01:42 UTC
Fixed, using some intermediate revision from WTP's CVS.

Fixed in eclipse-wtp-servertools-3.3.1-4.fc17

Comment 3 Andrew Overholt 2012-02-10 22:10:05 UTC
Thanks for fixing this, Vladimir!

I suspect the version and/or release should be slightly different:

https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Pre-Release_packages

(unless you just back-ported a patch)

Comment 4 Vladimir Kostadinov 2012-02-12 10:28:37 UTC
It's more of a back-port patch, so I think it's better to leave the version untouched.

Comment 5 Alexander Kurtakov 2012-02-14 09:31:23 UTC
Closing as there is a koji build fixing the problem.