This is the mail archive of the
cygwin
mailing list for the Cygwin project.
Re: Script broken after updating bash to 4.3.46-7?
- From: Gene Pavlovsky <gene dot pavlovsky at gmail dot com>
- To: cygwin at cygwin dot com
- Date: Sat, 27 Aug 2016 21:31:07 +0300
- Subject: Re: Script broken after updating bash to 4.3.46-7?
- Authentication-results: sourceware.org; auth=none
- References: <CAPTiy3PW0+bk20d30-J=sLr=PPnFOeU0Fb495hh1bBVtbtPrzw@mail.gmail.com> <CAPTiy3PXKi+8c+7VGOe+KMjqKmHAYfLERtsLcYKsz-5=C6nUaA@mail.gmail.com> <1635303092.20160827191501@yandex.ru>
Andrey,
That's what I personally think, none of the scripts I use have CRs,
and this is why I'd prefer not using the `igncr` option.
However the recent change to how `read` works makes it necessary to
modify existing scripts which interoperate with Windows console
programs (in my case, Windows build of mysql (MariaDB)) that produce
CRLF line endings.
--Gene
On 27 August 2016 at 19:15, Andrey Repin <anrdaemon@yandex.ru> wrote:
> Greetings, Gene Pavlovsky!
>
>> Looks like it's related to a recent change in bash, which is `read`
>> now honors Cygwin-specific `igncr` shell option (`set -o igncr`),
>> which I didn't enable.
>> Adding `set -o igncr` to the top of the script does the job, however
>> I'd like to know how many more scripts are potentially malfunctioning
>> now?
>> It's lucky that one sent e-mails on errors, some others might just
>> break silently.
>> Is it advisable to add `set -o igncr` to /etc/profile or SHELLOPTS? I
>> didn't use that feature before and am worried about some other
>> negative side effects.
>
> It is advisable to not have CR's in your scripts to begin with.
>
>
> --
> With best regards,
> Andrey Repin
> Saturday, August 27, 2016 19:14:10
>
> Sorry for my terrible english...
>
--
Problem reports: http://cygwin.com/problems.html
FAQ: http://cygwin.com/faq/
Documentation: http://cygwin.com/docs.html
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple