From: Daan De Meyer <daan.j.demeyer@gmail.com>
Date: Tue, 5 Mar 2024 12:03:10 +0000 (+0100)
Subject: Don't explicitly run sync scripts as invoking user anymore
X-Git-Tag: v21~7
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=17f6bf0055ac57c925f889f7d8e234cf56f301ba;p=thirdparty%2Fmkosi.git

Don't explicitly run sync scripts as invoking user anymore

We now change user to the invoking user when running sync scripts so
there's no need to explicitly run sync scripts as the invoking user
anymore.
---

diff --git a/mkosi/__init__.py b/mkosi/__init__.py
index d949d4018..bbc0f8659 100644
--- a/mkosi/__init__.py
+++ b/mkosi/__init__.py
@@ -454,11 +454,6 @@ def run_sync_scripts(context: Context) -> None:
                     env=env | context.config.environment,
                     stdin=sys.stdin,
                     sandbox=context.sandbox(network=True, options=options, scripts=hd),
-                    # Make sure we run as the invoking user when we're running as root so that files are owned by the
-                    # right user. bubblewrap will automatically map the running user to root in the user namespace it
-                    # creates.
-                    user=INVOKING_USER.uid,
-                    group=INVOKING_USER.gid,
                 )