diff options
| author | Ivan Metlushko <[email protected]> | 2020-06-24 09:08:41 +0700 |
|---|---|---|
| committer | Ivan Metlushko <[email protected]> | 2020-06-24 09:29:50 +0700 |
| commit | 8cf9d15b823d91d2a74fc83832fccca2219342c9 (patch) | |
| tree | 410b4dc14e83a687dbe820550dd1a61bcb911ea6 /test/functional/test_runner.py | |
| parent | Merge #19240: build: macOS toolchain simplification and bump (diff) | |
| download | discoin-8cf9d15b823d91d2a74fc83832fccca2219342c9.tar.xz discoin-8cf9d15b823d91d2a74fc83832fccca2219342c9.zip | |
test: use pgrep for better compatibility
pidof is not available on BSD system, while pgrep is present on BSD, Linux and macOS
Diffstat (limited to 'test/functional/test_runner.py')
| -rwxr-xr-x | test/functional/test_runner.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/test/functional/test_runner.py b/test/functional/test_runner.py index 41f9bde18..2fa48006f 100755 --- a/test/functional/test_runner.py +++ b/test/functional/test_runner.py @@ -396,11 +396,12 @@ def run_tests(*, test_list, src_dir, build_dir, tmpdir, jobs=1, enable_coverage= args = args or [] # Warn if bitcoind is already running - # pidof might fail or return an empty string if bitcoind is not running try: - if subprocess.check_output(["pidof", "bitcoind"]) not in [b'']: + # pgrep exits with code zero when one or more matching processes found + if subprocess.run(["pgrep", "-x", "bitcoind"], stdout=subprocess.DEVNULL).returncode == 0: print("%sWARNING!%s There is already a bitcoind process running on this system. Tests may fail unexpectedly due to resource contention!" % (BOLD[1], BOLD[0])) - except (OSError, subprocess.SubprocessError): + except OSError: + # pgrep not supported pass # Warn if there is a cache directory |