CVS commit: src/usr.bin/pkill

2022-07-02 Thread Andrew Doran
Module Name:src Committed By: ad Date: Sat Jul 2 20:50:26 UTC 2022 Modified Files: src/usr.bin/pkill: pkill.c Log Message: Cosmetic tweak. No functional change. To generate a diff of this commit: cvs rdiff -u -r1.31 -r1.32 src/usr.bin/pkill/pkill.c Please note that di

CVS commit: src/usr.bin/pkill

2022-07-02 Thread Andrew Doran
Module Name:src Committed By: ad Date: Sat Jul 2 20:50:26 UTC 2022 Modified Files: src/usr.bin/pkill: pkill.c Log Message: Cosmetic tweak. No functional change. To generate a diff of this commit: cvs rdiff -u -r1.31 -r1.32 src/usr.bin/pkill/pkill.c Please note that di

CVS commit: src/usr.bin/pkill

2022-10-29 Thread Simon Burge
Module Name:src Committed By: simonb Date: Sat Oct 29 08:17:16 UTC 2022 Modified Files: src/usr.bin/pkill: pkill.1 pkill.c Log Message: Add a -q option for pgrep to not print anything; similar to grep -q. To generate a diff of this commit: cvs rdiff -u -r1.31 -r1.32 src/

CVS commit: src/usr.bin/pkill

2022-10-29 Thread Simon Burge
Module Name:src Committed By: simonb Date: Sat Oct 29 08:17:16 UTC 2022 Modified Files: src/usr.bin/pkill: pkill.1 pkill.c Log Message: Add a -q option for pgrep to not print anything; similar to grep -q. To generate a diff of this commit: cvs rdiff -u -r1.31 -r1.32 src/

Re: CVS commit: src/usr.bin/pkill

2010-12-05 Thread enami tsugutomo
> cvs rdiff -u -r1.25 -r1.26 src/usr.bin/pkill/pkill.c Looks like -l option is necessary for prenice also. enami.

Re: CVS commit: src/usr.bin/pkill

2012-11-20 Thread David Laight
On Tue, Nov 20, 2012 at 05:52:02PM -0500, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Tue Nov 20 22:52:02 UTC 2012 > > Modified Files: > src/usr.bin/pkill: pkill.c > > Log Message: > Don't use p_comm since it is only 16 characters long and you can fin

Re: CVS commit: src/usr.bin/pkill

2012-11-21 Thread Christos Zoulas
In article <20121121080326.gl...@snowdrop.l8s.co.uk>, David Laight wrote: >On Tue, Nov 20, 2012 at 05:52:02PM -0500, Christos Zoulas wrote: >> Module Name: src >> Committed By:christos >> Date:Tue Nov 20 22:52:02 UTC 2012 >> >> Modified Files: >> src/usr.bin/pkill: p

re: CVS commit: src/usr.bin/pkill

2012-11-21 Thread matthew green
> In article <20121121080326.gl...@snowdrop.l8s.co.uk>, > David Laight wrote: > >On Tue, Nov 20, 2012 at 05:52:02PM -0500, Christos Zoulas wrote: > >> Module Name: src > >> Committed By: christos > >> Date: Tue Nov 20 22:52:02 UTC 2012 > >> > >> Modified Files: > >>s

re: CVS commit: src/usr.bin/pkill

2012-11-21 Thread Christos Zoulas
On Nov 22, 4:39am, m...@eterna.com.au (matthew green) wrote: -- Subject: re: CVS commit: src/usr.bin/pkill | this seems like a fairly big semantic change to me. | | could you change it to take a new option to look in argv[0] instead | of p_comm? p_comm is not changeable by the user. You could

Re: CVS commit: src/usr.bin/pkill

2012-11-21 Thread David Laight
On Wed, Nov 21, 2012 at 12:43:20PM -0500, Christos Zoulas wrote: > On Nov 22, 4:39am, m...@eterna.com.au (matthew green) wrote: > -- Subject: re: CVS commit: src/usr.bin/pkill > > | this seems like a fairly big semantic change to me. > | > | could you change it to take a new

Re: CVS commit: src/usr.bin/pkill

2012-11-21 Thread Christos Zoulas
In article <20121121200309.gn...@snowdrop.l8s.co.uk>, David Laight wrote: >On Wed, Nov 21, 2012 at 12:43:20PM -0500, Christos Zoulas wrote: >> On Nov 22, 4:39am, m...@eterna.com.au (matthew green) wrote: >> -- Subject: re: CVS commit: src/usr.bin/pkill >> >&g

Re: CVS commit: src/usr.bin/pkill

2012-11-21 Thread Warner Losh
;>> -- Subject: re: CVS commit: src/usr.bin/pkill >>> >>> | this seems like a fairly big semantic change to me. >>> | >>> | could you change it to take a new option to look in argv[0] instead >>> | of p_comm? p_comm is not changeable by the us

Re: CVS commit: src/usr.bin/pkill

2012-11-21 Thread David Laight
On Wed, Nov 21, 2012 at 01:36:01PM -0700, Warner Losh wrote: > >> Is pkill() guaranteed to do an atomic traversal of the process list? > >> So it will kill something that keeps using fork() to change its pid. > > > > I don't think our libkvm offers that. > > How could you guarantee this short of

Re: CVS commit: src/usr.bin/pkill

2012-11-21 Thread Warner Losh
On Nov 21, 2012, at 1:55 PM, David Laight wrote: > On Wed, Nov 21, 2012 at 01:36:01PM -0700, Warner Losh wrote: Is pkill() guaranteed to do an atomic traversal of the process list? So it will kill something that keeps using fork() to change its pid. >>> >>> I don't think our libkvm off

Re: CVS commit: src/usr.bin/pkill

2012-11-21 Thread Christos Zoulas
On Nov 21, 1:36pm, i...@bsdimp.com (Warner Losh) wrote: -- Subject: Re: CVS commit: src/usr.bin/pkill | How could you guarantee this short of moving this into the kernel so you = | could do all the comparisons while keeping all forks from happening? You could not; this was more tongue-in-cheek

re: CVS commit: src/usr.bin/pkill

2012-11-22 Thread matthew green
> | this seems like a fairly big semantic change to me. > | > | could you change it to take a new option to look in argv[0] instead > | of p_comm? p_comm is not changeable by the user. > > You could already do this with -f. The point was to make the default case > behave intuitively. Adding an

Re: CVS commit: src/usr.bin/pkill

2012-11-22 Thread Christos Zoulas
In article <20404.1353628...@splode.eterna.com.au>, matthew green wrote: > >> | this seems like a fairly big semantic change to me. >> | >> | could you change it to take a new option to look in argv[0] instead >> | of p_comm? p_comm is not changeable by the user. >> >> You could already do thi