Remove SetUID Usage

debian/DEBIAN/control

@@ -4,4 +4,4 @@ Maintainer: TheBrokenRail <connor24nolan@live.com>
 Description: Fun with Blocks
 Homepage: https://www.minecraft.net/en-us/edition/pi
 Architecture: amd64
-Depends: docker.io, docker-compose, virgl-server, zenity
+Depends: docker.io, docker-compose, virgl-server, zenity, policykit-1

debian/usr/bin/minecraft-pi

@@ -2,11 +2,16 @@
 
 set -e
 
-export FEATURES="$(zenity --class minecraft-pi --list --checklist --column 'Enabled' --column 'Feature' FALSE 'Touch GUI' FALSE 'Survival Mode' FALSE 'Fix Bow & Arrow' FALSE 'Fix Attacking' FALSE 'Mob Spawning')"
+FEATURES="$(zenity --class minecraft-pi --list --checklist --column 'Enabled' --column 'Feature' FALSE 'Touch GUI' FALSE 'Survival Mode' FALSE 'Fix Bow & Arrow' FALSE 'Fix Attacking' FALSE 'Mob Spawning')"
+export FEATURES
 
 virgl_test_server &
 PID="$!"
 
+if ! id -Gn "$(whoami)" | grep '\bdocker\b' > /dev/null; then
+    pkexec adduser "$(whoami)" docker
+fi
+
 xhost local:root
 
 DOCKER_COMPOSE="docker-compose -f /usr/share/minecraft-pi/docker-compose.yml"

scripts/package.sh

@@ -5,9 +5,6 @@ set -e
 # Docker Messes With SetGID
 chmod -R g-s debian
 
-# Allow minecraft-pi Script To Use Docker
-chmod u+s debian/usr/bin/minecraft-pi
-
 # Clean out Directory
 rm -rf out
 mkdir out