Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 333034 - "Select Process" dialog in "Attach to Application" launcher should print available processes more verbosely
Summary: "Select Process" dialog in "Attach to Application" launcher should print avai...
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-dsf-gdb (show other bugs)
Version: 8.0   Edit
Hardware: PC Linux
: P3 enhancement with 2 votes (vote)
Target Milestone: 9.6.0   Edit
Assignee: Simeon Andreev CLA
QA Contact: Marc Khouzam CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-21 14:03 EST by Tim Black CLA
Modified: 2018-09-14 09:14 EDT (History)
9 users (show)

See Also:

Attachments
Screenshot showing the dialog with change 128899 and without. (56.46 KB, image/png)
2018-09-07 06:46 EDT, Simeon Andreev CLA 		no flags Details
After -- showing lower pane (76.34 KB, image/png)
2018-09-07 16:41 EDT, Jonah Graham CLA 		no flags Details
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 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.