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

Bug 333034

Summary: "Select Process" dialog in "Attach to Application" launcher should print available processes more verbosely
Product: [Tools] CDT Reporter: Tim Black <timblaktu>
Component: cdt-debug-dsf-gdbAssignee: Simeon Andreev <simeon.danailov.andreev>
Status: RESOLVED FIXED QA Contact: Marc Khouzam <marc.khouzam>
Severity: enhancement    
Priority: P3 CC: cdtdoug, dalexiev, jens.elmenthaler, jonah, loskutov, onurakdemir1, pawel.1.piech, siliconarm, simeon.danailov.andreev
Version: 8.0   
Target Milestone: 9.6.0   
Hardware: PC   
OS: Linux   
See Also: https://git.eclipse.org/r/128899
https://bugs.eclipse.org/bugs/show_bug.cgi?id=538785
https://bugs.eclipse.org/bugs/show_bug.cgi?id=538849
https://git.eclipse.org/c/cdt/org.eclipse.cdt.git/commit/?id=094543644b409d3fd939df9202373028c9fdefe0
Whiteboard:
Attachments:
Description Flags
Screenshot showing the dialog with change 128899 and without.
none
After -- showing lower pane none

Description Tim Black CLA 2010-12-21 14:03:24 EST
Build Identifier: M20100909-0800

It would be useful if the process dialog in eclipse included command line parameters. This would allow developers to more easily select the correct process without having to memorize pids or go to a terminal and cross-reference the "ps" output. I don't see any need for new options. It looks like the "Select Process" dialog has a pane at the bottom where the selected process is displayed verbosely. Currently, on my system, this is just the full path of the process executable. Since you've already, elegantly, created enough real-estate there for a verbose display, why not be verbose about it?

Reproducible: Always

Steps to Reproduce:
1.Launch an "Attach to Application" session.
2.Notice that processes that are the same executable/path but have different command-line parameters cannot be differentiated between.
Comment 1 Eclipse Genie CLA 2018-09-07 06:44:34 EDT
New Gerrit change created: https://git.eclipse.org/r/128899
Comment 2 Simeon Andreev CLA 2018-09-07 06:46:54 EDT
Created attachment 275739 [details]
Screenshot showing the dialog with change 128899 and without.
Comment 3 Simeon Andreev CLA 2018-09-07 06:49:08 EDT
We have a change from Jens Elmenthaler in our own R&D CDT tooling, which adds the process command line arguments to the dialog in question.

We would like to propagate this change to CDT, to simplify our tooling distribution process.

I've created: https://git.eclipse.org/r/#/c/128899/

Would be great of a CDT developer could take a look at this change.
Comment 4 Jonah Graham CLA 2018-09-07 16:41:21 EDT
Created attachment 275749 [details]
After -- showing lower pane
Comment 6 Jonah Graham CLA 2018-09-14 06:10:11 EDT
Thanks for the improvement Simeon.

Before I close this bug I would like it if you could put all your improvements in the CDT New and Noteworthy for CDT 9.6: https://wiki.eclipse.org/CDT/User/NewIn96
Comment 7 Simeon Andreev CLA 2018-09-14 06:24:58 EDT
Thanks for merging, credit for the patch goes to Jens Elmenthaler.

I'll update the New & Noteworthy, hopefully today or next Monday.
Comment 8 Simeon Andreev CLA 2018-09-14 07:57:35 EDT
Please check https://wiki.eclipse.org/CDT/User/NewIn96#Debug and let me know if there is something to improve.
Comment 9 Jonah Graham CLA 2018-09-14 09:14:44 EDT
Looks great, thanks for contribution.