maint: normalize leading-TAB indentation in Makefiles
authorJim Meyering <meyering@redhat.com>
Wed, 18 Mar 2009 11:20:32 +0000 (12:20 +0100)
committerJim Meyering <meyering@redhat.com>
Wed, 18 Mar 2009 11:40:07 +0000 (12:40 +0100)
* maint.mk (sc_makefile_TAB_only_indentation): New rule.
Replace each TAB+8-space sequence with two TABs.
* man/Makefile.am: Likewise.
* build-aux/check.mk: Likewise.
I used this command (run it more than once, if needed):
t=$'\t'; git grep -l -E "$t {8}"|grep -E 'Makefile|\.mk$' \
| xargs perl -pi -e 's/\t {8}/\t\t/'

build-aux/check.mk
maint.mk
man/Makefile.am

index 92935d9a3b7d6e5f429a65371a0d17ad90b7e2dd..5bfcc90cf908e1b30beb8d078758055d1cc310bb 100644 (file)
@@ -213,8 +213,8 @@ $(TEST_SUITE_LOG): $(TEST_LOGS)
            for f in $(TEST_LOGS);                                      \
            do                                                          \
              case $$(sed 1q $$f) in                                    \
-               SKIP:*|PASS:*|XFAIL:*);;                                \
-               *) echo; cat $$f;;                                      \
+               SKIP:*|PASS:*|XFAIL:*);;                                \
+               *) echo; cat $$f;;                                      \
              esac;                                                     \
            done;                                                       \
          } >$(TEST_SUITE_LOG).tmp;                                     \
index 31d75f4c0aac1a5d66d7ee2826eab222e200954a..0c1ad24a6b071d7fd731c3019f62a1f04f17564f 100644 (file)
--- a/maint.mk
+++ b/maint.mk
@@ -537,6 +537,12 @@ changelog-check:
          exit 1;                                                       \
        fi
 
+sc_makefile_TAB_only_indentation:
+       @grep -nE '^    [ ]{8}'                                         \
+           $$($(VC_LIST_EXCEPT) | grep -E 'akefile|\.mk$$')            \
+         && { echo '$(ME): found TAB-8-space indentation' 1>&2;        \
+              exit 1; } || :
+
 sc_m4_quote_check:
        @grep -nE '(AC_DEFINE(_UNQUOTED)?|AC_DEFUN)\([^[]'              \
            $$($(VC_LIST_EXCEPT) | grep -E '(^configure\.ac|\.m4)$$')   \
@@ -588,7 +594,7 @@ sc_makefile_path_separator_check:
 writable-files:
        if test -d $(release_archive_dir); then :; else                 \
          for file in $(distdir).tar.gz                                 \
-                     $(release_archive_dir)/$(distdir).tar.gz; do      \
+                     $(release_archive_dir)/$(distdir).tar.gz; do      \
            test -e $$file || continue;                                 \
            test -w $$file                                              \
              || { echo ERROR: $$file is not writable; fail=1; };       \
@@ -740,8 +746,8 @@ define coreutils-path-check
          && ln -sf ../src/true $(bin)/false            \
          && PATH=`pwd`/$(bin):$$PATH $(MAKE) -C tests check \
          && { test -d gnulib-tests                     \
-                && $(MAKE) -C gnulib-tests check       \
-                || :; }                                \
+                && $(MAKE) -C gnulib-tests check       \
+                || :; }                                \
          && rm -rf $(bin)                              \
          && fail=0;                                    \
     else                                               \
index 2b28052329ff48c3654095c1b8815ebf4aa8938e..ec5284b24de96bb0b77370ac6b8f5d0c4fdc73ae 100644 (file)
@@ -156,13 +156,13 @@ mapped_name = `echo $*|sed 's/^install$$/ginstall/; s/^test$$/[/'`
          *)                                                    \
            rm -f $@                                            \
            && { echo "Updating man page $@";                   \
-                rm -rf $t;                                     \
-                mkdir $t;                                      \
-                (cd $t && $(LN_S) ../../src/$(mapped_name) $*); \
-                   $(PERL) -- $(srcdir)/help2man               \
-                    --source='$(PACKAGE_STRING)'               \
-                    --include=$(srcdir)/$*.x                   \
-                    --output=$t/$@ $t/$*;                      \
+                rm -rf $t;                                     \
+                mkdir $t;                                      \
+                (cd $t && $(LN_S) ../../src/$(mapped_name) $*); \
+                   $(PERL) -- $(srcdir)/help2man               \
+                    --source='$(PACKAGE_STRING)'               \
+                    --include=$(srcdir)/$*.x                   \
+                    --output=$t/$@ $t/$*;                      \
               }                                                \
            && sed 's|$*\.td/||g' $t/$@ > $@                    \
            && rm -rf $t ;;                                     \