Skip to content

Feature/dont assume user

Thomas Nadovich requested to merge feature/dont-assume-user into devel

Description

Removed references to user/group robot from setup script and debian postinst script so that they can be run on non-Barrett hardware, and can be accessed by all accounts on the PC.

What to focus on

  • I just deleted the chown from both scripts which results in all /opt/barrett/burt files still being owned by root. I also tightened up the permissions from 777 to just read and write, since that's all that's actually needed. Execute privileges for /opt/barrett/burt/bin get set elsewhere so I don't think I need to worry about that here.
  • This is obviously just a personal pet peeve, so no rush on the review here.

Merge request reports