From 6738fdbab796a46339fd27671f45da904a3cce14 Mon Sep 17 00:00:00 2001 From: Ralph Sennhauser Date: Sat, 15 Feb 2025 08:16:48 +0100 Subject: [PATCH] Remove root check in update-workspace.sh If you run as root then created files will be owned by root, this is expected behaviour and not messing with permissions as stated in the error message. Running in a container the root user may map to the user starting the container while all other users would need mapping to be able to work with a bind mounted a checkout. Further Debian patches out the root check to be able to build on their builder. Given the above remove the check. Signed-off-by: Ralph Sennhauser --- build/workspaces/update-workspaces.sh | 5 ----- 1 file changed, 5 deletions(-) diff --git a/build/workspaces/update-workspaces.sh b/build/workspaces/update-workspaces.sh index c6fe4ea3ec..2a5fb381fe 100755 --- a/build/workspaces/update-workspaces.sh +++ b/build/workspaces/update-workspaces.sh @@ -1,10 +1,5 @@ #!/bin/sh -if [ "$(id -u)" = "0" ]; then - echo "Running as root will mess up file permissions. Aborting ..." 1>&2 - exit 1 -fi - die() { echo ERROR: "$*"