From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:470:142:3::10]:38543) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1hy0GV-0003fD-2b for guix-patches@gnu.org; Wed, 14 Aug 2019 16:53:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hy0GU-0005Et-2S for guix-patches@gnu.org; Wed, 14 Aug 2019 16:53:02 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:40971) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1hy0GT-0005Ep-Vm for guix-patches@gnu.org; Wed, 14 Aug 2019 16:53:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1hy0GT-0000kv-Sp for guix-patches@gnu.org; Wed, 14 Aug 2019 16:53:01 -0400 Subject: [bug#36957] [PATCH v2] machine: Allow non-root users to deploy. Resent-Message-ID: References: <87a7cl3zyy.fsf@sdf.lonestar.org> <87sgqcobds.fsf@dustycloud.org> <87pnlgjymv.fsf_-_@sdf.lonestar.org> <87y304vyyo.fsf@elephly.net> <87tvarjtgw.fsf@sdf.lonestar.org> From: Christopher Lemmer Webber In-reply-to: <87tvarjtgw.fsf@sdf.lonestar.org> Date: Wed, 14 Aug 2019 16:52:45 -0400 Message-ID: <87h86jxyea.fsf@dustycloud.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: "Jakob L. Kreuze" Cc: 36957@debbugs.gnu.org Jakob L. Kreuze writes: >> (What happens if /run/setuid-programs/sudo is not available on the >> target machine?) > > I'm a bit short on time before boarding this flight, so I can't test it > out at the moment, but I'm pretty sure the "failed to run..." message > condition would be thrown. I'll check and get back to you. Check, and if that's good, let's merge this in the patch series tomorrow. >> I=E2=80=99m just stumbling upon =E2=80=9Csocket-name=E2=80=9D. =E2=80= =9C/var/guix=E2=80=9D is not guaranteed to >> be the localstatedir. It would be better to use (guix config) to >> determine the configured value. >> >> This doesn=E2=80=99t block this patch, of course, but it would be good t= o change >> this in the future. > > Right, yeah. I may submit a separate patch for it shortly since it > should be a simple change. See if you can get it in the patch series (as the last patch)! If you can't, file a bug, and let's not block on it.