321778 – Lost preference: build only if changes in workspace

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

Bug 321778 - Lost preference: build only if changes in workspace

Summary: Lost preference: build only if changes in workspace

Status:	RESOLVED FIXED

Alias:	None

Product:	CDT
Classification:	Tools
Component:	cdt-build (show other bugs)
Version:	8.0
Hardware:	All All

Importance:	P3 major (vote)
Target Milestone:	8.0
Assignee:	Vivian Kong
QA Contact:	Andrew Gvozdev

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2010-08-04 14:54 EDT by Keith W. Campbell
Modified:	2010-09-29 05:23 EDT (History)
CC List:	1 user (show)

See Also:

Attachments
proposed patch (5.82 KB, patch) 2010-08-04 16:11 EDT, Keith W. Campbell	vivkong: iplog+	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Keith W. Campbell

2010-08-04 14:54:34 EDT

Build Identifier: helios

The preference 'Build configurations only when there are Eclipse resource changes within the project and its references' in the 'C/C++' group is reset (disabled) every time a workspace is restarted.

This preference was introduced for bug 219337.

Reproducible: Always

Steps to Reproduce:
I downloaded the CDT package for Win32 from http://www.eclipse.org/downloads/packages/eclipse-ide-cc-developers/heliosr.

In a new workspace, I enabled the preference 'Build configurations only when there are Eclipse resource changes within the project and its references' in the 'C/C++' group.

After restarting the workspace that preference is lost.

Worse, if you enable it, it doesn't take effect unless you press 'Apply' twice (or more), or press 'Apply' and then 'OK'.

Comment 1 Keith W. Campbell

2010-08-04 16:11:32 EDT

Created attachment 175885 [details]
proposed patch

Comment 2 Keith W. Campbell

2010-08-05 13:56:10 EDT

Vivian, seeing as you were involved in fixing bug 219337, I thought you might be able to comment on this (and commit it if meets with your approval). Thanks.

Comment 3 Vivian Kong

2010-09-28 11:37:50 EDT

I'll take a look

Comment 4 Vivian Kong

2010-09-28 11:39:16 EDT

Applied to HEAD with minor copyright statement changes.  Thanks Keith

Comment 5 CDT Genie

2010-09-28 12:23:03 EDT

*** cdt cvs genie on behalf of vkong ***
Bug 321778 Lost preference: build only if changes in workspace - patch by Keith W. Campbell

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

[*] CCorePreferenceInitializer.java 1.25 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.core/src/org/eclipse/cdt/internal/core/CCorePreferenceInitializer.java?root=Tools_Project&r1=1.24&r2=1.25

[*] CCorePreferenceConstants.java 1.16 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.core/src/org/eclipse/cdt/core/CCorePreferenceConstants.java?root=Tools_Project&r1=1.15&r2=1.16

[*] CPluginPreferencePage.java 1.31 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/preferences/CPluginPreferencePage.java?root=Tools_Project&r1=1.30&r2=1.31

Comment 6 CDT Genie

2010-09-29 05:23:02 EDT

*** cdt cvs genie on behalf of mschorn ***
Bug 321778: Fixes since tags.

[*] PathUtil.java 1.9 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt-core/org.eclipse.cdt.core/utils/org/eclipse/cdt/utils/PathUtil.java?root=Tools_Project&r1=1.8&r2=1.9

[*] CCorePreferenceConstants.java 1.17 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt-core/org.eclipse.cdt.core/src/org/eclipse/cdt/core/CCorePreferenceConstants.java?root=Tools_Project&r1=1.16&r2=1.17