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

RTIMULib copy should be optional #88

Open
teuteuguy opened this issue Dec 7, 2018 · 3 comments
Open

RTIMULib copy should be optional #88

teuteuguy opened this issue Dec 7, 2018 · 3 comments

Comments

@teuteuguy
Copy link

Copying the RTIMULib to the home folder requires access to the home folder, which in some cases, ie. running as root can become complex.

This should be a setting on load of the lib, to choose whether or not to copy the RTIMULib file or not.

Event, be able to provide path to file directly.

@XECDesign
Copy link
Contributor

In general, you should be guaranteed access to your home directory, if nowhere else. But yes, flexibility wouldn't hurt.

@J-Hurtta
Copy link

J-Hurtta commented Jan 30, 2020

Indeed. I'm trying to use python-sense-hat on on a limited access case (Home Assistant on Hass.io).

As the RTIMULib is not listed as a dependency, it is not installed on my device and thus I cannot use my sense hat.

@rzr
Copy link

rzr commented May 6, 2020

or at least requierement should be mentionned somewhere

Hints:
#58 (comment)
https://github.com/rzr/sense-hat-webthing/blob/master/requirements.txt

rzr added a commit to rzr/sense-hat-webthing that referenced this issue May 10, 2020
RTIMULib dep is missing from sense_hat so it's added here,
until it gets optional.

Relate-to: astro-pi/python-sense-hat#88
Signed-off-by: Philippe Coval <[email protected]>
Change-Id: Ibc222fdb9348b31a9a903ce2931a30068ec13fb7
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

No branches or pull requests

4 participants