[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 2/2] Move portable shell tests from the old to the new testsuite.
From: |
Peter Rosin |
Subject: |
[PATCH 2/2] Move portable shell tests from the old to the new testsuite. |
Date: |
Fri, 17 Sep 2010 12:37:30 +0200 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.8) Gecko/20100802 Thunderbird/3.1.2 |
>From c6195aeb806270200950a86cda955ca674e680f1 Mon Sep 17 00:00:00 2001
From: Peter Rosin <address@hidden>
Date: Fri, 17 Sep 2010 12:28:51 +0200
Subject: [PATCH 2/2] Move portable shell tests from the old to the new
testsuite.
* tests/sh.test: Move this...
* tests/sh.at: ...to here, and adjust to the new testsuite.
* Makefile.am: Update.
---
ChangeLog | 5 ++
Makefile.am | 2 +-
tests/sh.at | 143 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
tests/sh.test | 134 -----------------------------------------------------
4 files changed, 149 insertions(+), 135 deletions(-)
create mode 100644 tests/sh.at
delete mode 100755 tests/sh.test
diff --git a/ChangeLog b/ChangeLog
index a6f7d42..ba1c1fe 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
2010-09-17 Peter Rosin <address@hidden>
+ Move portable shell tests from the old to the new testsuite.
+ * tests/sh.test: Move this...
+ * tests/sh.at: ...to here, and adjust to the new testsuite.
+ * Makefile.am: Update.
+
* tests/sh.test: Detect missing 'test' in 'if "$foo" = ...'.
2010-09-16 Ralf Wildenhues <address@hidden>
diff --git a/Makefile.am b/Makefile.am
index dcd0876..db9ff78 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -440,6 +440,7 @@ dist-hook:
# The testsuite files are evaluated in the order given here.
TESTSUITE = tests/testsuite
TESTSUITE_AT = tests/testsuite.at \
+ tests/sh.at \
tests/getopt-m4sh.at \
tests/libtoolize.at \
tests/help.at \
@@ -687,7 +688,6 @@ COMMON_TESTS = \
tests/nomode.test \
tests/objectlist.test \
tests/quote.test \
- tests/sh.test \
tests/suffix.test \
tests/tagtrace.test \
tests/cdemo-static.test \
diff --git a/tests/sh.at b/tests/sh.at
new file mode 100644
index 0000000..7c5633c
--- /dev/null
+++ b/tests/sh.at
@@ -0,0 +1,143 @@
+# sh.at - check for some nonportable or dubious or undesired shell
+# constructs in shell scripts.
+#
+# Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2010 Free
+# Software Foundation, Inc.
+# Written by Gary V. Vaughan, 2003
+#
+# 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 downloaded from http://www.gnu.org/licenses/gpl.html,
+# or obtained by writing to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+####
+
+AT_BANNER([Maintainer checks.])
+
+AT_SETUP([portable m4sh scripts])
+
+m4dir=$top_srcdir/libltdl/m4
+auxdir=$top_srcdir/libltdl/config
+scripts="$auxdir/ltmain.m4sh $top_srcdir/libtoolize.m4sh"
+
+# Check all the "portable" shell scripts.
+status=:
+
+[
+# Check for bad binary operators.
+if $EGREP -n -e 'if[ ]+["'\'']?\$[^ ]+[ ]+(=|-[lg][te]|-eq|-ne)'
$scripts; then
+ echo "use \`if test \$something =' instead of \`if \$something ='"
+ status=false
+fi
+
+# Check for bad unary operators.
+if $EGREP -n -e 'if[ ]+-' $scripts; then
+ echo "use \`if test -X' instead of \`if -X'"
+ status=false
+fi
+
+# Check for using `@<:@' instead of `test'. @<:@ is a square bracket.
+if $EGREP -n -e 'if[ ]+\@<:@' $scripts; then
+ echo "use \`if test' instead of \`if @<:@'"
+ status=false
+fi
+
+# Check for using test X... instead of test "X...
+if $EGREP -n -e 'test[ ]+(![ ])?(-.[ ]+)?X' $scripts; then
+ echo "use \`test \"X...\"' instead of \`test X'"
+ status=false
+fi
+
+# Check for using test $... instead of test "$...
+if $EGREP -n -e 'test[ ]+(![ ])?(-.[ ]+)?X?\$' $scripts; then
+ echo "use \`test \"\$...\"' instead of \`test \$'"
+ status=false
+fi
+
+# Never use test -e.
+if $EGREP -n -e 'test[ ]+(![ ])?-e' $scripts; then
+ echo "use \`test -f' instead of \`test -e'"
+ status=false
+fi
+
+# Check for uses of Xsed without corresponding echo "X
+if $EGREP -n -e '\$Xsed' $scripts | $EGREP -v -n -e '\$ECHO \\*"X'; then
+ echo "occurrences of \`\$Xsed\' without \`echo \"X\' on the same line"
+ status=false
+fi
+
+# Check for quotes within backquotes within quotes "`"bar"`"
+if $EGREP -n -e '"[^`"]*`[^"`]*"[^"`]*".*`[^`"]*"' $scripts | \
+ $EGREP -v "### testsuite: skip nested quoting test$"; then
+ echo "nested quotes are dangerous"
+ status=false
+fi
+
+# Check for using set -- instead of set dummy
+if $EGREP -n -e 'set[ ]+--[ ]+' $scripts; then
+ echo "use \`set dummy ...' instead of \`set -- ...'"
+ status=false
+fi
+
+# Check for using shift after set dummy (same or following line).
+for s in $scripts
+do
+ if $SED -n '
+ /set[ ][ ]*dummy/{
+ /set.*dummy.*;.*shift/d
+ N
+ /\n.*shift/D
+ p
+ }' "$s" | $EGREP .; then
+ echo "use \`shift' after \`set dummy' in $s"
+ status=false
+ fi
+done
+
+# Check for opening brace on next line in shell function definition.
+# redirect stderr so we also barf when sed issues diagnostics.
+for s in $scripts
+do
+ if $SED -n '
+ /^func_.*(/{
+ N
+ /^func_[^ ]* ()\n{$/d
+ p
+ }' "$s" 2>&1 | $EGREP .; then
+ echo "Function definitions should look like this in $s:
+func_foo ()
+{
+ # ...
+}"
+ status=false
+ fi
+done
+
+# Check for correct usage of $cc_basename.
+# redirect stderr so we also barf when sed issues diagnostics.
+for s in "$m4dir/libtool.m4"
+do
+ if $SED -n '/case \$cc_basename in/,/esac/ {
+ /^[ ]*[a-zA-Z][a-zA-Z0-9+]*[^*][ ]*)/p
+ }' $s 2>&1 | $EGREP .; then
+ echo "\$cc_basename matches should include a trailing \`*' in $s."
+ status=false
+ fi
+done
+]
+
+AT_CHECK([$status], [], [ignore])
+
+AT_CLEANUP
diff --git a/tests/sh.test b/tests/sh.test
deleted file mode 100755
index 5324b31..0000000
--- a/tests/sh.test
+++ /dev/null
@@ -1,134 +0,0 @@
-#! /bin/sh
-# sh.test - check for some nonportable or dubious or undesired shell
-# constructs in shell scripts.
-#
-# Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2010 Free
-# Software Foundation, Inc.
-# Written by Gary V. Vaughan, 2003
-#
-# 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 downloaded from http://www.gnu.org/licenses/gpl.html,
-# or obtained by writing to the Free Software Foundation, Inc.,
-# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
-####
-
-. tests/defs || exit 1
-
-# Check all the "portable" shell scripts.
-status=$EXIT_SUCCESS
-
-# Check for bad binary operators.
-if $EGREP -n -e 'if[ ]+["'\'']?\$[^ ]+[ ]+(=|-[lg][te]|-eq|-ne)'
$scripts; then
- echo "use \`if test \$something =' instead of \`if \$something ='"
- status=$EXIT_FAILURE
-fi
-
-# Check for bad unary operators.
-if $EGREP -n -e 'if[ ]+-' $scripts; then
- echo "use \`if test -X' instead of \`if -X'"
- status=$EXIT_FAILURE
-fi
-
-# Check for using `[' instead of `test'.
-if $EGREP -n -e 'if[ ]+\[' $scripts; then
- echo "use \`if test' instead of \`if ['"
- status=$EXIT_FAILURE
-fi
-
-# Check for using test X... instead of test "X...
-if $EGREP -n -e 'test[ ]+(![ ])?(-.[ ]+)?X' $scripts; then
- echo "use \`test \"X...\"' instead of \`test X'"
- status=$EXIT_FAILURE
-fi
-
-# Check for using test $... instead of test "$...
-if $EGREP -n -e 'test[ ]+(![ ])?(-.[ ]+)?X?\$' $scripts; then
- echo "use \`test \"\$...\"' instead of \`test \$'"
- status=$EXIT_FAILURE
-fi
-
-# Never use test -e.
-if $EGREP -n -e 'test[ ]+(![ ])?-e' $scripts; then
- echo "use \`test -f' instead of \`test -e'"
- status=$EXIT_FAILURE
-fi
-
-# Check for uses of Xsed without corresponding echo "X
-if $EGREP -n -e '\$Xsed' $scripts | $EGREP -v -n -e '\$ECHO \\*"X'; then
- echo "occurrences of \`\$Xsed\' without \`echo \"X\' on the same line"
- status=$EXIT_FAILURE
-fi
-
-# Check for quotes within backquotes within quotes "`"bar"`"
-if $EGREP -n -e '"[^`"]*`[^"`]*"[^"`]*".*`[^`"]*"' $scripts | \
- $EGREP -v "### testsuite: skip nested quoting test$"; then
- echo "nested quotes are dangerous"
- status=$EXIT_FAILURE
-fi
-
-# Check for using set -- instead of set dummy
-if $EGREP -n -e 'set[ ]+--[ ]+' $scripts; then
- echo "use \`set dummy ...' instead of \`set -- ...'"
- status=$EXIT_FAILURE
-fi
-
-# Check for using shift after set dummy (same or following line).
-for s in $scripts
-do
- if $SED -n '
- /set[ ][ ]*dummy/{
- /set.*dummy.*;.*shift/d
- N
- /\n.*shift/D
- p
- }' "$s" | $EGREP .; then
- echo "use \`shift' after \`set dummy' in $s"
- status=$EXIT_FAILURE
- fi
-done
-
-# Check for opening brace on next line in shell function definition.
-# redirect stderr so we also barf when sed issues diagnostics.
-for s in $scripts
-do
- if $SED -n '
- /^func_.*(/{
- N
- /^func_[^ ]* ()\n{$/d
- p
- }' "$s" 2>&1 | $EGREP .; then
- echo "Function definitions should look like this in $s:
-func_foo ()
-{
- # ...
-}"
- status=$EXIT_FAILURE
- fi
-done
-
-# Check for correct usage of $cc_basename.
-# redirect stderr so we also barf when sed issues diagnostics.
-for s in "$m4dir/libtool.m4"
-do
- if $SED -n '/case \$cc_basename in/,/esac/ {
- /^[ ]*[a-zA-Z][a-zA-Z0-9+]*[^*][ ]*)/p
- }' $s 2>&1 | $EGREP .; then
- echo "\$cc_basename matches should include a trailing \`*' in $s."
- status=$EXIT_FAILURE
- fi
-done
-
-exit $status
--
1.7.1