Bug 293210 - Context launch launches shared launch configuration twice.
Summary: Context launch launches shared launch configuration twice.
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 3.6   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.6 M3   Edit
Assignee: Platform-Debug-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2009-10-23 16:08 EDT by Daniel Friederich CLA
Modified: 2009-10-26 12:04 EDT (History)
1 user (show)

See Also:

Attachments
add missing return so context launch does not go on. (810 bytes, patch)
2009-10-23 16:09 EDT, Daniel Friederich CLA 		no flags Details | Diff
fix merge bug in pref patch (919 bytes, patch)
2009-10-23 16:12 EDT, Daniel Friederich CLA 		Michael_Rennie: iplog+
Details | Diff
Show Obsolete (1) 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 Daniel Friederich CLA 2009-10-23 16:08:06 EDT 
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.3) Gecko/20090824 Firefox/3.5.3
Build Identifier: I20090611-1540

When sharing a launch configuration (in Common tab, Save as selected as Shared File) and this launch configuration is selected, the context launch first directly launches it and then goes on looking for what to launch. In the end two launches are created (or the second fails if the debug model does not support this).


Reproducible: Always

Steps to Reproduce:
1. Create a CDT project with a shared launch configuration
2. select it in the projects view
3. Click the launch button.
Expected : one running gdb
Get: two running gdb sessions.
Comment 1 Daniel Friederich CLA 2009-10-23 16:09:19 EDT 
Created attachment 150409 [details]
add missing return so context launch does not go on.
Comment 2 Daniel Friederich CLA 2009-10-23 16:12:15 EDT 
Created attachment 150410 [details]
fix merge bug in pref patch

Ups, initial patch did add the return at the wrong line. Did not hurt, but did not fix the bug either.
Comment 3 Michael Rennie CLA 2009-10-26 12:04:02 EDT 
applied patch to HEAD
Comment 4 Michael Rennie CLA 2009-10-26 12:04:49 EDT 
verified, thanks for the fix Daniel!