Replace ps for pgrep to list evobackup child procs #29

Merged
Ghost merged 1 commit from pgrep into master 2019-09-05 17:48:12 +02:00
First-time contributor

The command being used to identify evobackup child processes isn't
supported on OpenBSD.

$ ps h --ppid "${pid}" -o pid | tr -s '\n' ' '
ps: unknown option -- -
usage: ps [-AaceHhjkLlmrSTuvwx] [-M core] [-N system] [-O fmt] [-o fmt] [-p pid] [-t tty] [-U username] [-W swap]

That said both Linux and OpenBSD support the use of pgrep.

$ pgrep -P "${pid}"

I suggest to replace ps for pgrep in that case then.

The command being used to identify evobackup child processes isn't supported on OpenBSD. $ ps h --ppid "${pid}" -o pid | tr -s '\n' ' ' ps: unknown option -- - usage: ps [-AaceHhjkLlmrSTuvwx] [-M core] [-N system] [-O fmt] [-o fmt] [-p pid] [-t tty] [-U username] [-W swap] That said both Linux and OpenBSD support the use of pgrep. $ pgrep -P "${pid}" I suggest to replace ps for pgrep in that case then.
jlecour approved these changes 2019-09-05 12:00:07 +02:00
jlecour left a comment
Owner

Looks good to me

Looks good to me
Ghost closed this pull request 2019-09-05 17:48:08 +02:00
jlecour deleted branch pgrep 2019-09-07 13:16:05 +02:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: evolix/evobackup#29
No description provided.