Bug 14326 - [PATCH] pkill: add -e to display the name and PID of the process being killed
Summary: [PATCH] pkill: add -e to display the name and PID of the process being killed
Status: RESOLVED FIXED
Alias: None
Product: Busybox
Classification: Unclassified
Component: Other (show other bugs)
Version: unspecified
Hardware: All Linux
: P5 enhancement
Target Milestone: ---
Assignee: unassigned
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-11-05 00:16 UTC by Louis Sautier
Modified: 2022-10-14 12:36 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:

Attachments
0001-pkill-add-e-to-display-the-name-and-PID-of-the-proce.patch (2.52 KB, patch)
2021-11-05 00:16 UTC, Louis Sautier 		Details
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 Louis Sautier 2021-11-05 00:16:49 UTC 
Created attachment 9146 [details]
0001-pkill-add-e-to-display-the-name-and-PID-of-the-proce.patch

Hello,
I found this pkill feature very useful so I implemented it. Please let me know if the attached patch is OK.
Comment 1 Denys Vlasenko 2022-01-11 13:23:53 UTC 
pkill from procps-ng-3.3.15 has no such option. Do you propose it as an extension?
Comment 2 Louis Sautier 2022-01-11 15:11:51 UTC 
This seems weird to me, this option was added years ago in https://gitlab.com/procps-ng/procps/-/commit/bc974b3c335c354192e510b34928877dad206198

I tried on Debian Buster which has the version you mentioned:
root@9dce64e0e00d:/# pkill -h |& grep echo
 -e, --echo                display what is killed
root@9dce64e0e00d:/# pkill --version
pkill from procps-ng 3.3.15
Comment 3 Louis Sautier 2022-10-11 23:17:29 UTC 
(In reply to Denys Vlasenko from comment #1)
Hello Denys, did you see my previous reply?
Comment 4 Denys Vlasenko 2022-10-14 12:36:16 UTC 
Fixed in git