X-Recipient:	archive-cygwin AT delorie DOT com
X-SWARE-Spam-Status:	No, hits=-2.6 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,T_TO_NO_BRKTS_FREEMAIL
X-Spam-Check-By:	sourceware.org
Date:	Thu, 17 Mar 2011 23:08:43 +0100
From:	David Sastre <d DOT sastre DOT medina AT gmail DOT com>
To:	cygwin AT cygwin DOT com
Subject:	Re: Problems with the new base-files-4.0-5?
Message-ID:	<20110317220843.GA22548@jethro.local.lan>
References:	<4D825F42 DOT 3090900 AT redhat DOT com> <2BF01EB27B56CC478AD6E5A0A28931F20230042A AT A1DAL1SWPES19MB DOT ams DOT acs-inc DOT net>
MIME-Version:	1.0
In-Reply-To:	<2BF01EB27B56CC478AD6E5A0A28931F20230042A@A1DAL1SWPES19MB.ams.acs-inc.net>
User-Agent:	Mutt/1.5.20 (2009-06-14)
X-IsSubscribed:	yes
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Unsubscribe:	<mailto:cygwin-unsubscribe-archive-cygwin=delorie DOT com AT cygwin DOT com>
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com

--SUOF0GtieIMvvwua
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Mar 17, 2011 at 04:10:27PM -0500, Nellis, Kenneth wrote:
> > 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.
>=20
> I see the same thing.
>=20
> > > Perhaps you need this:
> > >
> > > - ([[ -e ${p} ]] && read -r PRINTER < ${p}) && PRINTER=3D${PRINTER%%,=
*}
> > > + ([[ -e ${p} ]] && read -r PRINTER < "${p}") &&
> > PRINTER=3D${PRINTER%%,*}

Please test if

[ -e "${p}" ] && read -r PRINTER < "${p}" && PRINTER=3D${PRINTER%%,*}

works as expected.=20

> > /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.
>=20
> Seems there are several instances of this:
>=20
> $ grep '\[\[' /etc/profile
> ([[ -e ${p} ]] && read -r PRINTER < ${p}) && PRINTER=3D${PRINTER%%,*}
> if [[ -n ${BASH_VERSION} ]]; then
>   [[ -f "/etc/bash.bashrc" ]] && . "/etc/bash.bashrc"
> elif [[ -n ${KSH_VERSION} ]]; then
> elif [[ -n ${ZSH_VERSION} ]]; then
> $

Corrected. Included in the next release.

> Is it reasonable to presume a quick turnaround on a fix?

Yes. It will be available ASAP.

--=20
Huella de clave primaria: 0FDA C36F F110 54F4 D42B  D0EB 617D 396C 448B 31EB

--SUOF0GtieIMvvwua
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)

iEYEARECAAYFAk2ChmsACgkQYX05bESLMevJQwCgwaJ7cQ9UZ4/K6EiqlUfJL55h
xTQAn19tkHh+PUee0p9pC4qxOySWNNxr
=2ZXh
-----END PGP SIGNATURE-----

--SUOF0GtieIMvvwua--

- Raw text -