Bug 14326

Summary: [PATCH] pkill: add -e to display the name and PID of the process being killed
Product: Busybox Reporter: Louis Sautier <sautier.louis>
Component: OtherAssignee: unassigned
Status: RESOLVED FIXED    
Severity: enhancement CC: busybox-cvs
Priority: P5    
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: Linux   
Host: Target:
Build:
Attachments: 0001-pkill-add-e-to-display-the-name-and-PID-of-the-proce.patch

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