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

Bug 333190

Summary:

Import/Export Settings Wizard for C/C++ not working correctly for workspace paths

Product:

[Tools] CDT

Reporter:

Gil Barash <scobido1984>

Component:

cdt-build

Assignee:

Andrew Gvozdev <angvoz.dev>

Status:

RESOLVED FIXED

QA Contact:

Andrew Gvozdev <angvoz.dev>

Severity:

minor

Priority:

CC:

cdtdoug

Version:

8.0

Target Milestone:

8.0

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
Patch to fix "workspace path" bug	angvoz.dev: iplog+

Description Gil Barash

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

2010-12-24 07:19:46 EST

Created attachment 185806 [details]
Patch to fix "workspace path" bug

Comment 2 Andrew Gvozdev

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 3 CDT Genie

2010-12-24 10:23:03 EST

*** cdt cvs genie on behalf of agvozdev ***
bug 333190: Moved refreshWorkspaceFiles() for general usage, in particular for the Import/Export Settings Wizard

[+] ResourcesUtil.java  http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.core/src/org/eclipse/cdt/core/resources/ResourcesUtil.java?root=Tools_Project&revision=1.1&view=markup

[*] CopyBuildLogAction.java 1.4 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/buildconsole/CopyBuildLogAction.java?root=Tools_Project&r1=1.3&r2=1.4
[*] BuildConsolePartitioner.java 1.17 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/buildconsole/BuildConsolePartitioner.java?root=Tools_Project&r1=1.16&r2=1.17
[*] BuildConsoleManager.java 1.26 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/buildconsole/BuildConsoleManager.java?root=Tools_Project&r1=1.25&r2=1.26

[*] ProjectSettingsExportStrategy.java 1.3 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/wizards/settingswizards/ProjectSettingsExportStrategy.java?root=Tools_Project&r1=1.2&r2=1.3

Comment 4 Andrew Gvozdev

2010-12-24 10:29:59 EST

Committed on HEAD (8.0) with minor change slightly renaming new constant. Thanks for the patch Gil!

Comment 5 CDT Genie

2010-12-24 11:23:02 EST

*** cdt cvs genie on behalf of agvozdev ***
bug 333190: Import/Export Settings Wizard for C/C++ not working correctly for workspace paths
Patch from Gil Barash

[*] IncludePathsSettingsProcessor.java 1.4 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/wizards/settingswizards/IncludePathsSettingsProcessor.java?root=Tools_Project&r1=1.3&r2=1.4

[*] ResourcesUtil.java 1.2 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.core/src/org/eclipse/cdt/core/resources/ResourcesUtil.java?root=Tools_Project&r1=1.1&r2=1.2