Bug 285904 - Control build-before-launch activity per launch configuation
Summary: Control build-before-launch activity per launch configuation
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug (show other bugs)
Version: 6.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 6.1   Edit
Assignee: Ken Ryall CLA
QA Contact: Ken Ryall CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-06 12:15 EDT by Ken Ryall CLA
Modified: 2010-05-28 15:37 EDT (History)
2 users (show)

See Also:
ken.ryall: review? (elaskavaia.cdt)
ken.ryall: review? (john.cortell)


Attachments
patch for this enhancement (19.18 KB, patch)
2009-08-06 12:17 EDT, Ken Ryall CLA
no flags Details | Diff
screenshot (12.44 KB, image/png)
2009-08-06 12:19 EDT, Ken Ryall CLA
no flags Details
updated patch (19.10 KB, patch)
2009-08-17 14:34 EDT, Ken Ryall CLA
cdtdoug: iplog-
Details | Diff
screenshot (5.58 KB, image/png)
2009-08-17 14:34 EDT, Ken Ryall CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ken Ryall CLA 2009-08-06 12:15:52 EDT
As a developer I want to be able to tell the debugger to build before launch in some cases but not others. For some projects I may have already built them outside of Eclipse and want to avoid the build-before-launch step. For others I want to make sure everything is built up-to-date before launch.
Comment 1 Ken Ryall CLA 2009-08-06 12:17:35 EDT
Created attachment 143688 [details]
patch for this enhancement

This patch adds this setting to CMainTab and the implementation to CAbstractLaunchDelegate.
Comment 2 Ken Ryall CLA 2009-08-06 12:19:39 EDT
Created attachment 143689 [details]
screenshot

Screenshot showing the Main launch configuration Tab with this setting.
Comment 3 Elena Laskavaia CLA 2009-08-16 12:22:13 EDT
Yes I need this feature too (as occasional C developer).

I think "Enable build" be a checkbox instead of radio button (grayed out when use workspace is selected). I would also suggest to compress a composite in 2 rows. something like

(*) Use Workspace Settings          Configure Workspace Settings...
[ ] Enable auto Build               Build Configuration: [Debug       v]      
Comment 4 Ken Ryall CLA 2009-08-17 14:34:21 EDT
Created attachment 144706 [details]
updated patch

New patch with CMainTab changes.
Comment 5 Ken Ryall CLA 2009-08-17 14:34:54 EDT
Created attachment 144707 [details]
screenshot

Updated screenshot
Comment 6 Ken Ryall CLA 2009-08-17 14:39:54 EDT
(In reply to comment #3)
> Yes I need this feature too (as occasional C developer).
> 
> I think "Enable build" be a checkbox instead of radio button (grayed out when
> use workspace is selected). I would also suggest to compress a composite in 2
> rows. something like
> 
> (*) Use Workspace Settings          Configure Workspace Settings...
> [ ] Enable auto Build               Build Configuration: [Debug       v]      
> 

Thanks for the feedback. I've updated the CMainTab UI to use both rows as you suggested. I left the options as there were as the radio buttons reflect the three choices: always build for this launch, never build for this launch, use the workspace settings. We experimented with making this two related choices but people told us they found that approach was more difficult to understand.
Comment 7 Elena Laskavaia CLA 2009-08-18 09:46:29 EDT
Ok thanks
Comment 8 Ken Ryall CLA 2009-09-06 00:10:17 EDT
Fixed in HEAD.