Bug 238919 - Add default build targets such as Incremental/Full/Clean to Make Target View, organized by configurations
Summary: Add default build targets such as Incremental/Full/Clean to Make Target View,...
Status: NEW
Alias: None
Product: CDT
Classification: Tools
Component: cdt-build (show other bugs)
Version: 5.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact: Jonah Graham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-06-29 17:57 EDT by Andrew Gvozdev CLA
Modified: 2020-09-04 15:16 EDT (History)
1 user (show)

See Also:


Attachments
Make Targets View with default Build targets (20.23 KB, image/png)
2008-06-29 18:00 EDT, Andrew Gvozdev CLA
no flags Details
the patch (30.60 KB, patch)
2008-06-29 18:01 EDT, Andrew Gvozdev CLA
no flags Details | Diff
icons for the view (6.40 KB, patch)
2008-06-29 18:02 EDT, Andrew Gvozdev CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Gvozdev CLA 2008-06-29 17:57:09 EDT
Here is an enhancement for Make Target View to allow display and run such actions as Full Build, Incremental Build, Clean Build. These actions are configuration specific, so they got organized in configurations category/folder. Look at the attached picture. Actually, I am interested in another problem, in having a make targets run in a specific configuration (see bug 224640), and this patch is pretty much a byproduct. But I figured this might be useful for somebody and here it is. Feedback is appreciated, if the community thinks it's worthy I would incorporate feedback and clean it up to be suitable for a commit.
Comment 1 Andrew Gvozdev CLA 2008-06-29 18:00:15 EDT
Created attachment 106086 [details]
Make Targets View with default Build targets
Comment 2 Andrew Gvozdev CLA 2008-06-29 18:01:23 EDT
Created attachment 106087 [details]
the patch
Comment 3 Andrew Gvozdev CLA 2008-06-29 18:02:29 EDT
Created attachment 106089 [details]
icons for the view
Comment 4 Anton Leherbauer CLA 2009-07-09 04:11:11 EDT
I think adding a dependency to managebuilder.core is problematic.
Also, when adding a dependency you should use either Require-Bundle or Import-Package, but not both.
Comment 5 Andrew Gvozdev CLA 2009-07-09 07:46:49 EDT
(In reply to comment #4)
Thanks. This is a year old patch and frankly requires quite a rework.

> I think adding a dependency to managebuilder.core is problematic.
> Also, when adding a dependency you should use either Require-Bundle or
> Import-Package, but not both.
What's the difference between those two?