Skip to content

Handle ZombieProcess and NoSuchProcess from psutil for OSX systems #1599

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
flipdazed wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Handle ZombieProcess and NoSuchProcess from psutil for OSX systems #1599

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3c00cb8
fix for Darwin systems that cont find a specific process as per https…
flipdazed Aug 21, 2022
b93be5f
added ZombieProcess as well
flipdazed Aug 21, 2022
a1a2553
changelog
flipdazed Aug 21, 2022
ff9b1d0
Merge branch 'master' into feature/NoSuchProcess
flipdazed Aug 21, 2022
11a4a05
better log title
flipdazed Aug 21, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Handle Vyper `>=0.3.4` coverage data generation breaking project compilation ([#1586](https://github.com/eth-brownie/brownie/pull/1586))
- Handle null value of `to` field in transaction receipt so that contract deploying with Anvil works properly ([#1573](https://github.com/eth-brownie/brownie/pull/1573))
- Avoid removing dependencies from the build ([#1564](https://github.com/eth-brownie/brownie/pull/1564))
- Handle `ZombieProcess` and `NoSuchProcess` from `psutil` for OSX systems ([#1599](https://github.com/eth-brownie/brownie/pull/1599))

## [1.19.0](https://github.com/eth-brownie/brownie/tree/v1.19.0) - 2022-05-29
### Added
Expand Down
7 changes: 6 additions & 1 deletion brownie/network/rpc/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -252,5 +252,10 @@ def _get_proc_pid(self, proc: psutil.Process) -> int:

def _find_proc_by_name(self, process_name: str) -> psutil.Process:
for proc in psutil.process_iter():
if process_name.lower() in proc.name().lower():
try:
proc_name = proc.name().lower()
except (psutil.NoSuchProcess, psutil.ZombieProcess):
continue

if process_name.lower() in proc_name:
return proc