diff options
| author | Jim Meyering <meyering@redhat.com> | 2011-04-26 09:27:32 +0200 |
|---|---|---|
| committer | Jim Meyering <meyering@redhat.com> | 2011-04-26 09:27:49 +0200 |
| commit | 74e5ad9cf7c515b69ae9d5a83168c70eb2604fec (patch) | |
| tree | 4f1596d46429e84c9bd3fa7c54c8e6813cd298b5 /src | |
| parent | 740516cd1205b58ad07f2318335d8177586a4ca4 (diff) | |
| download | coreutils-74e5ad9cf7c515b69ae9d5a83168c70eb2604fec.tar.xz | |
maint: tweak sc_tight_scope rule
* src/Makefile.am (sc_tight_scope): Remove useless quotes,
change 1>&2 to >&2, and combine a few short lines.
Diffstat (limited to 'src')
| -rw-r--r-- | src/Makefile.am | 20 |
1 files changed, 8 insertions, 12 deletions
diff --git a/src/Makefile.am b/src/Makefile.am index de0000dde..548f5ed4b 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -730,21 +730,17 @@ sc_tight_scope: $(bin_PROGRAMS) perl -lne '/^extern (?:enum )?\S+ \**(\S*) \(/ and print $$1' \ $$hdr; \ ) | $(ASSORT) -u | sed 's/^/^/;s/$$/$$/' > $$t; \ - nm -e *.$(OBJEXT) \ - | sed -n 's/.* T //p' \ - | sed 's/^_//' \ - | grep -Ev -f $$t && \ - { echo 'the above functions should have static scope' 1>&2; \ - exit 1; } || : ; \ + nm -e *.$(OBJEXT) | sed -n 's/.* T //p' \ + | sed 's/^_//' | grep -Ev -f $$t \ + && { echo the above functions should have static scope >&2; \ + exit 1; } || : ; \ ( printf '^program_name$$\n'; \ perl -lne '/^extern .*?\**(\w+);/ and print "^$$1\$$"' \ $$hdr *.h ) | $(ASSORT) -u > $$t; \ - nm -e *.$(OBJEXT) \ - | sed -n 's/.* [BD] //p' \ - | sed 's/^_//' \ - | grep -Ev -f $$t && \ - { echo 'the above variables should have static scope' 1>&2; \ - exit 1; } || : + nm -e *.$(OBJEXT) | sed -n 's/.* [BD] //p' \ + | sed 's/^_//' | grep -Ev -f $$t \ + && { echo the above variables should have static scope >&2; \ + exit 1; } || : # Use the just-built ./ginstall, when not cross-compiling. if CROSS_COMPILING |