From owner-freebsd-audit Fri Mar 15 17:16:20 2002 Delivered-To: freebsd-audit@freebsd.org Received: from sunny.newgold.net (durham-ar1-174-172.durham.dsl.gtei.net [4.40.174.172]) by hub.freebsd.org (Postfix) with ESMTP id 9B5F337B400; Fri, 15 Mar 2002 17:16:14 -0800 (PST) Received: from sunny.newgold.net (freebsd@localhost [IPv6:::1]) by sunny.newgold.net (8.12.1/8.12.1) with ESMTP id g2G0uTDF019070; Sat, 16 Mar 2002 00:57:24 GMT Received: (from freebsd@localhost) by sunny.newgold.net (8.12.1/8.12.1/Submit) id g2G0uS5k020520; Sat, 16 Mar 2002 00:56:28 GMT Date: Sat, 16 Mar 2002 00:55:33 +0000 From: "J. Mallett" To: Garance A Drosihn Cc: "J. Mallett" , freebsd-audit@FreeBSD.ORG Subject: Re: patch for review: xargs standards compliance Message-ID: <20020316005533.A20903@FreeBSD.ORG> References: <20020315231100.A20942@FreeBSD.ORG> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.3.21i Organisation: FreeBSD Sender: owner-freebsd-audit@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Fri, Mar 15, 2002 at 08:04:29PM -0500, Garance A Drosihn wrote: > At 11:11 PM +0000 3/15/02, J. Mallett wrote: > >The patch is available here: > > http://people.freebsd.org/~jmallett/xargs.patch > > > >Relative to -rHEAD right now. > > > >In detail, it adds: -E, -I, -L, -t; As far as flags go. > > I like the idea, I particularly like the idea when someone > else writes all the code... :-) Then credit goes about half to me and half to Tim Robbins and um an extra half to Mike Barcroft for not letting me let things rot. > I don't see why you added: > +# $xMach: Makefile,v 1.1.1.2 2002/02/22 19:31:35 jmallett Exp $ > to the makefile. But it probably will need a $FreeBSD$ in there > somewhere, or you'll have trouble when you go to check it in. It's because I synched the entire dir with what xMach has (though not path*.h as it changed not at all). I can remove it trivially. > For the new routine strnsubst.c, I believe the current practice > for including $FreeBSD$ in a '.c' file is not to add it to a > comment, but to add: > > #include > __FBSDID("$FreeBSD$"); > > after that first comment block. (I believe there's supposed to > be a blank line between the closing '*/' and the include for > sys/cdefs.h, and no blank line between that and the __FBSDID > line) You're right, sorry about that, I was thinking for a second that was only for the file with main in it, but the id string is static so it doesn't matter does it. > The actual update looks OK, but I've only skimmed through it > and I haven't tried to apply it and put it through any torture > testing. The actual update looks like what the standards ask for; It's rather evil. > >I'm looking to commit this after the weekend, so review would > >be appreciated, comments and flames welcome... Three days > >sound good? > > It would be better to wait more than just a weekend. Despite > popular folklore, many of us do sometimes go thru a whole > weekend without checking the mailing lists for new code... :-) I can include monday with that without a problem. I just know if this sits in my TODO list much longer it's going to rot away to death. I'll update the patch with what you've mentioned... Tomorrow, I'll leave this message marked unread, and I'll reply mentioning when a new one is available... Comments on other {,de}merits of the patch welcome. Thanks, /j. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-audit" in the body of the message