From 7a8ae85b5aaf52f8e799042df07fa33478b3e9ba Mon Sep 17 00:00:00 2001 From: Roland Rossgotterer Date: Tue, 11 Dec 2018 17:16:24 +0100 Subject: [PATCH] Do not require root privileges when installing on MacOS Requiring general root privileges raised some trust concerns by MacOS users. This reverts the latest changes to require root to install DBus machine-id into /etc. Now the user has to do that manually via sudo. A check will be added before running into an assert. --- installer/installbuilder/project.xml | 21 +++++++++++++-------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/installer/installbuilder/project.xml b/installer/installbuilder/project.xml index 1382f4e50..4b75e4a47 100644 --- a/installer/installbuilder/project.xml +++ b/installer/installbuilder/project.xml @@ -108,15 +108,23 @@ it and relaunch the installer. ${installdir}/bin/dbus-uuidgen - --ensure=/etc/machine-id + --ensure=${installdir}/share/dbus-1/machine-id osx + + {user_home_directory}/Library/LaunchAgents + + + osx + + + - /Library/LaunchAgents/org.freedesktop.dbus-session.plist + {user_home_directory}/Library/LaunchAgents/org.freedesktop.dbus-session.plist utf-8 @@ -151,8 +159,7 @@ it and relaunch the installer. launchctl - load -w /Library/LaunchAgents/org.freedesktop.dbus-session.plist - ${env(USER)} + load -w ${user_home_directory}/Library/LaunchAgents/org.freedesktop.dbus-session.plist osx @@ -232,7 +239,7 @@ it and relaunch the uninstaller. - /Library/LaunchAgents/org.freedesktop.dbus-session.plist + ${user_home_directory}/Library/LaunchAgents/org.freedesktop.dbus-session.plist osx @@ -240,7 +247,7 @@ it and relaunch the uninstaller. - /etc/machine-id + ${installdir}/share/dbus-1/machine-id osx @@ -306,7 +313,6 @@ it and relaunch the uninstaller. open ${Installdir}/bin/swiftlauncher.app --args -i --bootstrapurl https://datastore.swift-project.org/shared/ & Run swift launcher wizard - ${env(USER)} osx @@ -351,7 +357,6 @@ it and relaunch the uninstaller. 430 osx - 1 430