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

Bug 342096

Summary: Executables view shows binaries for all build configurations instead of just the active ones
Product: [Tools] CDT Reporter: John Cortell <john.cortell>
Component: cdt-debugAssignee: cdt-debug-inbox <cdt-debug-inbox>
Status: RESOLVED FIXED QA Contact: Ken Ryall <ken.ryall>
Severity: normal    
Priority: P3 CC: cdtdoug, jamesblackburn+eclipse, pawel.1.piech
Version: 8.0Flags: jamesblackburn+eclipse: review+
Target Milestone: 8.0   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Fix john.cortell: iplog-

Description John Cortell CLA 2011-04-06 21:24:51 EDT 
The Executables view gathers the list of executables from one or more providers that are responsible for searching through the open CDT projects and identifying the executables. It's expected the providers may dish out an answer based on which build configuration is active, as a configuration switch causes the view to re-query its providers. The stock provider in CDT (StandardExecutableProvider), however, answers with all binaries in all configurations. The end result is that the Executables view can quickly become very cluttered in an environment with many projects with many configurations. 

I've discussed this with Ken and he confirmed that the lack of filtering in StandardExecutableProvider is an oversight. So, I'll be adding the filtering. I'll also go one step further and update the JavaDoc to specify that providers should provide binaries relevant only to the active build configuration.
Comment 1 John Cortell CLA 2011-04-06 21:28:00 EDT 
Reproducibility steps:

1. Create a simple project using the wizard. By default it gives you Debug and Release targets.
2. Build all configurations. 
3. Open the executables view. 
Note that it shows the executables from both configurations. It should only show the one created by the active configuration
Comment 2 John Cortell CLA 2011-04-06 23:36:26 EDT 
Created attachment 192691 [details]
Fix
Comment 3 James Blackburn CLA 2011-04-07 07:51:13 EDT 
The cdt.core calling code looks fine to me John.
Comment 4 John Cortell CLA 2011-04-07 08:30:23 EDT 
Fixed