Bug 835283

Summary: [patch] Include org.eclipse.wst.xsl.feature
Product: [Fedora] Fedora Reporter: Gerard Ryan <fedora>
Component: eclipse-wtp-sourceeditingAssignee: Vladimir Kostadinov <vladimir.kostadinov>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: akurtako, kdaniel, mgoldman, 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-07-12 07:54:46 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:
Embargoed:
Bug Depends On: 834944, 834946, 834947, 834948    
Bug Blocks:    
Attachments:
Description Flags
[patch] Include org.eclipse.wst.xsl.feature
none
[patch] Include org.eclipse.wst.xsl.feature none

Description Gerard Ryan 2012-06-25 20:38:23 UTC
Created attachment 594280 [details]
[patch] Include org.eclipse.wst.xsl.feature

This patch adds a feature that wasn't previously being built: org.eclipse.wst.xsl.feature

Comment 1 Gerard Ryan 2012-06-25 23:11:04 UTC
Created attachment 594312 [details]
[patch] Include org.eclipse.wst.xsl.feature

Updated patch

Comment 2 Vladimir Kostadinov 2012-06-27 18:27:16 UTC
The requirement xalan-j2 >= 2.7.1-10 refers to non-existing build of the package. 

Is there any specific reason for that?

Comment 3 Marek Goldmann 2012-06-27 19:07:11 UTC
(In reply to comment #2)
> The requirement xalan-j2 >= 2.7.1-10 refers to non-existing build of the
> package. 
> 
> Is there any specific reason for that?

See bug 834948. I need to apply the patch there and will bump the version to -10, but I don't think there is any specific reason to require that version here.

Comment 4 Gerard Ryan 2012-06-27 21:13:45 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > The requirement xalan-j2 >= 2.7.1-10 refers to non-existing build of the
> > package. 
> > 
> > Is there any specific reason for that?
> 
> See bug 834948. I need to apply the patch there and will bump the version to
> -10, but I don't think there is any specific reason to require that version
> here.

I can't remember exactly, I think that version might be needed: might be why I made that change to xalan-j2. Although it might have been for the other features that I had originally planned on including, but then extracted them into their own package to avoid cyclic dependencies (eclipse-wtp-jst-web - bug 835338)

Comment 5 Vladimir Kostadinov 2012-07-02 08:03:40 UTC
Please notify me when the 2.7.1-10 is ready so I can apply your patch.

Comment 6 Gerard Ryan 2012-07-10 21:32:34 UTC
(In reply to comment #5)
> Please notify me when the 2.7.1-10 is ready so I can apply your patch.

It's ready now, thanks! :)

Comment 7 Vladimir Kostadinov 2012-07-12 07:54:46 UTC
Done.