Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 312681 - Launching by hitting run from Launch Config Dialog fails
Summary: Launching by hitting run from Launch Config Dialog fails
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 3.6   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.6 RC1   Edit
Assignee: Platform-Debug-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-12 14:23 EDT by Chris Goldthorpe CLA
Modified: 2010-05-14 13:15 EDT (History)
2 users (show)

See Also:
Michael_Rennie: review+

Attachments
Screenshot (18.74 KB, image/png)
2010-05-12 14:26 EDT, Chris Goldthorpe CLA 		no flags Details
patch (6.74 KB, patch)
2010-05-12 15:11 EDT, Darin Wright CLA 		no flags Details | Diff
simpler patch (3.55 KB, patch)
2010-05-12 15:25 EDT, Darin Wright CLA 		no flags 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 Chris Goldthorpe CLA 2010-05-12 14:23:44 EDT 
I20100511-0800

I used a clean workspace when reproducing this.

Run/Run Configurations
Create a new Eclipse Configuration
Change the name to "Eclipse2"

Hit Run without hitting Apply first, so the name change does not yet show in the tree

An error dialog is shown, see attachment
Comment 1 Chris Goldthorpe CLA 2010-05-12 14:26:02 EDT 
Created attachment 168221 [details]
Screenshot
Comment 2 Darin Wright CLA 2010-05-12 14:40:41 EDT 
This was caused by the fix to bug 311017.
Comment 3 Darin Wright CLA 2010-05-12 15:11:09 EDT 
Created attachment 168238 [details]
patch
Comment 4 Darin Wright CLA 2010-05-12 15:11:31 EDT 
please review, Mike.
Comment 5 Darin Wright CLA 2010-05-12 15:25:32 EDT 
Created attachment 168239 [details]
simpler patch
Comment 6 Michael Rennie CLA 2010-05-12 15:39:07 EDT 
+1 much better than the first patch
Comment 7 Darin Wright CLA 2010-05-12 15:40:51 EDT 
Applied/Fixed.
Comment 8 Chris Goldthorpe CLA 2010-05-14 13:15:37 EDT 
Verified in I20100513-1500