rdhyee / py-applescript

An easy-to-use Python wrapper for NSAppleScript, allowing Python scripts to communicate with AppleScripts and AppleScriptable applications.
Creative Commons Zero v1.0 Universal
140 stars 7 forks source link

Required pyobjc dependency can be reduced to only a few #7

Closed mhucka closed 3 years ago

mhucka commented 3 years ago

setup.py currently requires pyobj, but it does not appear to need all of pyobjc. It only appears to need the following:

pyobjc-core
pyobjc-framework-AppleScriptKit
pyobjc-framework-AppleScriptObjC

If this is correct, it would be beneficial to change setup.py to limit what it requires. The downside of doing a blanket pyobjc requirement is not only that it is relatively time-consuming installation step: there is at least one package (pyobjc-framework-PubSub) that ends up conflicting with a Python package (pypubsub) because both install files into site-packages/PubSub.

glyph commented 3 years ago

I translated this into a really simple PR, although I haven't tested this to see if it causes problems :)

rdhyee commented 3 years ago

@mhucka @glyph Thanks for reporting this issue and translating the issue into a PR. I'm sorry that I just noticed this PR. I'll take a look at it this weekend.