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

Bug 363212

Summary: Plugins that are used only by Gprof and Gcov were added as a dependency for oprofile and valgrind
Product: z_Archived Reporter: Otavio Pontes <obusatto>
Component: LinuxToolsAssignee: Xavier Raynaud <xraynaud>
Status: RESOLVED FIXED QA Contact: Xavier Raynaud <xraynaud>
Severity: normal    
Priority: P3 CC: kksebasti, lilia.ghachem
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Remove the dependency from profiling feature
none
Adding the removed dependencies back to gprof and gcov none

Description Otavio Pontes CLA 2011-11-08 12:53:51 EST 
Build Identifier: I20110613-1736

The plugins org.eclipse.linuxtools.binutils, org.eclipse.linuxtools.dataviewers, 
org.eclipse.linuxtools.dataviewers.charts, org.eclipse.linuxtools.dataviewers.annotatedsourceeditor that are only used by gprof and gcov were added as a dependency for org.eclipse.linuxtools.profiling-feature.
The problem is that oprofile and valgrind depends on org.eclipse.linuxtools.profiling feature, so they will require the installation of these unnecessary plugins. The greater problem for me is that this also adds indirectly the dependency for BIRT, DTP and EMF, that are large packages.
Since these packages are only used by gprof and gcov I suggest to remove them from org.eclipse.linuxtools.profiling-feature/feature.xml and add them to org.eclipse.linuxtools.gprof-feature/feature.xml and org.eclipse.linuxtools.gcov-feature/feature.xml.

Reproducible: Always

Steps to Reproduce:
1. Install only oprofile plugin in an eclipse installation
2. Note that some useless plugins were installed, like BIRT and org.eclipse.linuxtools.dataviewers.charts
Comment 1 Otavio Pontes CLA 2011-11-09 06:54:12 EST 
Created attachment 206676 [details]
Remove the dependency from profiling feature

This revert the commit that adds unnecessary dependencies into profiling.
Comment 2 Otavio Pontes CLA 2011-11-09 06:55:02 EST 
Created attachment 206677 [details]
Adding the removed dependencies back to gprof and gcov

This patch adds the dependencies back to gprof and gcov
Comment 3 Otavio Pontes CLA 2011-11-10 09:54:10 EST 
Jeff Johnston created a new feature for dataviewers and binutils. 
This bug was fixed by commits 50937355abf3d5d6e75206865d8a409a8d4615a0 and 9745c96f06cdae797a0fd2baca5ef893207adff3