Skip to content
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.

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

Update requirements necessary on Darwin platforms #33

Merged
merged 1 commit into from
Apr 15, 2020
Merged

Conversation

boydj
Copy link
Contributor

@boydj boydj commented Apr 15, 2020

Should fix "No enumerators available" - on macOS #27, I have not tested yet. However manually installing Cython and then pyobjus makes get_monitors() return successfully on OS X.

from screeninfo import get_monitors, Enumerator
for m in get_monitors(Enumerator.OSX):
... print(str(m))
...
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.7/site-packages/screeninfo/screeninfo.py", line 29, in get_monitors
return _get_monitors(enumerator)
File "/usr/local/lib/python3.7/site-packages/screeninfo/screeninfo.py", line 19, in _get_monitors
return list(ENUMERATOR_MAP[enumerator].enumerate_monitors())
File "/usr/local/lib/python3.7/site-packages/screeninfo/enumerators/osx.py", line 7, in enumerate_monitors
from pyobjus import autoclass
ModuleNotFoundError: No module named 'pyobjus'

Fixes "No enumerators available" - on macOS rr-#27
@rr- rr- merged commit 8dde392 into rr-:master Apr 15, 2020
@rr-
Copy link
Owner

rr- commented Apr 15, 2020

Thanks, released as 0.6.4 on PyPI

boydj added a commit to boydj/screeninfo that referenced this pull request Apr 15, 2020
Now I've actually tested the code change I submitted as untested in PR rr-#33 which didn't do anything :) This has been tested and works to install Cython and pyobjus as requirements.
rr- pushed a commit that referenced this pull request Apr 15, 2020
Now I've actually tested the code change I submitted as untested in PR #33 which didn't do anything :) This has been tested and works to install Cython and pyobjus as requirements.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants