public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Eric Blake <eblake@redhat.com>
To: cygwin@cygwin.com
Subject: Re: [ANNOUNCEMENT] Updated: dash-0.5.9.1-1
Date: Thu, 23 Feb 2017 20:41:00 -0000	[thread overview]
Message-ID: <d00649dc-0be4-3b8a-5d10-84d58dd1c60e@redhat.com> (raw)
In-Reply-To: <74aa0c8b-c268-dbd4-88b7-cda78186f321@SystematicSw.ab.ca>


[-- Attachment #1.1: Type: text/plain, Size: 1620 bytes --]

On 02/23/2017 01:59 PM, Brian Inglis wrote:
>> If script succeeds, then yes sh is now a symlink to Dash, but only
>> for my system. I still have to use #!/bin/dash with my scripts,
>> because otherwise people with Bash -> sh will not get the speed
>> boost. The whole point of this thread is that it should be changed
>> for everyone.
> 
> And should be done using update_alternatives so it and associated 
> man pages can easily be changed if anyone suffers breakage.
> Some people may continue to need or prefer using bash in POSIX mode.

Or more likely, many people likely have pre-existing scripts wrongly
written as #!/bin/sh but which use bash-ism rather than portable
POSIX-specified shell (if it is portable, then bash vs. dash executing
the script shouldn't matter - but as Debian found out when they switched
to dash, there are a lot of non-portable scripts).  On the other hand,
Debian has already fixed much of the pre-packaged badness out there and
pushed fixes upstream where feasible, so these days it's not as hairy to
switch as when Debian first did it.

But part of me is still reluctant to make the switch as long as Fedora
still favors bash over dash.

One problem with using update_alternatives: the alternatives system
relies on symlinks, but Windows is not so smart about executing through
symlinks; are there people that will have a broken setup if /bin/sh is a
symlink rather than an actual executable (hard links are fine, but
symlinks is where I worry).

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]

  reply	other threads:[~2017-02-23 20:41 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-17 14:01 Eric Blake (cygwin)
2017-02-23  4:20 ` Steven Penny
2017-02-23 17:50   ` Andrey Repin
2017-02-23 19:46     ` Steven Penny
2017-02-23 19:59       ` Brian Inglis
2017-02-23 20:41         ` Eric Blake [this message]
2017-02-24  7:19           ` Yaakov Selkowitz
2017-02-23 23:44         ` Steven Penny
2017-02-24 14:32           ` Eric Blake
2017-02-24 19:19             ` Brian Inglis
2017-02-23 22:05       ` Andrey Repin
2017-02-23 23:01         ` Tony Kelman
2017-02-23 23:04           ` Eliot Moss
2017-02-23 23:12             ` Kenneth Wolcott
2017-02-23 23:30             ` Vince Rice
2017-02-23 23:35           ` Andrey Repin
2017-02-24  3:16           ` Larry Hall (Cygwin)
2017-02-24  3:18           ` Larry Hall (Cygwin)
2017-02-24  4:57             ` Steven Penny
2017-02-24 14:43               ` Eric Blake
2017-02-24 15:05                 ` Lee Dilkie
2017-02-25 16:55                 ` Steven Penny
2017-03-01 23:57                   ` Matt Seitz (matseitz)
2017-02-23 23:33         ` Steven Penny
2017-02-25 16:46       ` cyg Simple
2017-02-25 17:09         ` Steven Penny
2017-02-27 10:20           ` Csaba Raduly
2017-02-27 12:48             ` Steven Penny
2017-02-27 23:13           ` Duncan Roe
2017-02-28  0:51             ` Steven Penny
2017-02-28 20:52               ` cyg Simple
2017-02-28 21:43                 ` Steven Penny
2017-03-01 14:42                   ` cyg Simple
2017-03-02  0:22                     ` Steven Penny
2017-03-02  3:46                       ` Vince Rice
2017-03-02  5:27                         ` Steven Penny
2017-03-02  5:31                           ` Vince Rice
2017-03-02 12:36                             ` Steven Penny
2017-03-02 13:37                               ` Marco Atzeri
2017-03-02 14:29                                 ` Eric Blake
2017-03-02 16:16                                   ` Nellis, Kenneth (Conduent)
2017-03-02 17:28                                   ` Brian Inglis
2017-03-02 18:45                                     ` Eric Blake
2017-03-02 23:23                                       ` Brian Inglis
2017-03-02 18:28                                   ` Achim Gratz
2017-03-02 14:31                                 ` Brian Inglis

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=d00649dc-0be4-3b8a-5d10-84d58dd1c60e@redhat.com \
    --to=eblake@redhat.com \
    --cc=cygwin@cygwin.com \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).