From: Jim Meyering <meyering@redhat.com>
Date: Fri, 27 Feb 2009 16:10:19 +0000 (+0100)
Subject: doc: HACKING: mention the new space-only indentation policy
X-Git-Tag: v7.6~74
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=ddfcccce51f8b74d50cf3efa0e6da61b6f96946d;p=thirdparty%2Fcoreutils.git

doc: HACKING: mention the new space-only indentation policy
---

diff --git a/HACKING b/HACKING
index 928f0c5044..b14b6047dc 100644
--- a/HACKING
+++ b/HACKING
@@ -233,20 +233,25 @@ Try to make the summary line fit one of the following forms:
   maint: change-description
 
 
-Use SPACE-only indentation in new files.
-========================================
-In any new file, eliminate all leading TABs (e.g., via running GNU indent
-with --no-tabs) and put these lines at the end of the file:
-[FIXME: suggest vim syntax to do same thing, if it can be done safely.
- Most distros now "set nomodeline" by default for a good reason. ]
+Use SPACE-only indentation in all[*] files
+==========================================
+We use space-only indentation in nearly all files.
+If you use Emacs and your coreutils working directory name matches,
+this code enables the right mode:
 
-/*
- * Local variables:
- * indent-tabs-mode: nil
- * End:
- */
+  ;; In coreutils, indent with spaces everywhere (not TABs).
+  ;; Exceptions: Makefile and ChangeLog modes.
+  (add-hook 'find-file-hook '(lambda ()
+    (if (and buffer-file-name
+             (string-match "/coreutils\\>" (buffer-file-name))
+             (not (string-equal mode-name "Change Log"))
+             (not (string-equal mode-name "Makefile")))
+        (setq indent-tabs-mode nil))))
 
-Do not change TABs to spaces or vice versa in any existing file.
+[*] Makefile and ChangeLog files are exempt, of course.
+
+[FIXME: suggest vim syntax to do same thing, if it can be done safely.
+ Most distros now "set nomodeline" by default for a good reason. ]
 
 
 Send patches to the address listed in --help output