From owner-cvs-all Sat Jan 6 5:21:38 2001 From owner-cvs-all@FreeBSD.ORG Sat Jan 6 05:21:33 2001 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from lucifer.ninth-circle.org (lucifer.bart.nl [194.158.168.74]) by hub.freebsd.org (Postfix) with ESMTP id CBB1B37B400; Sat, 6 Jan 2001 05:21:32 -0800 (PST) Received: (from asmodai@localhost) by lucifer.ninth-circle.org (8.11.1/8.11.0) id f06DL1249387; Sat, 6 Jan 2001 14:21:01 +0100 (CET) (envelope-from asmodai) Date: Sat, 6 Jan 2001 14:21:01 +0100 From: Jeroen Ruigrok van der Werven To: Bruce Evans Cc: Alexander Langer , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/modules/s3 Makefile Message-ID: <20010106142101.B49254@lucifer.bart.nl> References: <20010106125323.A2966@cichlids.cichlids.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: ; from bde@zeta.org.au on Sat, Jan 06, 2001 at 11:39:35PM +1100 Organisation: VIA Net.Works The Netherlands Sender: asmodai@lucifer.ninth-circle.org Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG -On [20010106 13:40], Bruce Evans (bde@zeta.org.au) wrote: >On Sat, 6 Jan 2001, Alexander Langer wrote: > >> Thus spake Jeroen Ruigrok van der Werven (asmodai@FreeBSD.org): >> >> > We don't use += for SRCS in Makefiles, we simply use the >> > line continuation character and state the rest of the sourcefiles >> > on the next line. >> >> Uhm: >> alex:/sys/modules $ grep 'SRCS[^\t ]+=' */Makefile | wc -l >> 38 > >Correcting the pattern to 'SRCS[ ]*+=' (hard tab in this) gives >a count of 127. I was and am aware of that, however, when I make such a commit I want to touch/correct them in one sweep. Personally I am a great bastard when it comes to consistency and/or consistent style. >These are mostly style bugs, although not as bad as non-null whitespace >between variables and assignment operators (the bad example of >modules/fxp/Makefile seems to have been cloned all over). `+=' may >be right for separating logically diferent parts of SRCS, e.g., generated >header files from non-generated .c files. Or the files which need to be included depending on which piece is #defined as with most NIC Makefiles. -- Jeroen Ruigrok van der Werven VIA Net.Works The Netherlands BSD: Technical excellence at its best Network- and systemadministrator D78D D0AD 244D 1D12 C9CA 7152 035C 1138 546A B867 I know your name, but I am just your Fallen Angel of Doom... To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message