Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 343546 - org.eclipse.mylyn.git-feature missing dependency on o.e.m.versions-feature
Summary: org.eclipse.mylyn.git-feature missing dependency on o.e.m.versions-feature
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: 0.7.1   Edit
Assignee: Steffen Pingel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-21 10:15 EDT by Severin Gehwolf CLA
Modified: 2011-04-21 11:16 EDT (History)
0 users

See Also:

Attachments
Proposed patch. (633 bytes, patch)
2011-04-21 10:17 EDT, Severin Gehwolf CLA 		no flags Details | Diff
applied patch (638 bytes, patch)
2011-04-21 11:10 EDT, Steffen Pingel CLA 		no flags Details | Diff
mylyn/context/zip (714 bytes, application/octet-stream)
2011-04-21 11:11 EDT, Steffen Pingel CLA 		no flags Details
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Severin Gehwolf CLA 2011-04-21 10:15:51 EDT 
Looking at o.e.m.git.core, I see a Require-Bundle: org.eclipse.mylyn.versions.core. I think org.eclipse.mylyn.git-feature is missing this dependency in feature.xml. Either make it depend on the versions-feature or on versions.core plug-in. As long as the dependency is listed it should be fine. Thanks.
Comment 1 Severin Gehwolf CLA 2011-04-21 10:17:04 EDT 
Created attachment 193830 [details]
Proposed patch.

Feel free to change this to a plug-in depenency if that's the preferred way. Thanks!
Comment 2 Steffen Pingel CLA 2011-04-21 11:10:55 EDT 
Created attachment 193845 [details]
applied patch
Comment 3 Steffen Pingel CLA 2011-04-21 11:11:03 EDT 
Created attachment 193846 [details]
mylyn/context/zip
Comment 4 Steffen Pingel CLA 2011-04-21 11:16:54 EDT 
Thanks for debugging that. I have added a feature dependency. Committed to head and e_3_7_m_3_5_x branch.