From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark H Weaver Subject: Re: [PATCH] Improve shell script headers and pre-inst-env handling Date: Tue, 12 Feb 2013 17:44:40 -0500 Message-ID: <87obfpqhif.fsf@tines.lan> References: <87haliz4nt.fsf@tines.lan> <87fw11v83n.fsf@gnu.org> <87d2w5xtg5.fsf@tines.lan> <87y5etnqyj.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([208.118.235.92]:47032) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1U5ObD-0005qv-5y for bug-guix@gnu.org; Tue, 12 Feb 2013 17:45:17 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1U5ObB-00035C-6L for bug-guix@gnu.org; Tue, 12 Feb 2013 17:45:15 -0500 In-Reply-To: <87y5etnqyj.fsf@gnu.org> ("Ludovic \=\?utf-8\?Q\?Court\=C3\=A8s\=22'\?\= \=\?utf-8\?Q\?s\?\= message of "Tue, 12 Feb 2013 22:48:52 +0100") List-Id: Bug reports for GNU Guix List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-guix-bounces+gcggb-bug-guix=m.gmane.org@gnu.org Sender: bug-guix-bounces+gcggb-bug-guix=m.gmane.org@gnu.org To: Ludovic =?utf-8?Q?Court=C3=A8s?= Cc: bug-guix@gnu.org ludo@gnu.org (Ludovic Court=C3=A8s) writes: > Mark H Weaver skribis: > >> ludo@gnu.org (Ludovic Court=C3=A8s) writes: >>> Honestly, I wouldn=E2=80=99t worry about the propagation of $GUILE_LOAD= _PATH & >>> co. to subprocesses, because we know there=E2=80=99s none anyway. >> >> That policy will lead to future where libguile-using programs break in >> random ways when they happen to be subprocesses of each other. > > I agree in general with your feeling. > > However, in that case, we know that these command-line tools are just > wrappers around our Scheme APIs, and that they won=E2=80=99t ever launch = any > program (programs are a thing of the past; procedures are the future). > So it just seemed safe to me to do that in this particular case. > > What do you think? Ah, okay, I didn't realize that. When you said "we know there's none anyway", I thought you meant "no subprocesses that use Guile", but I guess you meant "no subprocesses at all". I guess guix-daemon is the only one with subprocesses, and by the time that's written in Guile hopefully Guile will have a command-line option to augment %load-compiled-path. In that case, I withdraw my proposal. I'll make a new patch. > (BTW, rather than $GUIX_UNINSTALLED, it just occurred to me that > $GUIX_LOAD_PATH would do just as well while being more generic and > easier to implement/use.) Sounds good. Mark