From owner-cvs-all@FreeBSD.ORG Thu Jun 17 21:48:50 2004 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 D5E7216A4CE; Thu, 17 Jun 2004 21:48:50 +0000 (GMT) Received: from electra.cse.Buffalo.EDU (electra.cse.Buffalo.EDU [128.205.32.2]) by mx1.FreeBSD.org (Postfix) with ESMTP id 5A21443D41; Thu, 17 Jun 2004 21:48:50 +0000 (GMT) (envelope-from kensmith@cse.Buffalo.EDU) Received: from electra.cse.Buffalo.EDU (kensmith@localhost [127.0.0.1]) i5HLmR2n007478; Thu, 17 Jun 2004 17:48:27 -0400 (EDT) Received: (from kensmith@localhost) by electra.cse.Buffalo.EDU (8.12.10/8.12.9/Submit) id i5HLmRbM007477; Thu, 17 Jun 2004 17:48:27 -0400 (EDT) Date: Thu, 17 Jun 2004 17:48:27 -0400 From: Ken Smith To: Max Khon Message-ID: <20040617214827.GB6029@electra.cse.Buffalo.EDU> References: <200406170008.i5H08NDt085108@repoman.freebsd.org> <20040617173854.GJ61448@elvis.mu.org> <20040617182031.GA8170@samodelkin.net> <20040617184518.GB831@electra.cse.Buffalo.EDU> <20040617204813.GA10670@samodelkin.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20040617204813.GA10670@samodelkin.net> User-Agent: Mutt/1.4.1i cc: src-committers@FreeBSD.org cc: Max Khon cc: Alfred Perlstein cc: cvs-all@FreeBSD.org cc: cvs-src@FreeBSD.org cc: Ken Smith Subject: Re: cvs commit: src/sys/sys mbuf.h src/sys/kern uipc_mbuf.c uipc_syscalls.c src/usr.bin/netstat mbuf.c src/lib/libc/sys sendfile.2 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: Thu, 17 Jun 2004 21:48:51 -0000 On Fri, Jun 18, 2004 at 03:48:13AM +0700, Max Khon wrote: > Hi, Ken! Hi, Max! ;-) I hope none of this is taken personally. I have some relatively new responsibilities being on re@ and I'm using this as a way to make sure I really do understand how things are supposed to work. I think I have a reasonably good handle on it but it never hurts to test it to make sure. :-) > On Thu, Jun 17, 2004 at 02:45:18PM -0400, Ken Smith wrote: > > > > > This will break scripts. > > > > > > > > Please back out the netstat change. > > > > > > No problems, but can you tell me which scripts or programs > > > has become broken after this change? > > > > The scripts that are now potentially broken don't necessarily need to > > be in the base system. > > I did not say that I am interested in base system scripts only :) Ok, sorry about that. > > They could very well be scripts Alfred wrote > > and uses himself for his own purposes. Once a branch goes into -STABLE > > status we try to not make 'user-visible' changes to it unless they are > > bugfixes or do not alter the way something had worked. > > As Bosko noted the change _adds_ a few lines to netstat -m output. > Old output is not altered. > Frankly speaking I can't imagine a script that could correctly parse > netstat -m output before, but now is broken. I may work with student programmers too much, but I'm afraid my imagination is a bit better than yours in this case. :-( I don't defend it as being the right way to do things, but in cases where the user is interested in "most of" the output of something they will often take the approach of removing things they don't want instead of selecting things they do want. This change would add an extra line to what the next stage of processing would see if that approach is used. This particular change is a case of nit-picking. It's small, hard to imagine how it could effect someone, etc. But even Bosko said more caution 'next time' would be good, I'm just emphasizing why. To some extent the output of programs has been an API ever since pipes were invented. And unless I'm severely mistaken one of the things we have tried to avoid is changing API's once a branch goes -STABLE. -- Ken Smith - From there to here, from here to | kensmith@cse.buffalo.edu there, funny things are everywhere. | - Theodore Geisel |