From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.io/gmane.emacs.gnus.general/64540 Path: news.gmane.org!not-for-mail From: Reiner Steib Newsgroups: gmane.emacs.gnus.general Subject: Re: Mixing whitespace and topical changes Date: Tue, 17 Apr 2007 19:25:32 +0200 Message-ID: References: Reply-To: Reiner Steib NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: sea.gmane.org 1176830956 16307 80.91.229.12 (17 Apr 2007 17:29:16 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Tue, 17 Apr 2007 17:29:16 +0000 (UTC) To: ding@gnus.org Original-X-From: ding-owner+M13051@lists.math.uh.edu Tue Apr 17 19:29:10 2007 Return-path: Envelope-to: ding-account@gmane.org Original-Received: from util0.math.uh.edu ([129.7.128.18]) by lo.gmane.org with esmtp (Exim 4.50) id 1HdrTw-0000QV-P8 for ding-account@gmane.org; Tue, 17 Apr 2007 19:28:45 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.math.uh.edu) by util0.math.uh.edu with smtp (Exim 4.63) (envelope-from ) id 1HdrSm-0006Bq-MB; Tue, 17 Apr 2007 12:27:32 -0500 Original-Received: from mx2.math.uh.edu ([129.7.128.33]) by util0.math.uh.edu with esmtps (TLSv1:AES256-SHA:256) (Exim 4.63) (envelope-from ) id 1HdrSl-0006Ba-5B for ding@lists.math.uh.edu; Tue, 17 Apr 2007 12:27:31 -0500 Original-Received: from quimby.gnus.org ([80.91.231.51]) by mx2.math.uh.edu with esmtp (Exim 4.63) (envelope-from ) id 1HdrSc-0004lT-HV for ding@lists.math.uh.edu; Tue, 17 Apr 2007 12:27:31 -0500 Original-Received: from main.gmane.org ([80.91.229.2] helo=ciao.gmane.org) by quimby.gnus.org with esmtp (Exim 3.35 #1 (Debian)) id 1HdrSb-0005Ou-00 for ; Tue, 17 Apr 2007 19:27:21 +0200 Original-Received: from list by ciao.gmane.org with local (Exim 4.43) id 1HdrSR-0006i4-9a for ding@gnus.org; Tue, 17 Apr 2007 19:27:11 +0200 Original-Received: from bridgekeeper.physik.uni-ulm.de ([134.60.10.123]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Tue, 17 Apr 2007 19:27:11 +0200 Original-Received: from Reiner.Steib by bridgekeeper.physik.uni-ulm.de with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Tue, 17 Apr 2007 19:27:11 +0200 X-Injected-Via-Gmane: http://gmane.org/ Mail-Followup-To: ding@gnus.org Original-Lines: 120 Original-X-Complaints-To: usenet@sea.gmane.org X-Gmane-NNTP-Posting-Host: bridgekeeper.physik.uni-ulm.de X-Face: .*T0'iU(sujq_j9\J>-d4fg;N/1++U#U$_5ii6k.=|"-n'?5O:Hyz&wi'-!I~,}7~GgT=0S /&-R5sbkNy5+Xo1y{Tw2KKxi@Xh"g@]Qc|.U<*]WDd)qvGowFDvfU1F]{EDho:7P0@|oOD=Bc{K4?> WP68K[Mx:}=`ZT'6g4'f+g?;`vri2!)xGy}3:=l'(/Cea0l4lo^H5#@/Z3ev Mail-Copies-To: nobody User-Agent: Gnus/5.110006 (No Gnus v0.6) Emacs/22.0.97 (gnu/linux) Cancel-Lock: sha1:SxuK+VKCE/onrt9zAgbW/k1lm2I= X-Spam-Score: -2.6 (--) List-ID: Precedence: bulk Xref: news.gmane.org gmane.emacs.gnus.general:64540 Archived-At: On Mon, Apr 16 2007, Didier Verna wrote: > Reiner Steib wrote: > >> please avoid committing lots of whitespace changes mixed with topical >> changes. > > Reiner, > > I can understand your concern (I guess it mainly has to do with > synchronizing between the GNU Emacs version and the CVS version), - Synchronizing also happens between the trunk and v5-10 (i.e. 5.10.6 ... 5.10.nn). Introducing lots of whitespace changes in the trunk will make it much more likely that merging bug-fixes from v5-10 to the trunk fails. This puts additional burden on Miles. When I integrated Oort Gnus in Emacs 22, I've spent much time do clean up the problems because we didn't follow Emacs conventions closely (e.g. properly attributing ChangeLog entries). As long as Miles is doing the syncing semi-automatically, we will not run into this trouble again for No Gnus. Additionally we get longer active maintainance of the stable series for free (BTW, also XEmacs benefits from this, see 5.10.8 in the prerelease packages). As the syncing is very important for the future development of Gnus, we *must not* make Miles' job more complicated than necessary. The fact that doing mass whitespace change (and even mixing them with topical changes) makes syncing harder, already is a k.o. argument against switching to such a policy. - Unrelated whitespace changes make many CVS operations less useful (diff, annotate, ...). E.g. in your recent commits, it was hard for me to identify the topical changes. I had to add `-w' temporarily to `vc-diff-switches' to recognize the topical changes. - Automatic whitespace removal may even lead to wrong code... ,----[ flow-fill.el ] | revision 7.13 | date: 2005/10/27 17:55:26; author: rsteib; state: Exp; lines: +35 -30 | (fill-flowed-encode-tests): Use concatenated string to protect trailing | whitespace. | ---------------------------- | revision 7.12 | date: 2005/10/27 17:36:02; author: rsteib; state: Exp; lines: +9 -9 | (fill-flowed-encode-tests): Restore trailing | whitespace removed in revision 7.8. `---- Well, this was some code for testing, but it may also change "? " to "?" or similar (IIRC). - As Gnus is part of Emacs we should follow the same policies whenever possible. * It's consensus among Emacs developers _not_ to do mass whitespace removal without prior coordination. * It's consensus among Emacs developers to use the Emacs defaults for editing Emacs files (e.g. fill-column, tab-width, coding style, ...). `whitespace-cleanup' in `write-file-hooks' is not the default in Emacs. > but I'm not ready to change my setup (all whitespace cleanup is done > automatically and I never see them in diffs) until this issue has > been officially resolved. I've tried to raise it several times in > the past but didn't get any interest from anybody. Note that I will > accept whatever decision the majority adopts, even if it goes > against me :-) Didier, in the past few years, I've never seen any other Gnus developer committing mixed changes except you. The only discussions[*] on ding about this topic were initiated after your commits, BTW. In these discussions, at least four developers (Katsumi, Miles, Romain and me) have expressed concerns about or argued against these changes. Beside yourself, nobody argued in favor of those "by-the-way whitespace changes". If also Larsi has said "Don't use whitespace.el or equivalent when changing Gnus." (as Katsumi mentioned), isn't this sufficient to convince you? > Let me restate my position: I believe that the right thing to do is to > have whitespace-cleanup on write-file-hooks for everybody committing to > Gnus. I agree that we should _not introduce_ trailing whitespace. > This is simple to achieve, this effectively removes any such > problems and this makes the files [automatically] cleaner. There's > also the question of the info files on which I only got a "I've > heard that it is better this way" sort of message.[1] > > Sorry to be a little pushy on this, but whitespace.el functions on a > major-mode basis and not on a file-name basis, so I would need some > trickery to make it stop working on Gnus files, so I really want a > discussion about this before. You are kind of arguing that we should change our policy because of a bug/missing feature in `whitespace.el'. BTW, instead of... (add-hook 'write-file-hooks #'(lambda () (whitespace-cleanup) nil)) ... it should be simple enough to add check `default-directory' and don't switch it on if it is not /foo/bar/gnus. > Footnotes: > [1] FWIW, texinfo-mode is on whitespace-modes by default, and I'd never > heard any complaint about this until a few days ago. As we already have discussed[*], it often makes the formatted info files ugly (e.g. example lisp code). Bye, Reiner. [*] http://thread.gmane.org/gmane.emacs.gnus.general/64459 http://thread.gmane.org/gmane.emacs.gnus.commits/3222/focus=60498 -- ,,, (o o) ---ooO-(_)-Ooo--- | PGP key available | http://rsteib.home.pages.de/