summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJim Meyering <meyering@redhat.com>2011-05-11 17:17:42 +0200
committerJim Meyering <meyering@redhat.com>2011-05-11 17:20:33 +0200
commit9418e01abeb92f066d429ba33e7dfb7edf0f8522 (patch)
tree0801bdbbedfcb2f4e155f93198d84efc9169507e
parent2141543a3703cdb29cda41b1562f53f79215959c (diff)
downloadcoreutils-9418e01abeb92f066d429ba33e7dfb7edf0f8522.tar.xz
maint: remove syntax-checking sc_tight_scope rule
* src/Makefile.am (sc_tight_scope): Remove rule. Now it's provided via gnulib's maint.mk. * cfg.mk (sc_tight_scope): Likewise.
-rw-r--r--cfg.mk4
-rw-r--r--src/Makefile.am40
2 files changed, 0 insertions, 44 deletions
diff --git a/cfg.mk b/cfg.mk
index ae41ce3fa..9869ebab8 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -128,10 +128,6 @@ sc_sun_os_names:
{ echo '$(ME): found misuse of Sun OS version numbers' 1>&2; \
exit 1; } || :
-ALL_RECURSIVE_TARGETS += sc_tight_scope
-sc_tight_scope:
- @$(MAKE) -s -C src $@
-
ALL_RECURSIVE_TARGETS += sc_check-AUTHORS
sc_check-AUTHORS:
@$(MAKE) -s -C src $@
diff --git a/src/Makefile.am b/src/Makefile.am
index 548f5ed4b..6083b396e 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -702,46 +702,6 @@ sc_check-AUTHORS: $(all_programs)
sed -n '/^[^ ][^ ]*:/p' $(top_srcdir)/AUTHORS > $(au_dotdot) && \
diff $(au_actual) $(au_dotdot) && rm -f $(au_actual) $(au_dotdot)
-# The following rule is not designed to be portable,
-# and relies on tools that not everyone has.
-
-# Most functions in src/*.c should have static scope.
-# Any that don't must be marked with `extern', but `main'
-# and `usage' are exceptions: they're always extern, but
-# do not need to be marked. Also functions starting with __
-# are exempted due to possibly being added by the compiler
-# (when compiled as a shared library for example).
-#
-# The second nm|grep checks for file-scope variables with `extern' scope.
-.PHONY: sc_tight_scope
-sc_tight_scope: $(bin_PROGRAMS)
- @t=exceptions-$$$$; \
- trap 's=$$?; rm -f $$t; exit $$s' 0; \
- for sig in 1 2 3 13 15; do \
- eval "trap 'v=`expr $$sig + 128`; (exit $$v); exit $$v' $$sig"; \
- done; \
- src=`for f in $(SOURCES); do \
- test -f $$f && d= || d=$(srcdir)/; echo $$d$$f; done`; \
- hdr=`for f in $(noinst_HEADERS); do \
- test -f $$f && d= || d=$(srcdir)/; echo $$d$$f; done`; \
- ( printf 'main\nusage\n_.*\n'; \
- grep -h -A1 '^extern .*[^;]$$' $$src \
- | grep -vE '^(extern |--)' | sed 's/ .*//'; \
- 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 >&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 >&2; \
- exit 1; } || :
-
# Use the just-built ./ginstall, when not cross-compiling.
if CROSS_COMPILING
cu_install_program = @INSTALL_PROGRAM@