Fix the error that not all Linux hosts have pgrep installed

google / mobly

E2E test framework for tests with complex environment requirements.

https://github.com/google/mobly

Apache License 2.0

621 stars 174 forks source link

Fix the error that not all Linux hosts have pgrep installed #921

Closed mhaoli closed 1 month ago

mhaoli commented 1 month ago

In #920, we modified utils.stop_standing_subprocess to use pgrep to collect subprocess tree. While it turned out that not all Linux hosts has pgrep installed.

Thus, this PR changes to use pgrep for Mac OS while ps for Linux. I verified that most Mac OS hosts have pgrep installed, otherwise we can let users install it.

This change is