Split usb and ble requirements#21
Conversation
|
thanks for this! the check if these modules are available where intended for the case that pyoprf is installed via distro package managers, not via pip. i do see how this is beneficial to do also for pip installs. i have a question, some pip installs allow to say |
|
I am also a bit new to this. In our project we install it through pip and I saw a lot of extra packages installed because of the requirements of these packages, so this is a 'nice to have' cleanup. I don't have any experience with the extras through distro package managers. Checking this documentation: https://setuptools.pypa.io/en/latest/userguide/dependency_management.html#optional-dependencies For the pip part, with this PR it is possible to do We can add an |
|
wow. that would be super-plus-awesome <3 |
|
Added the |
|
|
much obliged. <3 |



There were already checks if the needed package is available:
liboprf/python/pyoprf/multiplexer.py
Lines 7 to 14 in 321eaad
I have moved the packages in
setup.pyto theextras_requireoption.I think this is a breaking change as those dependencies are now not installed by default anymore.
If something needs to be changed, let me know.