This is the mail archive of the
cygwin
mailing list for the Cygwin project.
RE: Problems with the new base-files-4.0-5?
- From: "Nellis, Kenneth" <Kenneth dot Nellis at acs-inc dot com>
- To: <cygwin at cygwin dot com>
- Date: Thu, 17 Mar 2011 16:10:27 -0500
- Subject: RE: Problems with the new base-files-4.0-5?
- References: <4D825F42.3090900@redhat.com>
> From: Eric Blake
> On 03/17/2011 12:59 PM, Angelo Graziosi wrote:
> > Il 17/03/2011 19.00, Angelo Graziosi ha scritto:
> >> I have just updated Cygwin, and now starting a console (Cygwin.bat
> or
> >> MinTTY) prints:
> >>
> >> -bash: ${p}: ambiguous redirect
> >>
> >> and PRINTER variable isn't set.
> >>
> >> Reverting to base-files-4.0-4 works just fine and PRINTER is set
> >> correctly to my default printer.
> >>
> >> In 4.0-5 it seems something is broken in /etc/profile.
I see the same thing.
> > Perhaps you need this:
> >
> > - ([[ -e ${p} ]] && read -r PRINTER < ${p}) && PRINTER=${PRINTER%%,*}
> > + ([[ -e ${p} ]] && read -r PRINTER < "${p}") &&
> PRINTER=${PRINTER%%,*}
>
> /etc/profile is using [[ ]]? Bad. That's not required by POSIX, and
> will break for anyone who logs in with a shell that doesn't understand
> it (for example, dash or posh). Bashisms should not appear in
> /etc/profile.
Seems there are several instances of this:
$ grep '\[\[' /etc/profile
([[ -e ${p} ]] && read -r PRINTER < ${p}) && PRINTER=${PRINTER%%,*}
if [[ -n ${BASH_VERSION} ]]; then
[[ -f "/etc/bash.bashrc" ]] && . "/etc/bash.bashrc"
elif [[ -n ${KSH_VERSION} ]]; then
elif [[ -n ${ZSH_VERSION} ]]; then
$
Is it reasonable to presume a quick turnaround on a fix?
--Ken Nellis