From: Jesse Gibbons <jgibbons2357@gmail.com>
To: Mark H Weaver <mhw@netris.org>, Tobias Geerinckx-Rice <me@tobias.gr>
Cc: guix-devel@gnu.org
Subject: Re: 01/01: services: Add ‘/usr/bin/env’ special file.
Date: Fri, 06 Sep 2019 23:05:52 -0600 [thread overview]
Message-ID: <6384138c4454d584c43e0a6144ed18bf458c551a.camel@gmail.com> (raw)
In-Reply-To: <874l1p80zr.fsf@netris.org>
On Fri, 2019-09-06 at 19:21 -0400, Mark H Weaver wrote:
> Hi Tobias,
>
> Tobias Geerinckx-Rice <me@tobias.gr> writes:
>
> > Christopher,
> >
> > Christopher Baines 写道:
> > > This seems to me like quite a big change, and I'd be interested
> > > in
> > > knowing what your motivation was [1]?
> >
> > It's not, really. It's equivalent to the impure /bin/sh that Guix
> > Systems already provide, but actually useful: ‘use #!/usr/bin/env,
> > not
> > #!/bin/sh!’ was already a mantra when I wrote my first shell script
> > —
> > and I'm not that young.
> >
> > There's no good argument for not being able to run the vast
> > majority
> > of well-written scripts, out of the box, on Guix Systems.
> >
> > I hope that's sufficient, but I don't think it needed to go in the
> > commit message.
>
> This should have been discussed more widely before pushing to
> 'master'.
> It is not a new idea. We talked about it long ago, and decided that
> it
> should *not* be included. I don't have the energy right now to
> restate
> the arguments or find the old discussions.
>
> Mark
>
Here's a post with what I think is a good argument against adding
/usr/bin/env. I think the standard patch-shebang phase does a good job
at preventing the potential issue, but the argument still applies to
scripts generated by make, as I learned while attempting to port
pysolfc. I recommend you read the thread too.
https://lists.gnu.org/archive/html/guix-devel/2015-11/msg00505.html
next prev parent reply other threads:[~2019-09-07 5:06 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20190906102509.28951.2772@vcs0.savannah.gnu.org>
[not found] ` <20190906102510.002BE21324@vcs0.savannah.gnu.org>
2019-09-06 10:36 ` 01/01: services: Add ‘/usr/bin/env’ special file Christopher Baines
2019-09-06 10:44 ` pelzflorian (Florian Pelz)
2019-09-06 10:47 ` pelzflorian (Florian Pelz)
2019-09-06 15:54 ` Tobias Geerinckx-Rice
2019-09-06 23:21 ` Mark H Weaver
2019-09-07 5:05 ` Jesse Gibbons [this message]
2019-09-07 7:52 ` Tobias Geerinckx-Rice via Development of GNU Guix and the GNU System distribution.
2019-09-07 15:33 ` Jesse Gibbons
2019-09-07 10:06 ` Tobias Geerinckx-Rice
2019-09-07 15:03 ` Jesse Gibbons
2019-09-08 21:48 ` Ludovic Courtès
2019-09-08 23:53 ` Jesse Gibbons
2019-09-08 22:19 ` Tobias Geerinckx-Rice
2019-09-06 23:47 ` Mark H Weaver
2019-09-07 8:54 ` Tobias Geerinckx-Rice
2019-09-07 14:41 ` Marius Bakke
2019-09-07 17:56 ` Ricardo Wurmus
2019-09-08 11:55 ` Konrad Hinsen
2019-09-08 18:31 ` Hartmut Goebel
2019-09-08 22:07 ` Ludovic Courtès
2019-09-09 7:01 ` Bengt Richter
2019-09-09 8:13 ` Ludovic Courtès
2019-09-09 1:37 ` Chris Marusich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6384138c4454d584c43e0a6144ed18bf458c551a.camel@gmail.com \
--to=jgibbons2357@gmail.com \
--cc=guix-devel@gnu.org \
--cc=me@tobias.gr \
--cc=mhw@netris.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.