This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v2] sotruss: convert from ksh to bash
- From: "Patrick 'P. J.' McDermott" <pj+sourceware-ml at pehjota dot net>
- To: "Joseph S. Myers" <joseph at codesourcery dot com>
- Cc: Mike Frysinger <vapier at gentoo dot org>, libc-alpha at sourceware dot org, Roland McGrath <roland at hack dot frob dot com>
- Date: Thu, 13 Mar 2014 20:31:28 -0400
- Subject: Re: [PATCH v2] sotruss: convert from ksh to bash
- Authentication-results: sourceware.org; auth=none
- References: <1390644580-25234-1-git-send-email-vapier at gentoo dot org> <1513046 dot Pq6yaohzbx at vapier> <20140313223005 dot 7D2937448F at topped-with-meat dot com> <3306042 dot flf76EZhzQ at vapier> <Pine dot LNX dot 4 dot 64 dot 1403132334470 dot 9004 at digraph dot polyomino dot org dot uk>
On 2014-03-13 19:36, Joseph S. Myers wrote:
> On Thu, 13 Mar 2014, Mike Frysinger wrote:
>
>> afaict, the only reason bash is required is for localization of the messages.
>> but even that works w/POSIX shell, albeit with slightly funky display -- you
>> get a literal $ in the output of each string.
>
> We know we want to avoid $"" (bug 13853) - some patches were posted in Nov
> 2012. I suppose those need revising to take account of any patch reviews
> and updating as needed.
Yes, I (the submitter of those patches) have been meaning to revisit
this. I may post an RFC and/or patches for this later in the 2.20 or
2.21 cycle, unless someone beats me to it.
As Mike notes, scripts with `$""` already run fine on any shell, just
with `$` in the output. (And as noted in BZ #13853, xgettext prints
lots of warnings.)
AFAIK sotruss is the last install-time or run-time script that requires
a specific shell (now that ldd, tzselect, and a couple of scripts under
sysdeps/unix/ have recently become more portable). So with Mike's
patch, Bash is no longer in any way required to install or run glibc.
--
Patrick "P. J." McDermott
http://www.pehjota.net/
Lead Developer, ProteanOS
http://www.proteanos.com/