mirror of
git://git.savannah.gnu.org/libtool.git
synced 2024-11-27 06:09:57 +08:00
8fad6be03e
Until now, libtool sources have used braced variable names seemingly at random! Almost always the braces are just noise, so remove all the unnecessary ones. * cfg.mk (sc_useless_braces_in_variable_derefs): New syntax check rule to ensure we only reintroduce braced variable dereferences if they are followed by a valid variable name character. build-aux/general.m4sh, build-aux/git-hooks/commit-msg, build-aux/ltmain.m4sh, build-aux/options-parser, configure.ac, libltdl/configure.ac, m4/libtool.m4, m4/ltdl.m4, m4/ltoptions.m4, tests/defs.m4sh, tests/demo-nopic.test, tests/depdemo/configure.ac, tests/flags.at, tests/link.test, tests/objectlist.test, tests/quote.test, tests/static.at: Remove spurious braces. Signed-off-by: Gary V. Vaughan <gary@gnu.org>
223 lines
8.1 KiB
Makefile
223 lines
8.1 KiB
Makefile
# cfg.mk -- Configuration for maintainer-makefile
|
|
#
|
|
# Copyright (c) 2011 Free Software Foundation, Inc.
|
|
# Written by Gary V. Vaughan, 2011
|
|
#
|
|
# This file is part of GNU Libtool.
|
|
#
|
|
# GNU Libtool is free software; you can redistribute it and/or modify
|
|
# it under the terms of the GNU General Public License as published by
|
|
# the Free Software Foundation; either version 2 of the License, or
|
|
# (at your option) any later version.
|
|
#
|
|
# GNU Libtool is distributed in the hope that it will be useful,
|
|
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
# GNU General Public License for more details.
|
|
#
|
|
# You should have received a copy of the GNU General Public License
|
|
# along with GNU Libtool; see the file COPYING. If not, a copy
|
|
# can be downlodad from http://www.gnu.org/licenses/gpl.html,
|
|
# or obtained by writing to the Free Software Foundation, Inc.,
|
|
# 51 Franklin Street, Boston, # MA 02111-1301, USA.
|
|
|
|
# Set format of NEWS
|
|
old_NEWS_hash := d41d8cd98f00b204e9800998ecf8427e
|
|
|
|
manual_title = Portable Dynamic Shared Object Management
|
|
|
|
# Set the release announcement email addresses, maint.mk will email the
|
|
# translation-project instead of autotools-announce otherwise.
|
|
ifeq ($(RELEASE_TYPE),stable)
|
|
announcement_Cc_ = autotools-announce@gnu.org
|
|
else
|
|
announcement_Cc_ = autotools-announce@gnu.org, $(PACKAGE_BUGREPORT)
|
|
endif
|
|
|
|
# Don't syntax check the mail subdirectory.
|
|
VC_LIST_ALWAYS_EXCLUDE_REGEX = ^mail/
|
|
|
|
local-checks-to-fix = \
|
|
sc_prohibit_always-defined_macros \
|
|
sc_require_config_h \
|
|
sc_require_config_h_first
|
|
|
|
local-checks-to-skip = \
|
|
$(local-checks-to-fix) \
|
|
sc_GPL_version \
|
|
sc_cast_of_x_alloc_return_value \
|
|
sc_trailing_blank \
|
|
sc_unmarked_diagnostics
|
|
|
|
# GPL_version: checks for GPLv3, which we don't use
|
|
# cast_of_x_alloc_return_value:
|
|
# We support C++ compilation which does require casting here.
|
|
# trailing_blank: flags valid rfc3676 separators
|
|
# unmarked_diagnostics: libtool isn't internationalized
|
|
|
|
|
|
# Check for correct usage of $cc_basename in libtool.m4.
|
|
sc_libtool_m4_cc_basename:
|
|
@sed -n '/case \$$cc_basename in/,/esac/ { \
|
|
/^[ ]*[a-zA-Z][a-zA-Z0-9+]*[^*][ ]*)/p; \
|
|
}' '$(srcdir)/$(macro_dir)/libtool.m4' | grep . && { \
|
|
msg="\$$cc_basename matches should include a trailing \`*'." \
|
|
$(_sc_say_and_exit) } || :
|
|
|
|
# Check for uses of Xsed without corresponding echo "X
|
|
exclude_file_name_regexp--sc_prohibit_Xsed_without_X = ^cfg.mk$$
|
|
sc_prohibit_Xsed_without_X:
|
|
@files=$$($(VC_LIST_EXCEPT)); \
|
|
if test -n "$$files"; then \
|
|
grep -nE '\$$Xsed' $$files | grep -vE '(func_echo_all|\$$ECHO) \\*"X' && { \
|
|
msg='occurrences of `$$Xsed'\'' without `$$ECHO "X'\'' on the same line' \
|
|
$(_sc_say_and_exit) } || :; \
|
|
else :; \
|
|
fi || :
|
|
|
|
# Use a consistent dirname and basename idiom.
|
|
sc_prohibit_bare_basename:
|
|
@prohibit='\|[ ]*\$$(base|dir)name' \
|
|
halt='use `|$$SED "$$basename"'\'' instead of `|$$basename'\' \
|
|
$(_sc_search_regexp)
|
|
|
|
sc_prohibit_basename_with_dollar_sed:
|
|
@prohibit='(base|dir)name="?(\$$SED|sed)[ "]' \
|
|
halt='use `basename='\''s|^.*/||'\'' instead of `basename="$$SED...' \
|
|
$(_sc_search_regexp)
|
|
|
|
# Check for using `[' instead of `test'.
|
|
exclude_file_name_regexp--sc_prohibit_bracket_as_test = ^cfg.mk$$
|
|
sc_prohibit_bracket_as_test:
|
|
@prohibit='if[ ]+\[' \
|
|
halt="use \`if test' instead of \`if ['" \
|
|
$(_sc_search_regexp)
|
|
|
|
# Check for quotes within backquotes within quotes "`"bar"`"
|
|
exclude_file_name_regexp--sc_prohibit_nested_quotes = ^cfg.mk$$
|
|
sc_prohibit_nested_quotes:
|
|
@prohibit='"[^`"]*`[^"`]*"[^"`]*".*`[^`"]*"' \
|
|
halt='found nested double quotes' \
|
|
$(_sc_search_regexp_or_exclude)
|
|
|
|
# Check for using shift after set dummy (same or following line).
|
|
exclude_file_name_regexp--sc_prohibit_set_dummy_without_shift = ^cfg.mk$$
|
|
sc_prohibit_set_dummy_without_shift:
|
|
@files=$$($(VC_LIST_EXCEPT)); \
|
|
if test -n "$$files"; then \
|
|
grep -nE '(set dummy|shift)' $$files | \
|
|
sed -n '/set[ ][ ]*dummy/{ \
|
|
/set.*dummy.*;.*shift/d; \
|
|
N; \
|
|
/\n.*shift/D; \
|
|
p; \
|
|
}' | grep -n . && { \
|
|
msg="use \`shift' after \`set dummy'" \
|
|
$(_sc_say_and_exit) } || :; \
|
|
else :; \
|
|
fi || :
|
|
|
|
# Check for using set -- instead of set dummy
|
|
exclude_file_name_regexp--sc_prohibit_set_minus_minus = ^cfg.mk$$
|
|
sc_prohibit_set_minus_minus:
|
|
@prohibit='set[ ]+--[ ]+' \
|
|
halt="use \`set dummy ...' instead of \`set -- ...'" \
|
|
$(_sc_search_regexp)
|
|
|
|
# Check for using test X"... instead of test "X...
|
|
exclude_file_name_regexp--sc_prohibit_test_X = ^cfg.mk$$
|
|
sc_prohibit_test_X:
|
|
@prohibit='test[ ]+(![ ])?(-.[ ]+)?[Xx]["'\'']' \
|
|
halt='use `test "X..."'\'' instead of `test X"'\' \
|
|
$(_sc_search_regexp)
|
|
|
|
# Check for bad binary operators.
|
|
sc_prohibit_test_binary_operators:
|
|
@prohibit='if[ ]+["'\'']?\$$[^ ]+[ ]+(=|-[lg][te]|-eq|-ne)' \
|
|
halt="Use \`if test \$$something =' instead of \`if \$$something ='" \
|
|
$(_sc_search_regexp)
|
|
|
|
# Check for using test $... instead of test "$...
|
|
exclude_file_name_regexp--sc_prohibit_test_dollar = ^cfg.mk$$
|
|
sc_prohibit_test_dollar:
|
|
@prohibit='test[ ]+(![ ])?(-.[ ]+)?X?\$$[^?#]' \
|
|
halt='use `test "$$..."'\'' instead of `test $$'\' \
|
|
$(_sc_search_regexp)
|
|
|
|
# Never use test -e.
|
|
exclude_file_name_regexp--sc_prohibit_test_minus_e = ^cfg.mk$$
|
|
sc_prohibit_test_minus_e:
|
|
@prohibit='test[ ]+(![ ])?-e' \
|
|
halt="use \`test -f' instead of \`test -e'" \
|
|
$(_sc_search_regexp)
|
|
|
|
# Check for bad unary operators.
|
|
sc_prohibit_test_unary_operators:
|
|
@prohibit='if[ ]+-[a-z]' \
|
|
halt="use \`if test -X' instead of \`if -X'" \
|
|
$(_sc_search_regexp)
|
|
|
|
# Check for opening brace on next line in shell function definition.
|
|
exclude_file_name_regexp--sc_require_function_nl_brace = (^HACKING|\.[ch])$$
|
|
sc_require_function_nl_brace:
|
|
@for file in $$($(VC_LIST_EXCEPT)); do \
|
|
sed -n '/^func_[^ ]*[ ]*(/{ \
|
|
N; \
|
|
/^func_[^ ]* ()\n{$$/d; \
|
|
p; \
|
|
}' $$file | grep -E . && { \
|
|
msg="found malformed function_definition in $$file" \
|
|
$(_sc_say_and_exit) } || :; \
|
|
done
|
|
|
|
sc_trailing_blank-non-rfc3676:
|
|
@prohibit='([^-][^-][ ][ ]*|^[ ][ ]*)$$' \
|
|
halt='found trailing blank(s)' \
|
|
$(_sc_search_regexp)
|
|
|
|
# Avoid useless quotes around assignments with no shell metacharacters.
|
|
# Backtick and dollar expansions are not resplit on the RHS of an
|
|
# assignment, so those metachars are not listed in the prohibit regex,
|
|
# although @ is listed, since it most likely indicates that something
|
|
# will be spliced in before the shell executes, and it may need to be
|
|
# quoted if it contains any metacharacters after splicing.
|
|
define _sc_search_regexp_or_exclude
|
|
files=$$($(VC_LIST_EXCEPT)); \
|
|
if test -n "$$files"; then \
|
|
grep -nE "$$prohibit" $$files | grep -v '## exclude from $@' \
|
|
&& { msg="$$halt" $(_sc_say_and_exit) } || :; \
|
|
else :; \
|
|
fi || :;
|
|
endef
|
|
|
|
exclude_file_name_regexp--sc_useless_braces_in_variable_derefs = /cvsu$$
|
|
sc_useless_braces_in_variable_derefs:
|
|
@prohibit='\$${[0-9A-Za-z_]+}[^0-9A-Za-z_]' \
|
|
halt='found spurious braces around variable dereference' \
|
|
$(_sc_search_regexp)
|
|
|
|
sc_useless_quotes_in_assignment:
|
|
@prohibit='^[ ]*[A-Za-z_][A-Za-z0-9_]*="[^ !#&()*;<>?@~^{|}]*"$$' \
|
|
halt='found spurious quotes around assignment value' \
|
|
$(_sc_search_regexp_or_exclude)
|
|
|
|
# Avoid useless quotes around case arguments such as:
|
|
# case "$foo" in ...
|
|
exclude_file_name_regexp--sc_useless_quotes_in_case = ^cfg.mk$$
|
|
sc_useless_quotes_in_case:
|
|
@prohibit='case "[^ "]*"[ ][ ]*in' \
|
|
halt='found spurious quotes around case argument' \
|
|
$(_sc_search_regexp)
|
|
|
|
# List syntax-check exempted files.
|
|
exclude_file_name_regexp--sc_bindtextdomain = ^tests/.*demo[0-9]*/.*\.c$$
|
|
exclude_file_name_regexp--sc_error_message_uppercase = \
|
|
^$(_build-aux)/cvsu$$
|
|
exclude_file_name_regexp--sc_program_name = ^tests/.*demo[0-9]*/.*\.c$$
|
|
exclude_file_name_regexp--sc_prohibit_strcmp = \
|
|
^(doc/libtool.texi|tests/.*demo/.*\.c)$$
|
|
exclude_file_name_regexp--sc_prohibit_test_minus_ao = \
|
|
^m4/libtool.m4$$
|
|
exclude_file_name_regexp--sc_space_tab = \.diff$$
|
|
exclude_file_name_regexp--sc_trailing_blank-non-rfc3676 = \.diff$$
|