commit 37374732e80b3555d441ce053343f25970160aff
author Dan Callaghan <dcallagh@google.com> Thu Jul 21 15:33:53 2022 +1000
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Jul 21 12:09:20 2022 +0000
tree 8d490dc9f3a47854789b65dc991738b430db78d5
parent c3bd2b6718dc7f62beeffbc44590a2c60cf71f0d

scripts/setup: add user to plugdev group inside the chroot

BUG=none
TEST=scripts/setup  # after removing self from plugdev

Change-Id: Ibe286665a2a3ee5cdde6b1aedc01a3977ee09a85
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/hps-firmware/+/3778948
Tested-by: Dan Callaghan <dcallagh@chromium.org>
Commit-Queue: Dan Callaghan <dcallagh@chromium.org>
Reviewed-by: David Lattimore <dml@chromium.org>

scripts/setup

diff --git a/scripts/setup b/scripts/setup
index c54f19c..05ec1a7 100755
--- a/scripts/setup
+++ b/scripts/setup
@@ -39,4 +39,17 @@
     fi
 fi
 
+# Make sure $USER is in the plugdev group inside the chroot, for accessing the
+# MCP2221 and FT4232H USB devices.
+
+if ! id -Gn | grep -qw plugdev ; then
+    echo "${USER} is not in plugdev group" >&2
+    echo -n "Add [y]/n? " >&2
+    read -r answer
+    if [ "${answer}" == "y" ] || [ -z "${answer}" ]; then
+        sudo usermod -aG plugdev "${USER}"
+        echo "Re-enter chroot for group changes to take effect." >&2
+    fi
+fi
+
 echo "Reminder: also run scripts/setup-proto2-on-host outside the chroot if necessary." >&2