Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 333190 - Import/Export Settings Wizard for C/C++ not working correctly for workspace paths
Summary: Import/Export Settings Wizard for C/C++ not working correctly for workspace p...
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-build (show other bugs)
Version: 8.0   Edit
Hardware: PC Windows XP
: P3 minor (vote)
Target Milestone: 8.0   Edit
Assignee: Andrew Gvozdev CLA
QA Contact: Andrew Gvozdev CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-24 07:17 EST by Gil Barash CLA
Modified: 2010-12-24 11:23 EST (History)
1 user (show)

See Also:


Attachments
Patch to fix "workspace path" bug (2.72 KB, patch)
2010-12-24 07:19 EST, Gil Barash CLA
angvoz.dev: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gil Barash CLA 2010-12-24 07:17:52 EST
Build Identifier: I20100805-1700 ; CDT 8

When using the Import/Export Settings Wizard for C/C++ to export the include paths of a project, if one of those paths is a workspace path, it would be imported (using the same wizard) as a regular path.

Reproducible: Always

Steps to Reproduce:
1. Export include paths of a project when one of those paths is a workspace path.
2. Import the settings to a different project.
3. See the include paths of the project (we just imported the settings to). The include path which was a workspace path is displayed as a regular path.
Comment 1 Gil Barash CLA 2010-12-24 07:19:46 EST
Created attachment 185806 [details]
Patch to fix "workspace path" bug
Comment 2 Andrew Gvozdev CLA 2010-12-24 10:05:37 EST
I added an ability to refresh the export file if it happened to be in workspace. While strictly it is not related to the patch, it makes the testing easier and I wanted to do it by some other reasons anyway.
Comment 4 Andrew Gvozdev CLA 2010-12-24 10:29:59 EST
Committed on HEAD (8.0) with minor change slightly renaming new constant. Thanks for the patch Gil!