Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 349385

Summary: Remove photran dependency from PLDT plugins
Product: [Tools] PTP Reporter: Greg Watson <g.watson>
Component: PLDTAssignee: Beth Tibbitts <beth>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: beth, com-eclipse-dot-org
Version: 5.0Flags: g.watson: review? (beth)
Target Milestone: 6.0   
Hardware: Macintosh   
OS: Mac OS X   
Whiteboard:
Attachments:
Description Flags
Patch to removed photran dependency
none
New plugins containing photran dependencies
none
Updated patch none

Description Greg Watson CLA 2011-06-14 19:17:20 EDT
Currently the org.eclipse.ptp.pldt.mpi.core and org.eclipse.ptp.pldt.openmp.core plugins have dependencies on org.eclipse.photran. Even though these dependencies are optional, P2 will install the plugins if it can find them. It is not possible to remove the plugins as the installation manager only allows whole features to be removed, and if photran was installed as a dependency, the features are not installed, only the plugins.

Instead, I suggest that the photran dependencies be moved to plugins that belong to a separate feature.
Comment 1 Greg Watson CLA 2011-06-14 19:18:06 EDT
Created attachment 197991 [details]
Patch to removed photran dependency

This patch removes the photran dependent parts from the plugins
Comment 2 Greg Watson CLA 2011-06-14 19:21:49 EDT
Created attachment 197992 [details]
New plugins containing photran dependencies

This tar file contains two new plugins with the photran dependent components.
Comment 3 Greg Watson CLA 2011-06-15 08:45:25 EDT
Created attachment 198013 [details]
Updated patch
Comment 4 Beth Tibbitts CLA 2011-06-20 14:15:50 EDT
checked in.  Greg can you add the feature?
Comment 5 Jeffrey Overbey CLA 2011-06-20 16:46:22 EDT
Where are the new projects checked in at?  I didn't find them in /org.eclipse.ptp/tools with the other PLDT projects...
Comment 6 Beth Tibbitts CLA 2011-06-20 22:24:25 EDT
Because I forgot to share the new projects. :)
Try now.
Comment 7 Beth Tibbitts CLA 2011-07-09 11:09:28 EDT
Didn't we revert this for the 5.0 branch because it would amount to API change?
Comment 8 Greg Watson CLA 2011-07-11 10:01:34 EDT
Change target to 6.0