From owner-freebsd-current@FreeBSD.ORG Thu Jan 14 23:33:52 2010 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3E58F106568F; Thu, 14 Jan 2010 23:33:52 +0000 (UTC) (envelope-from jilles@stack.nl) Received: from mx1.stack.nl (relay04.stack.nl [IPv6:2001:610:1108:5010::107]) by mx1.freebsd.org (Postfix) with ESMTP id 02E3F8FC0A; Thu, 14 Jan 2010 23:33:52 +0000 (UTC) Received: from toad.stack.nl (toad.stack.nl [IPv6:2001:610:1108:5010::135]) by mx1.stack.nl (Postfix) with ESMTP id D30DD1DD483; Fri, 15 Jan 2010 00:33:50 +0100 (CET) Received: by toad.stack.nl (Postfix, from userid 1677) id B919E73FA1; Fri, 15 Jan 2010 00:33:50 +0100 (CET) Date: Fri, 15 Jan 2010 00:33:50 +0100 From: Jilles Tjoelker To: Antoine Brodin Message-ID: <20100114233350.GA77512@stack.nl> References: <20100114155755.GA77799@mr-happy.com> <20100114184758.3c4e18ca.antoine@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100114184758.3c4e18ca.antoine@FreeBSD.org> User-Agent: Mutt/1.5.18 (2008-05-17) Cc: freebsd-current@freebsd.org, Jeff Blank Subject: Re: Regression in sh(1) ? (Was: make delete-old fails when removing catpages) X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 14 Jan 2010 23:33:52 -0000 On Thu, Jan 14, 2010 at 06:47:58PM +0100, Antoine Brodin wrote: > This seems to be a regression in sh(1). > Simple test case: > This succeeds on stable/8: > %%% > > sh > $ touch /tmp/foo; 3<&0; echo /tmp/foo | while read i; do rm -vi ${i} <&3 ; done > remove /tmp/foo? y > /tmp/foo > %%% > and fails on head: > %%% > > sh > $ touch /tmp/foo; 3<&0; echo /tmp/foo | while read i; do rm -vi ${i} <&3 ; done > remove /tmp/foo? $ > %%% This is because src/Makefile.inc1 had a bug, 'exec' is required to have a redirection persist after the command. I have fixed this. -- Jilles Tjoelker