From owner-svn-src-head@FreeBSD.ORG Sun Jul 25 20:17:25 2010 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 31E761065674 for ; Sun, 25 Jul 2010 20:17:25 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from mail2.fluidhosting.com (mx21.fluidhosting.com [204.14.89.4]) by mx1.freebsd.org (Postfix) with ESMTP id E80748FC1D for ; Sun, 25 Jul 2010 20:17:24 +0000 (UTC) Received: (qmail 19507 invoked by uid 399); 25 Jul 2010 20:17:23 -0000 Received: from localhost (HELO laptop.dougb.net) (dougb@dougbarton.us@127.0.0.1) by localhost with ESMTPAM; 25 Jul 2010 20:17:23 -0000 X-Originating-IP: 127.0.0.1 X-Sender: dougb@dougbarton.us Date: Sun, 25 Jul 2010 13:17:22 -0700 (PDT) From: Doug Barton To: Gabor Kovesdan In-Reply-To: <201007251857.o6PIvmoS031484@svn.freebsd.org> Message-ID: References: <201007251857.o6PIvmoS031484@svn.freebsd.org> User-Agent: Alpine 2.00 (BSF 1167 2008-08-23) X-message-flag: Outlook -- Not just for spreading viruses anymore! OpenPGP: id=1A1ABC84 Organization: http://SupersetSolutions.com/ MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, delphij@freebsd.org Subject: Re: svn commit: r210479 - head/usr.bin/grep X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 25 Jul 2010 20:17:25 -0000 On Sun, 25 Jul 2010, Gabor Kovesdan wrote: > Author: gabor > Date: Sun Jul 25 18:57:48 2010 > New Revision: 210479 > URL: http://svn.freebsd.org/changeset/base/210479 > > Log: > - Fix -l and -L by really surpressing output and just showing filenames > > Submitted by: swell.k@gmail.com .... and dougb. :) (It was part of the patch I sent on Friday.) And not to pile on, but regression testing should have caught this. First by you, and if all else fails, by your mentor. At very least the thought process could have gone something like: "Let's see, the patch that I/Gabor generated to fix this problem is different than the one Doug generated. Now Doug is not a _complete_ moron, so let's take a careful look at this change and see if it really does what we think it should do." Aside from the fact that despite its innocuous appearance grep really is an important tool; I've expressed frustration in the past that mentors are not in place to simply rubber stamp things. Mentors serve at least 2 critical functions; to introduce mentees to "The FreeBSD Culture," and to sanity-check changes. No matter how technically competent the mentee is (and obviously Gabor is a sharp guy) both of these functions are critical to their successful introduction into the wonderful world of DougBS^W I mean, uh, FreeBSD. :) Doug > Approved by: delphij (mentor) > > Modified: > head/usr.bin/grep/util.c > > Modified: head/usr.bin/grep/util.c > ============================================================================== > --- head/usr.bin/grep/util.c Sun Jul 25 18:32:59 2010 (r210478) > +++ head/usr.bin/grep/util.c Sun Jul 25 18:57:48 2010 (r210479) > @@ -343,7 +343,7 @@ procline(struct str *l, int nottext) > return (c); /* Binary file */ > > /* Dealing with the context */ > - if ((tail || c) && !cflag && !qflag) { > + if ((tail || c) && !cflag && !qflag && !lflag && !Lflag) { > if (c) { > if (!first && !prev && !tail && Aflag) > printf("--\n"); > > -- Improve the effectiveness of your Internet presence with a domain name makeover! http://SupersetSolutions.com/ Computers are useless. They can only give you answers. -- Pablo Picasso