From owner-freebsd-hackers@FreeBSD.ORG Fri Dec 30 06:27:11 2011 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 7F150106564A for ; Fri, 30 Dec 2011 06:27:09 +0000 (UTC) (envelope-from pluknet@gmail.com) Received: from mail-tul01m020-f182.google.com (mail-tul01m020-f182.google.com [209.85.214.182]) by mx1.freebsd.org (Postfix) with ESMTP id 0ACD58FC15 for ; Fri, 30 Dec 2011 06:27:08 +0000 (UTC) Received: by obbwd18 with SMTP id wd18so15149059obb.13 for ; Thu, 29 Dec 2011 22:27:08 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; bh=ch+kjrPGCQTLH4cjM54zM7H4aJZikBW0OUAqEd+aArc=; b=SB8EXESK8GfBAakWvckU03CQimPGOY7z74fYvy3YDdJcwu9gQ2kHDnmyqJitlSuyCq 28uzyAmA7Iki99gl3RDCPGHJqJnJB+RJRc2bBx0/uUeOb817n3nH6V+2zswK+nndWCGG 6h76w0vU9n117KYC4wT7SNceYGyEBeUuOYYcU= MIME-Version: 1.0 Received: by 10.182.15.104 with SMTP id w8mr33491323obc.20.1325226428411; Thu, 29 Dec 2011 22:27:08 -0800 (PST) Received: by 10.182.171.67 with HTTP; Thu, 29 Dec 2011 22:27:08 -0800 (PST) In-Reply-To: <02b401ccc4ff$b475b7e0$1d6127a0$@fisglobal.com> References: <02b401ccc4ff$b475b7e0$1d6127a0$@fisglobal.com> Date: Fri, 30 Dec 2011 09:27:08 +0300 Message-ID: From: Sergey Kandaurov To: Devin Teske Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Cc: Garrett Cooper , freebsd-hackers@freebsd.org Subject: Re: [PATCH] Fix kenv(1) output in w/respect to new boot loader variables X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 30 Dec 2011 06:27:11 -0000 On 28 December 2011 05:26, Devin Teske wrote: > D'Oh! Attached wrong (OLD; already applied) patch. > > Please find appropriate patch attached! Hi. I committed your patch to head as svn r228985. Thank you! > >> -----Original Message----- >> From: Devin Teske [mailto:devin.teske@fisglobal.com] >> Sent: Tuesday, December 27, 2011 5:24 PM >> To: 'freebsd-hackers@freebsd.org' >> Cc: Garrett Cooper; devin.teske@fisglobal.com >> Subject: [PATCH] Fix kenv(1) output in w/respect to new boot loader vari= ables >> >> Garrett Cooper and a few others have requested that I write a patch to f= ix a >> regression w/respect to kenv(1) output in FreeBSD-9.0 and HEAD. >> >> The issue is with the new boot loader menu. It adds many loader variable= s >> including ones that contain ANSI color escapes. >> >> Obviously, these ANSI codes don't play well with serial consoles when ke= nv(1) > is >> executed without arguments (reports vary as to what happens, but it's ne= ver >> pretty). >> >> Attached is a patch to the Forth code that clears-out the menu-associate= d >> variables before invoking the kernel. >> >> The net-effect is that kenv(1) no longer reports menu-related variables. >> >> In essence, kenv(1) output should now appear the same as on RELENG_8 (wh= ich >> lacks the new boot loader and didn't use any such variables). Thus, rest= oring >> serial console glory. >> -- >> Devin > > _____________ > The information contained in this message is proprietary and/or confident= ial. If you are not the intended recipient, please: (i) delete the message = and all copies; (ii) do not disclose, distribute or use the message in any = manner; and (iii) notify the sender immediately. In addition, please be awa= re that any message addressed to our domain is subject to archiving and rev= iew by persons other than the intended recipient. Thank you. Great! --=20 wbr, pluknet