From owner-freebsd-hackers Sun Jul 15 22:25:25 2001 Delivered-To: freebsd-hackers@freebsd.org Received: from mail.rpi.edu (mail.rpi.edu [128.113.22.40]) by hub.freebsd.org (Postfix) with ESMTP id 0403137B401; Sun, 15 Jul 2001 22:25:22 -0700 (PDT) (envelope-from drosih@rpi.edu) Received: from [128.113.24.47] (gilead.acs.rpi.edu [128.113.24.47]) by mail.rpi.edu (8.11.3/8.11.3) with ESMTP id f6G5N5V136202; Mon, 16 Jul 2001 01:23:05 -0400 Mime-Version: 1.0 X-Sender: drosih@mail.rpi.edu Message-Id: In-Reply-To: <20010715221527.B1084@schweikhardt.net> References: <20010715155941.C1165@schweikhardt.net> <20010715111956.A71874@dragon.nuxi.com> <20010715221527.B1084@schweikhardt.net> Date: Mon, 16 Jul 2001 01:23:03 -0400 To: Jens Schweikhardt , "David O'Brien" From: Garance A Drosihn Subject: Re: Whitespace at end of line Cc: hackers@FreeBSD.ORG Content-Type: text/plain; charset="us-ascii" ; format="flowed" Sender: owner-freebsd-hackers@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG At 10:15 PM +0200 7/15/01, Jens Schweikhardt wrote: >David et al, > >On Sun, Jul 15, 2001 at 11:19:57AM -0700, David O'Brien wrote: ># ># Please do not do this to the .[ch] files. It makes diffs to stable ># harder. > >I see. Dima suggested in private email that I/we MFC such changes >relatively fast in order to avoid that; of course this would be done >only after md5 "approval" of the binaries. Also note that your reason for changing this is the disk space consumed by the extra blanks. You're going to be combatting that by using up disk space for patches (at least as far as my systems are concerned, because I cvsup the CVS tree and not any specific release). ># Please leave such cleanup to ># when people are in the files cleaning up other style nits so we don't ># have 1 million commits, each taking care of just one point of style(9). > >OK, then I think it's best to tell developers@ about my script in >freefall:~schweikh/bin/ws-at-eol which by default will only print the >names of the files and optionally the line numbers/lines. Committers are >encouraged to make use of it as they see fit. Folks, please remember to >make separate commits for content changes and whitespace changes. I am not in favor of a generic sweep to remove blank space, but the script will be useful to use when working on some source for other reasons. Thanks. -- Garance Alistair Drosehn = gad@eclipse.acs.rpi.edu Senior Systems Programmer or gad@freebsd.org Rensselaer Polytechnic Institute or drosih@rpi.edu To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-hackers" in the body of the message