Bug 312225 - Need way to Clean All configurations in Workspace
Summary: Need way to Clean All configurations in Workspace
Status: ASSIGNED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-build (show other bugs)
Version: 7.0   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: cdt-build-inbox@eclipse.org CLA
QA Contact: Jonah Graham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-10 05:13 EDT by James Blackburn CLA
Modified: 2020-09-04 15:27 EDT (History)
1 user (show)

See Also:


Attachments
patch 1 (3.62 KB, patch)
2010-10-19 08:29 EDT, James Blackburn CLA
jamesblackburn+eclipse: iplog-
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description James Blackburn CLA 2010-05-10 05:13:16 EDT
Say you have 40 project and 120 configurations in the workspace. There's no way to easily clean all configurations.

Andrew's new UI for Build Selected... allows you to select configurations to build / clean, but it would be a pain to select all of them.

So a couple of improvements:
  - Override Project > Clean to show the new Build Selected... dialog
  - Add a Select All button to this dialog
  - Non-CDT projects should be shown in the tree view -- just without configurations, so there's no loss of functionality.

We already override the BuildAction in CDT so referenced projects aren't built by the UI, so perhaps we should override the Clean UI as well so we can do something sensible.

The only alternative is external scripts to empty all the build output directories.
Comment 1 Andrew Gvozdev CLA 2010-05-10 09:35:24 EDT
(In reply to comment #0)
> Say you have 40 project and 120 configurations in the workspace. There's no way
> to easily clean all configurations.
There is "Clean All" item in the menu under "Build Configurations" next to "Build Selected...". I thought that was the easy way.

> Andrew's new UI for Build Selected... allows you to select configurations to
> build / clean, but it would be a pain to select all of them.
In the dialog you could click on project checkbox and that would de-select or select all configurations. Well, I guess if there are several projects you can't select all of them at one click. I thought it would be a stretch but we could add a root checkbox. I think that would be obvious enough and we don't need ugly "Select All"/"Unselect All" buttons.
Comment 2 James Blackburn CLA 2010-05-10 09:39:16 EDT
(In reply to comment #1)
> There is "Clean All" item in the menu under "Build Configurations" next to
> "Build Selected...". I thought that was the easy way.

But that's only available when your selection only contains CDT projects.  If your selection contains closed projects, or non-CDT projects, this doesn't show.  
I know you can do this with working sets, but it means that, at the moment, there's no way to ctrl-A > Clean All.  i.e. number of clicks is large.

> I thought it would be a stretch but we
> could add a root checkbox. I think that would be obvious enough and we don't
> need ugly "Select All"/"Unselect All" buttons.

Again the project set of doom here has grown to 47 projects... => 50-something clicks to clean/build seems a bit much.
Comment 3 Andrew Gvozdev CLA 2010-05-10 09:50:11 EDT
It sounds more and more that logically it should be placed (in addition to context menu) to the main menu - Project->Build Configuration. I tried to do that but had problems getting it there from managed builder plugin.
Comment 4 James Blackburn CLA 2010-10-19 08:29:42 EDT
Created attachment 181176 [details]
patch 1

Simple patch to add workspace root to clean and build dialog if more than one project is selected when the action is performed.
Comment 5 Andrew Gvozdev CLA 2010-10-19 09:58:16 EDT
I think we should use workspace.gif icon from cdt.ui for the root.
Also, it would be more intuitive to *select* all after the first click rather than deselect. I haven't done that in original implementation and it was bothering me since.