From owner-svn-src-stable@FreeBSD.ORG Tue Nov 9 03:13:43 2010 Return-Path: Delivered-To: svn-src-stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C3D381065673; Tue, 9 Nov 2010 03:13:43 +0000 (UTC) (envelope-from obrien@NUXI.org) Received: from dragon.nuxi.org (trang.nuxi.org [74.95.12.85]) by mx1.freebsd.org (Postfix) with ESMTP id 9D7328FC21; Tue, 9 Nov 2010 03:13:43 +0000 (UTC) Received: from dragon.nuxi.org (obrien@localhost [127.0.0.1]) by dragon.nuxi.org (8.14.4/8.14.4) with ESMTP id oA93Dhv5097762; Mon, 8 Nov 2010 19:13:43 -0800 (PST) (envelope-from obrien@dragon.nuxi.org) Received: (from obrien@localhost) by dragon.nuxi.org (8.14.4/8.14.4/Submit) id oA93Dgbf097732; Mon, 8 Nov 2010 19:13:42 -0800 (PST) (envelope-from obrien) Date: Mon, 8 Nov 2010 19:13:42 -0800 From: "David O'Brien" To: Jilles Tjoelker Message-ID: <20101109031342.GC62119@dragon.NUXI.org> References: <201010182310.o9INAWPU089616@svn.freebsd.org> <20101023212023.GA10891@stack.nl> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20101023212023.GA10891@stack.nl> X-Operating-System: FreeBSD 9.0-CURRENT X-to-the-FBI-CIA-and-NSA: HI! HOW YA DOIN? User-Agent: Mutt/1.5.16 (2007-06-09) Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-8@freebsd.org Subject: Re: svn commit: r214045 - in stable/8/tools/regression/bin/sh: builtins errors execution expansion set-e X-BeenThere: svn-src-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: obrien@freebsd.org List-Id: SVN commit messages for all the -stable branches of the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 09 Nov 2010 03:13:43 -0000 On Sat, Oct 23, 2010 at 11:20:24PM +0200, Jilles Tjoelker wrote: > On Mon, Oct 18, 2010 at 11:10:32PM +0000, David E. O'Brien wrote: > > +++ stable/8/tools/regression/bin/sh/builtins/command10.0 Mon Oct 18 23:10:32 2010 (r214045, copy of r204802, head/tools/regression/bin/sh/builtins/command10.0) ... > > +++ stable/8/tools/regression/bin/sh/builtins/command9.0 Mon Oct 18 23:10:32 2010 (r214045, copy of r204801, head/tools/regression/bin/sh/builtins/command9.0) [..] > These work on stable/8, but for the wrong reason. In stable/8, 'command' > does not allow executing builtin utilities, and therefore 'eval' cannot > be found. This is a different error than the expected passing of a > non-number to 'shift', and does not allow testing for the bug this is > supposed to check for. [..] > Because this does not cause any extra failures, I don't think it is > necessary to revert it. However, it seems unusual to MFC a test for a > bugfix without also MFCing the bugfix. I MFCed all passing test cases. If the test case passes when it should not, it seems to me the test case isn't providing sufficient coverage. That said, I would like to MFC the various ash bug fixes that are missing from 8.1. I know you've MFC'ed some of the ones in HEAD, but are ones not MFCed that you feel should be? Or ones that should not be? Are the missing fixes just due to lack of time to MFC? thanks, -- -- David (obrien@FreeBSD.org)