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

Add udev rules #590

Merged
merged 2 commits into from
Oct 20, 2018
Merged

Add udev rules #590

merged 2 commits into from
Oct 20, 2018

Conversation

rpavlik
Copy link
Member

@rpavlik rpavlik commented Jul 9, 2018

Not sure why I didn't commit these long ago - most of them have been sitting around in a Gist since 2015.

No build system touches them yet but this at least puts them in the source tree.

Question - is MODE="0660" GROUP="plugdev" the best thing to do to provide USB access? It's different from what the Vive linux support does - https://github.com/ValveSoftware/SteamVR-for-Linux#usb-device-requirements

This had been published externally for a long time, but
for some reason I never put it in the repo. Doesn't get
installed anywhere, but exists now for ease of packaging
and use, etc.
Just like the other rules file, this one doesn't get installed
(yet) but it's here for installers/packagers to use.
@rpavlik rpavlik requested a review from godbyk July 9, 2018 19:48
@godbyk
Copy link
Contributor

godbyk commented Jul 10, 2018

Can you think of any benefit to restricting access to the device to a subset of users on a system?

The Debian wiki says that the plugdev group "Allows members to mount (only with the options nodev and nosuid, for security reasons) and umount removable devices through pmount."

The Ubuntu wiki says the group provides "access external storage devices" and that "users in the plugdev group can send commands to HAL (this is probably deprecated)."

Other distributions seem to lack this group, so it's probably easiest to just omit the group entirely from the udev rule, allowing all users to access the devices.

@rpavlik
Copy link
Member Author

rpavlik commented Jul 10, 2018 via email

@rpavlik
Copy link
Member Author

rpavlik commented Oct 20, 2018

This is where that "Move discussion to new issue" button in GitLab would be handy :D Just spent 5 minutes looking for this, only to discover I had never merged it.

@rpavlik rpavlik merged commit 35e0967 into master Oct 20, 2018
@rpavlik rpavlik deleted the add-udev-rules branch October 20, 2018 18:11
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