From owner-cvs-all@FreeBSD.ORG Mon May 16 10:15:19 2005 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 7093F16A4CE; Mon, 16 May 2005 10:15:19 +0000 (GMT) Received: from aiolos.otenet.gr (aiolos.otenet.gr [195.170.0.23]) by mx1.FreeBSD.org (Postfix) with ESMTP id 87E0A43D58; Mon, 16 May 2005 10:15:18 +0000 (GMT) (envelope-from keramida@freebsd.org) Received: from orion.daedalusnetworks.priv (aris.bedc.ondsl.gr [62.103.39.226])j4GADc8l008690; Mon, 16 May 2005 13:13:38 +0300 Received: from orion.daedalusnetworks.priv (orion [127.0.0.1]) j4GAFFW4000728; Mon, 16 May 2005 13:15:15 +0300 (EEST) (envelope-from keramida@freebsd.org) Received: (from keramida@localhost)j4GAFFQS000727; Mon, 16 May 2005 13:15:15 +0300 (EEST) (envelope-from keramida@freebsd.org) Date: Mon, 16 May 2005 13:15:15 +0300 From: Giorgos Keramidas To: "David O'Brien" Message-ID: <20050516101515.GA676@orion.daedalusnetworks.priv> References: <200504161543.j3GFhclO075103@repoman.freebsd.org> <20050516001837.GA65506@dragon.NUXI.org> <20050516004059.GA15002@gothmog.gr> <20050516032743.GA89903@dragon.NUXI.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050516032743.GA89903@dragon.NUXI.org> cc: cvs-src@freebsd.org cc: src-committers@freebsd.org cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/usr.bin/top machine.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 May 2005 10:15:19 -0000 On 2005-05-15 20:27, David O'Brien wrote: >On Mon, May 16, 2005 at 03:40:59AM +0300, Giorgos Keramidas wrote: >>On 2005-05-15 17:18, David O'Brien wrote: >>>On Sat, Apr 16, 2005 at 03:43:38PM +0000, Giorgos Keramidas wrote: >>>> Reduce the width of the THR column to 4 characters, to avoid wrap-around >>>> of lines in SMP machines (which are wider), until we have a better way >>>> of handling window sizes & columns in top. >>>> >>>> Caught by: ache, Andre Guibert de Bruet >>>> Point hat: keramida >>>> >>>> Revision Changes Path >>>> 1.71 +5 -5 src/usr.bin/top/machine.c >>> >>> Please back machine.c back to revsion 1.69, ASAP. >>> The current top is redicious: >..snip.. >> Your USERNAME column is too wide (12 columns). Do you happen to have >> very long names? > > The dovecot port adds a the 'dovecot-auth' user. > The top(1) code supports 13 character usernames on SMP systems and 15 on > UP systems. Your rev 1.70-1.71 changes failed to take that into account. Because there's nothing to take into account. The bug is not in the THR column added by those revisions but in the dynamic resizing of the USERNAME column :-) >> If that is true, would it be ok to wait 1-2 days until I have a >> machine.c version that prints USERNAME using only 8 columns instead of >> using arbitrarily long columns, like now? > > I don't know why to wait 1-2 days. I made the change in 1 minute. Or is > there debate if that change will be acceptable? If there isn't a debate > about it, let me know and I'll commit this diff. I'm not an src committer, so I need a proper review by a real src committer. The 1-2 day estimate was the time it has taken in the past for making the change, testing, testing again, and then getting a review. If you think that hard-limiting USERNAME to 8 columns is fine, then I'm all for it, please commit :-) - Giorgos