Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 345151 - Executable View should create virtual folders when importing executables
Summary: Executable View should create virtual folders when importing executables
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug (show other bugs)
Version: 8.0   Edit
Hardware: PC Windows XP
: P3 minor (vote)
Target Milestone: 8.1.0   Edit
Assignee: cdt-debug-inbox@eclipse.org CLA
QA Contact: Ken Ryall CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-09 10:54 EDT by Teodor Madan CLA
Modified: 2014-03-10 05:08 EDT (History)
2 users (show)

See Also:


Attachments
Mark folders as deirved fix (1.40 KB, patch)
2011-05-09 11:06 EDT, Teodor Madan CLA
teodor.madan: iplog-
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Teodor Madan CLA 2011-05-09 10:54:07 EDT
Executable view creates a container CDT project "Executables", when importing an external binary in the view. While doing so, StandardExecutableImporter will replicate absolute path to binary as subfolders in "Executable" project. The folder structure is replicated to act as a namespace to ensure uniqueness of the binary file.  

Instead of using a path from physical local folders, standard executable importer should instead create a chain of virtual folders and mark them as derived. 
- Creating virtual folders is a lot more lighter operation. 
- Marking as derived will ensure that a good chunk of resource listeners will disregard the event for derived resources.
Comment 1 Teodor Madan CLA 2011-05-09 11:06:02 EDT
Created attachment 195083 [details]
Mark folders as deirved fix

Ken, could you please review?

Thank you,
Teo
Comment 2 CDT Genie CLA 2011-07-17 06:20:03 EDT
*** cdt git genie on behalf of 345151 ***

    Bug 345151 - Executable View should create virtual folders when
    importing executables

[*] http://git.eclipse.org/c/cdt/org.eclipse.cdt.git/commit/?id=e6b07afaf6e6ef100cbc5d09fa8442a0718c3c96
Comment 3 Teodor Madan CLA 2014-03-10 05:08:18 EDT
Mark as resolved. The fix was included in CDT 8.1