315490 – [launch] JTAG debug launch default should be set for the launch configuration type

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 315490 - [launch] JTAG debug launch default should be set for the launch configuration type

Summary: [launch] JTAG debug launch default should be set for the launch configuration...

Status:	RESOLVED FIXED

Alias:	None

Product:	CDT
Classification:	Tools
Component:	cdt-debug (show other bugs)
Version:	7.0
Hardware:	PC Linux

Importance:	P3 normal (vote)
Target Milestone:	8.0
Assignee:	Marc Khouzam
QA Contact:	Ken Ryall

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2010-06-02 16:29 EDT by Marc Khouzam
Modified:	2010-06-22 12:12 EDT (History)
CC List:	2 users (show)

See Also:

Flags:	john.cortell: review+

Attachments
Fix (4.41 KB, patch) 2010-06-22 11:17 EDT, Marc Khouzam	marc.khouzam: iplog-	Details \| Diff
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 Marc Khouzam

2010-06-02 16:29:20 EDT

The same fix as used for Bug 312997 should be used for the JTAG launch configuration type.

Comment 1 Marc Khouzam

2010-06-22 11:17:03 EDT

Created attachment 172427 [details]
Fix

Fix based on the way we now do things to set the default delegate for other launch configuration types.

Comment 2 Marc Khouzam

2010-06-22 11:20:40 EDT

Committed to HEAD.

John, you've recently dabbled in these plugins, can you have a look?

P.S. I didn't think this was important enough for 7.0.1 but I could be convinced otherwise.

Comment 3 CDT Genie

2010-06-22 11:23:02 EDT

*** cdt cvs genie on behalf of mkhouzam ***
Bug 315490: Update JTag launch to set the default launch delegate more efficiently.

[*] GDBJtagDSFCMainTab.java 1.2 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/jtag/org.eclipse.cdt.debug.gdbjtag.ui/src/org/eclipse/cdt/debug/gdbjtag/internal/ui/GDBJtagDSFCMainTab.java?root=Tools_Project&r1=1.1&r2=1.2

[*] Activator.java 1.6 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/jtag/org.eclipse.cdt.debug.gdbjtag.ui/src/org/eclipse/cdt/debug/gdbjtag/ui/Activator.java?root=Tools_Project&r1=1.5&r2=1.6

Comment 4 John Cortell

2010-06-22 12:12:01 EDT

Nice solution.