syntax-check: fix violations in tests of sc_prohibit_test_const_follows_var.

To safely use a non-literal first argument to 'test', you must always
prepend a literal non-'-' character, but ofter the second operand is a
constant that doesn't begin with a '-' already, so always use 'test a =
"$b"' instead of noisy 'test "X$b" = Xa'.
* tests/bindir.at, tests/defs.m4sh, tests/demo-relink.test,
tests/demo/configure.ac, tests/depdemo-relink.test, tests/destdir.at,
tests/duplicate_conv.at, tests/fail.at, tests/help.at,
tests/libtoolize.at, tests/link-2.test, tests/link-order2.at,
tests/lt_dlopenext.at, tests/mdemo/configure.ac,
tests/mdemo2/configure.ac, tests/need_lib_prefix.at, tests/nocase.at,
tests/pdemo/configure.ac, tests/pic_flag.at, tests/search-path.at,
tests/shlibpath.at, tests/static.at, tests/sysroot.at,
tests/tagtrace.test, tests/testsuite.at, tests/with-pic.at: Swap
operands and remove resulting spurious characters.

Signed-off-by: Gary V. Vaughan <gary@gnu.org>

diff --git a/tests/bindir.at b/tests/bindir.at
index fab30aa378e9e779ed7ad004692cd9b1b43a5b5e..063404b71f907bb794a42781d72329022da320ac 100644 (file)
--- a/tests/bindir.at
+++ b/tests/bindir.at
@@ -182,7 +182,7 @@ case $host_os in
 esac
 
 eval "`$LIBTOOL --config | grep '^build_libtool_libs='`"
-AT_CHECK([test "$build_libtool_libs" = yes || exit 77])
+AT_CHECK([test yes = "$build_libtool_libs" || exit 77])
 
 ####
 # These routines save the PATH before a test and restore it after,

diff --git a/tests/defs.m4sh b/tests/defs.m4sh
index bf7127d791e468e220b6a7d660d27cfe5a38d84f..0b9ed38f9e12ff94416a41ec8b977b1da5bf096b 100644 (file)
--- a/tests/defs.m4sh
+++ b/tests/defs.m4sh
@@ -329,7 +329,7 @@ func_exec ()
     if eval $my_program $my_exp_output; then :
     else
       shift
-      test "x$1" = x || shift
+      test -z "$1" || shift
       func_error "$0: cannot execute $my_program ${1+$@}"
 
       if test "$build" != "$host"; then

diff --git a/tests/demo-relink.test b/tests/demo-relink.test
index 02cc4b590ba6d9eb84daf266d3a011968d89ee9e..4b32fba6a2e7e8968af32f5c4892b729456c6f3e 100755 (executable)
--- a/tests/demo-relink.test
+++ b/tests/demo-relink.test
@@ -67,7 +67,7 @@ rm -f libhello.la "$objdir"/libhello.*
 func_msg "running demo/hell"
 if ./hell$EXEEXT; then
   :
-elif test "x,$hardcode_action,$hardcode_direct" = x,relink,yes; then
+elif test relink,yes = "$hardcode_action,$hardcode_direct"; then
   func_msg "Ok, uninstalled programs fail after uninstalled libraries are removed."
   func_msg "This works in other configurations, but not in this particular one."
 else
@@ -87,12 +87,12 @@ else
   func_msg "Failed, as expected"
 fi
 
-if test "x$hardcode_action" = xrelink; then
+if test relink = "$hardcode_action"; then
   func_msg "Exiting: install-time relinking is required"
   exit $EXIT_SUCCESS
 fi
 
-if test "$shlibpath_overrides_runpath" != yes; then
+if test yes != "$shlibpath_overrides_runpath"; then
   rm -f $objdir/lt-hell$EXEEXT || exit $EXIT_FAILURE
   cp $objdir/hell$EXEEXT $objdir/lt-hell$EXEEXT || exit $EXIT_FAILURE
   func_msg "running demo/hell with installed libhello.la"

diff --git a/tests/demo/configure.ac b/tests/demo/configure.ac
index 7a844415031e5011a7f3d855547b5875ce169035..27a0da569932d9676ca03c46d0985a5226ca07bc 100644 (file)
--- a/tests/demo/configure.ac
+++ b/tests/demo/configure.ac
@@ -51,13 +51,13 @@ LT_INIT([dlopen win32-dll])
 AC_SUBST([LIBTOOL_DEPS])
 
 STATIC=
-test "X$enable_static" = Xyes && STATIC=-static
+test yes = "$enable_static" && STATIC=-static
 AC_SUBST([STATIC])
 
 case $lt_cv_sys_global_symbol_pipe in
   ?*) binary_helldl=yes ;;
 esac
-AM_CONDITIONAL([BINARY_HELLDL], [test "X$binary_helldl" = Xyes])
+AM_CONDITIONAL([BINARY_HELLDL], [test yes = "$binary_helldl"])
 
 
 ## --------------------------- ##

diff --git a/tests/depdemo-relink.test b/tests/depdemo-relink.test
index f8d5a11af133cffbca658dbe15f221abb900ce1b..0d01279c35c2b3ea518f1630e1d339947c84a72d 100755 (executable)
--- a/tests/depdemo-relink.test
+++ b/tests/depdemo-relink.test
@@ -82,7 +82,7 @@ if ./depdemo$EXEEXT ||
    # but it's definitely not enough of a reason for the test to fail.
    ./depdemo$EXEEXT -alt; then
   :
-elif test "x,$hardcode_action,$hardcode_direct" = x,relink,yes; then
+elif test relink,yes = "$hardcode_action,$hardcode_direct"; then
   func_msg "Ok, uninstalled programs fail after uninstalled libraries are removed"
   func_msg "This works in other configurations, but not in this particular one"
 elif test "$build" != "$host"; then
@@ -109,12 +109,12 @@ else
   func_msg "Failed, as expected"
 fi
 
-if test "x$hardcode_action" = xrelink; then
+if test relink = "$hardcode_action"; then
   func_msg "Exiting: install-time relinking is required"
   exit $EXIT_SUCCESS
 fi
 
-if test "$shlibpath_overrides_runpath" != yes; then
+if test yes != "$shlibpath_overrides_runpath"; then
   rm -f $objdir/lt-depdemo || exit $EXIT_FAILURE
   cp $objdir/depdemo $objdir/lt-depdemo || exit $EXIT_FAILURE
   func_msg "running depdemo/depdemo with installed libl3.la"

diff --git a/tests/destdir.at b/tests/destdir.at
index af1cb22c21b5ede25ef286b84693325eafc46fbc..dac5351fcc2b41f0b1e942868917d78704c0212e 100644 (file)
--- a/tests/destdir.at
+++ b/tests/destdir.at
@@ -127,7 +127,7 @@ done
 LT_AT_EXEC_CHECK([$bindir/m$EXEEXT])
 
 # TODO: make this more portable:
-if test "$OBJDUMP" != false && ($OBJDUMP -p $bindir/m$EXEEXT) >/dev/null 2>&1; then
+if test false != "$OBJDUMP" && ($OBJDUMP -p $bindir/m$EXEEXT) >/dev/null 2>&1; then
   AT_CHECK([$OBJDUMP -p $bindir/m$EXEEXT | $EGREP -i "R(UN)?PATH.*$DESTDIR"], [1])
   . $libdir/liba.la
   set x $library_names

diff --git a/tests/duplicate_conv.at b/tests/duplicate_conv.at
index 5829bcc7fffc74cd5daf2eeb8eb4717528596ddd..b1035f8db0ab8ff34323e3bba7bbf360ecb18c6f 100644 (file)
--- a/tests/duplicate_conv.at
+++ b/tests/duplicate_conv.at
@@ -77,7 +77,7 @@ AT_CHECK([$LIBTOOL --mode=link $CC $CFLAGS $LDFLAGS -o main$EXEEXT main.$OBJEXT
 LT_AT_EXEC_CHECK([./main],[0],[ignore],[ignore])
 $LIBTOOL --mode=clean rm -f libcee.la
 
-AT_CHECK([test "x$reload_cmds" = xfalse && exit 77], [1])
+AT_CHECK([test false = "$reload_cmds" && exit 77], [1])
 
 # Test whether this works with reloadable objects as well.
 AT_CHECK([$LIBTOOL --mode=link --tag=CC $CC $CFLAGS $LDFLAGS -o cee.$OBJEXT c.lo a/liba.la b/liba.la],

diff --git a/tests/fail.at b/tests/fail.at
index 82bda6cbf42c04f04fd6e3341f57468363cd99a4..cdc2df791264dc79017b822863c4d5a61d4918a7 100644 (file)
--- a/tests/fail.at
+++ b/tests/fail.at
@@ -103,8 +103,8 @@ case $build_libtool_libs in yes)
   $LIBTOOL --mode=compile $CC $CPPFLAGS $CFLAGS -c c.c
   (
     . ./c.lo
-    test "$pic_object" != none && echo choke me >"$pic_object"
-    test "$non_pic_object" != none && echo choke me >"$non_pic_object"
+    test none != "$pic_object" && echo choke me >"$pic_object"
+    test none != "$non_pic_object" && echo choke me >"$non_pic_object"
   )
   FAIL_CHECK([$LIBTOOL --mode=link $CC $CFLAGS $LDFLAGS -o liba.la a.lo b.lo c.lo -rpath /foo])
   AT_CHECK([test -f liba.la], [1])

diff --git a/tests/help.at b/tests/help.at
index 067c746ac96ac88c1942aba74f6a7145867665ff..3d80900612ff57d42ba6b71109ca1832cc7ed000 100644 (file)
--- a/tests/help.at
+++ b/tests/help.at
@@ -121,7 +121,7 @@ AT_CHECK([case "$LIBTOOL $CC $CPPFLAGS $CFLAGS $LDFLAGS " in ]dnl
 
 check_trace ()
 {
-  if test "X$trace" = X--debug; then
+  if test X--debug = "X$trace"; then
     AT_CHECK([grep 'enabling shell trace mode' stdout stderr], [0], [ignore])
     AT_CHECK([grep ' --mode' stderr], [0], [ignore])
   else
@@ -164,7 +164,7 @@ for trace in '' --debug; do
   AT_CHECK([$orig_LIBTOOL --mode=install $lt_INSTALL liba.la libb.la $libdir],
           [], [stdout], [stderr])
   if grep ': relinking ' stdout stderr; then
-    if test "X$trace" = X--debug; then
+    if test X--debug = "X$trace"; then
       AT_CHECK([grep ' --mode=relink' stdout stderr | grep ' --debug '],
               [0], [ignore])
     else

diff --git a/tests/libtoolize.at b/tests/libtoolize.at
index 531b879ea7445af935ee9f793fc4a377155f00f3..dd5f918edfcecba540ef980dca7c544fa2f16650 100644 (file)
--- a/tests/libtoolize.at
+++ b/tests/libtoolize.at
@@ -724,7 +724,7 @@ func_serial ()
     # in the file that AC_DEFUNs MACRO_REGEX.
     my_serial=
     if test -z "$my_macro_regex" ||
-       test "$my_filename" = aclocal.m4 ||
+       test aclocal.m4 = "$my_filename" ||
        test "$my_macro_regex" = `echo "$my_filename" | $SED "$basename"` ||
        func_grep '^AC_DEFUN(\@<:@'"$my_macro_regex" "$my_filename"
     then

diff --git a/tests/link-2.test b/tests/link-2.test
index dcdf41ad290345780befe22fa9df5f5984d6d121..d7c18959b6b0f7de8e382ea9304388ff5f098223 100755 (executable)
--- a/tests/link-2.test
+++ b/tests/link-2.test
@@ -38,7 +38,7 @@ res=$?
 
 rm -f hell.lo
 
-test "$res" -eq 0 || exit $EXIT_FAILURE
+test 0 -eq "$res" || exit $EXIT_FAILURE
 
 echo "$linkresult"
 case $linkresult in

diff --git a/tests/link-order2.at b/tests/link-order2.at
index 7d15643d5bcd2a15aeebbcc2b20a6a3fb137e704..abf4492b3eaa2da3f71fa9f2a40c617eaa26a3a1 100644 (file)
--- a/tests/link-order2.at
+++ b/tests/link-order2.at
@@ -104,7 +104,7 @@ for type_of_depdepl in libtool non-libtool; do
     # Simulate a non-Libtool system library.
     rm $deflibdir/liba1.la
     addpath=$deflibdir
-    if test "$shlibpath_var" = PATH; then
+    if test PATH = "$shlibpath_var"; then
       addpath=$defbindir
     fi
     sep=
@@ -114,9 +114,9 @@ for type_of_depdepl in libtool non-libtool; do
   fi
   for static in '' -static-libtool-libs; do
     case `$LIBTOOL --features` in
-    *disable\ static\ libraries*) test "$static" = '' || continue;;
+    *disable\ static\ libraries*) test -z "$static" || continue;;
     esac
-    test "$type_of_depdepl,$static" = "non-libtool,-static-libtool-libs" &&
+    test non-libtool,-static-libtool-libs = "$type_of_depdepl,$static" &&
        static=-all-static
     $LIBTOOL --mode=link $CC $CFLAGS $LDFLAGS $static -o liba1.la a1.lo -rpath $libdir
     $LIBTOOL --mode=link $CC $CFLAGS $LDFLAGS $static -o libb.la b.lo liba1.la -rpath $libdir

diff --git a/tests/lt_dlopenext.at b/tests/lt_dlopenext.at
index f40e77a67df58c3ca39ff592689c2afbdae5c479..36906ba450ff33e513436c06678c20654f6b30fd 100644 (file)
--- a/tests/lt_dlopenext.at
+++ b/tests/lt_dlopenext.at
@@ -222,7 +222,7 @@ else
   have=with
 fi
 
-if test "$shlibpath_var" = PATH; then
+if test PATH = "$shlibpath_var"; then
   $unset shlibpath_var || shlibpath_var=
 fi
 

diff --git a/tests/mdemo/configure.ac b/tests/mdemo/configure.ac
index ed9a152fa369f2c2b3ca674534027a16f5c008cf..8476b5d5ed1b3b043b08f3d03cd10dea75199264 100644 (file)
--- a/tests/mdemo/configure.ac
+++ b/tests/mdemo/configure.ac
@@ -55,7 +55,7 @@ LT_INIT([dlopen win32-dll])
 AC_SUBST(LIBTOOL_DEPS)
 
 STATIC=
-test "X$enable_static" = Xyes && STATIC="-static"
+test yes = "$enable_static" && STATIC="-static"
 AC_SUBST([STATIC])
 
 

diff --git a/tests/mdemo2/configure.ac b/tests/mdemo2/configure.ac
index 23ad6df3ed9d6f1a8ae3b5ffba4309048cdb3db6..b25e455599933261777872189c8498632a8b909e 100644 (file)
--- a/tests/mdemo2/configure.ac
+++ b/tests/mdemo2/configure.ac
@@ -51,7 +51,7 @@ LT_INIT([dlopen])
 AC_SUBST(LIBTOOL_DEPS)
 
 STATIC=
-test "X$enable_static" = Xyes && STATIC="-static"
+test yes = "$enable_static" && STATIC="-static"
 AC_SUBST([STATIC])
 
 

diff --git a/tests/need_lib_prefix.at b/tests/need_lib_prefix.at
index 4717e0d352ec5dcf0a057e8945e89be4b3a61ed5..0994c24aeecca0dab3a613249269e1ddab160b11 100644 (file)
--- a/tests/need_lib_prefix.at
+++ b/tests/need_lib_prefix.at
@@ -158,7 +158,7 @@ LDFLAGS=$LDFLAGS
 eval "`$LIBTOOL --config | $EGREP '^(libname_spec)='`"
 name=
 eval libname=\"$libname_spec\"
-AT_CHECK([test "$libname" = lib || exit 77])
+AT_CHECK([test lib = "$libname" || exit 77])
 
 # Create our own libtool, forcing need_lib_prefix setting
 sed 's|^\(need_lib_prefix\)=.*$|\1=unknown|' $LIBTOOL > ./libtool

diff --git a/tests/nocase.at b/tests/nocase.at
index 3cce16ea3e90327e4cf353a5b9f4b81a20bc34e7..c749d1816ced8deac943e6ed79a6f079d2ee68df 100644 (file)
--- a/tests/nocase.at
+++ b/tests/nocase.at
@@ -28,7 +28,7 @@ AT_KEYWORDS([libtool])
 
 eval `$LIBTOOL --config | $EGREP '^(want_nocaseglob|file_magic_glob)='`
 
-AT_CHECK([test "$want_nocaseglob" != yes && ]dnl
+AT_CHECK([test yes != "$want_nocaseglob" && ]dnl
          [test -z "$file_magic_glob" && exit 77],
          [1], [ignore], [ignore])
 

diff --git a/tests/pdemo/configure.ac b/tests/pdemo/configure.ac
index 740603a46d440dde48d35ae62c04011c14904ee3..5fe89c7b8310f2f34f20dea53f74f0075e548f6c 100644 (file)
--- a/tests/pdemo/configure.ac
+++ b/tests/pdemo/configure.ac
@@ -51,13 +51,13 @@ LT_INIT([dlopen])
 AC_SUBST([LIBTOOL_DEPS])
 
 STATIC=
-test "X$enable_static" = Xyes && STATIC=-static
+test yes = "$enable_static" && STATIC=-static
 AC_SUBST([STATIC])
 
 case $lt_cv_sys_global_symbol_pipe in
   ?*) binary_helldl=yes ;;
 esac
-AM_CONDITIONAL([BINARY_HELLDL], [test "X$binary_helldl" = Xyes])
+AM_CONDITIONAL([BINARY_HELLDL], [test yes = "$binary_helldl"])
 
 
 ## ---------------------------- ##

diff --git a/tests/pic_flag.at b/tests/pic_flag.at
index fd9fc9479e0c8a245be685113e5c470e051574af..20beec0ff4802462e984e70773cbf59c394d5af4 100644 (file)
--- a/tests/pic_flag.at
+++ b/tests/pic_flag.at
@@ -44,7 +44,7 @@ if $CXX $CPPFLAGS $CXXFLAGS $CXX_pic_flag -c foo.cpp; then :; else
   CXX_pic_flag=
 fi
 
-AT_CHECK([test "$at_srcdir" != . || exit 77])
+AT_CHECK([test . != "$at_srcdir" || exit 77])
 LT_AT_CONFIGURE([lt_cv_prog_compiler_pic="$C_pic_flag" ]dnl
                [lt_cv_prog_compiler_pic_CXX="$CXX_pic_flag"],
                ["$abs_top_srcdir"/configure --disable-silent-rules])

diff --git a/tests/search-path.at b/tests/search-path.at
index 4050aefae0aaecde195e1a5aa8f5630718990df4..5a01977e881ef3c069cd4258b900ca7af3ed25b2 100644 (file)
--- a/tests/search-path.at
+++ b/tests/search-path.at
@@ -78,7 +78,7 @@ HCURSOR get_cursor (void)
 ]])
 
 eval "`$LIBTOOL --config | grep '^build_libtool_libs='`"
-AT_CHECK([test "$build_libtool_libs" = yes || exit 77])
+AT_CHECK([test yes = "$build_libtool_libs" || exit 77])
 
 AT_CHECK([$LIBTOOL --mode=compile --tag=CC \
          $CC $CPPFLAGS $CFLAGS -o gc.lo -c gc.c || exit 77],

diff --git a/tests/shlibpath.at b/tests/shlibpath.at
index 45dede8534d2fcf77aa5dfe95fc50bb1a5a47e94..ad82fdf44029ff3ad9e998b931122e518539220c 100644 (file)
--- a/tests/shlibpath.at
+++ b/tests/shlibpath.at
@@ -53,7 +53,7 @@ eval `$LIBTOOL --config | $EGREP '^(shlibpath_var|shlibpath_overrides_runpath)='
 # No point checking a system with static libraries:
 LT_AT_EXEC_CHECK([./m], [1], [ignore], [ignore], [|| exit 1 && exit 77])
 
-if test "$shlibpath_var" = PATH; then
+if test PATH = "$shlibpath_var"; then
   addpath=`pwd`/moved/bin
 else
   addpath=`pwd`/moved/lib
@@ -62,7 +62,7 @@ sep=
 eval test -n \"\$$shlibpath_var\" && sep=:
 eval $shlibpath_var='$addpath$sep$'$shlibpath_var
 export $shlibpath_var
-if test "$shlibpath_overrides_runpath" != no; then
+if test no != "$shlibpath_overrides_runpath"; then
   LT_AT_EXEC_CHECK([./m], [0], [ignore], [ignore])
 else
   LT_AT_EXEC_CHECK([./m], [1], [ignore], [ignore], [|| exit 1])

diff --git a/tests/static.at b/tests/static.at
index decde56d89b045db901872b76b689d301b0e6a95..a2e87c959b961a85241e7b5438bc22c76169549d 100644 (file)
--- a/tests/static.at
+++ b/tests/static.at
@@ -130,7 +130,7 @@ func_fix_path ()
   # against a broken library but the good one would come later in the PATH.
   # So we let the caller of this function set the order: the "other" two
   # come first.
-  if test "$shlibpath_var" = PATH; then
+  if test PATH = "$shlibpath_var"; then
     save_PATH=$PATH
     sep=
     test -z "$PATH" || sep=:
@@ -141,7 +141,7 @@ func_fix_path ()
 
 func_restore_path ()
 {
-  if test "$shlibpath_var" = PATH; then
+  if test PATH = "$shlibpath_var"; then
     PATH=$save_PATH
   fi
 }
@@ -244,11 +244,11 @@ for withdep in no yes; do
   done
 
   ### install the libraries.
-  test "$withdep" = yes && $LIBTOOL --mode=install cp a1/liba1dep.la $libdir1/liba1dep.la
+  test yes = "$withdep" && $LIBTOOL --mode=install cp a1/liba1dep.la $libdir1/liba1dep.la
   $LIBTOOL --mode=install cp a1/liba1.la $libdir1/liba1.la
   $LIBTOOL --mode=install cp a3/liba3.la $libdir3/liba3.la
   $LIBTOOL --mode=clean rm -f a1/liba1.la a3/liba3.la
-  test "$withdep" = yes && $LIBTOOL --mode=clean rm -f a1/liba1dep.la
+  test yes = "$withdep" && $LIBTOOL --mode=clean rm -f a1/liba1dep.la
   # simulate a non-libtool lib:
   rm -f $libdir3/liba3.la
 

diff --git a/tests/sysroot.at b/tests/sysroot.at
index 063d05189315d4d4f758ad9ce84325e3c583276e..e96ea0ebaaf6a0d9bafaf8ff8097d7f072a1f6e0 100644 (file)
--- a/tests/sysroot.at
+++ b/tests/sysroot.at
@@ -67,7 +67,7 @@ LDFLAGS="$LDFLAGS --sysroot=$sysroot -no-undefined"
 configure_options="$configure_options --with-sysroot=$sysroot --prefix=$prefix"
 
 #???
-if test "$shlibpath_var" = PATH; then
+if test PATH = "$shlibpath_var"; then
   configure_options="$configure_options --libdir=/$prefix/bin"
 fi
 

diff --git a/tests/tagtrace.test b/tests/tagtrace.test
index 5b189fff25349e4e6d70fc260e7a7f23cd361f56..27bea64d246f3ff645777c060393078efee827cf 100755 (executable)
--- a/tests/tagtrace.test
+++ b/tests/tagtrace.test
@@ -35,10 +35,10 @@ fi
 
 ( cd "$abs_srcdir" && $AUTOCONF --trace 'LT_SUPPORTED_TAG:$1' ) >/dev/null
 ret=$?
-if test "$ret" -eq 63 || test "$ret" -eq 1; then
+if test 63 -eq "$ret" || test 1 -eq "$ret"; then
   func_error "This test requires the same Autoconf version"
   func_skip  "as the one that was used to bootstrap Libtool"
-elif test "$ret" -ne 0; then
+elif test 0 -ne "$ret"; then
   func_fatal_error "\`$AUTOCONF --trace' exited $ret"
 fi
 

diff --git a/tests/testsuite.at b/tests/testsuite.at
index c4f58aec90572a857c478a8ab357262723808eb1..bb4b875cc1ef5526ed4447a7b6f78d4cf1dbf5c5 100644 (file)
--- a/tests/testsuite.at
+++ b/tests/testsuite.at
@@ -29,7 +29,7 @@ m4_divert_push([PREPARE_TESTS])dnl
 : ${AUTOCONF=autoconf}
 : ${AUTOMAKE=automake}
 : ${AUTORECONF=autoreconf}
-test "${ECHO+set}" = set || ECHO=${as_echo-'printf %s\n'}
+test set = "${ECHO+set}" || ECHO=${as_echo-'printf %s\n'}
 : ${ECHO=$as_echo}
 for tool in ACLOCAL AUTOHEADER AUTOCONF AUTOMAKE AUTORECONF; do
   if eval \$$tool --version >/dev/null 2>&1; then :; else eval $tool=no; fi
@@ -112,7 +112,7 @@ AT_CHECK([LT_AT_LIBTOOLIZE([$1])],
 # --------------------------
 m4_define([LT_AT_ACLOCAL],
 [AT_CHECK([$ACLOCAL $1], [0], [ignore], [ignore])
-AT_XFAIL_IF([test "$ACLOCAL" = no])
+AT_XFAIL_IF([test no = "$ACLOCAL"])
 AT_KEYWORDS([automake])
 ])
 
@@ -121,7 +121,7 @@ AT_KEYWORDS([automake])
 # --------------------------
 m4_define([LT_AT_AUTOCONF],
 [AT_CHECK([$AUTOCONF $1], [0], [ignore], [ignore])
-AT_XFAIL_IF([test "$AUTOCONF" = no])
+AT_XFAIL_IF([test no = "$AUTOCONF"])
 AT_KEYWORDS([autoconf])
 ])
 
@@ -131,7 +131,7 @@ AT_KEYWORDS([autoconf])
 m4_define([LT_AT_AUTOMAKE],
 [AT_CHECK([$AUTOMAKE $1], [0], [ignore], [stderr],
   [AT_CHECK([grep 'require .*but have' stderr && (exit 77)], [1])])
-AT_XFAIL_IF([test "$AUTOMAKE" = no])
+AT_XFAIL_IF([test no = "$AUTOMAKE"])
 AT_KEYWORDS([automake])
 ])
 
@@ -140,7 +140,7 @@ AT_KEYWORDS([automake])
 # ---------------------------
 m4_define([LT_AT_AUTOHEADER],
 [AT_CHECK([$AUTOHEADER $1], [0], [ignore], [ignore])
-AT_XFAIL_IF([test "$AUTOHEADER" = no])
+AT_XFAIL_IF([test no = "$AUTOHEADER"])
 AT_KEYWORDS([autoconf])
 ])
 
@@ -295,7 +295,7 @@ m4_define([LT_AT_TAG],
 AT_CHECK([{ test -n "[$]$1" && test "X[$]$1" != Xno; } || (exit 77)])
 m4_case([$1],
   [CXX],
-  [AT_CHECK([test "X$CXX" != Xg++ || (g++ -v >/dev/null 2>&1) || (exit 77)])],
+  [AT_CHECK([test g++ != "$CXX" || (g++ -v >/dev/null 2>&1) || (exit 77)])],
   [GCJ],
   [# There are just too many broken gcj installations out there, either missing
    # libgcj.spec or unable to find it.  Skip the test for them.

diff --git a/tests/with-pic.at b/tests/with-pic.at
index d38f6ffde5493ea5ec4c06aa4c5f66538b9ba23e..915acf598e6d2d768321fc6f8261cc1def7b5a9f 100644 (file)
--- a/tests/with-pic.at
+++ b/tests/with-pic.at
@@ -29,7 +29,7 @@ case " $pic_flag " in
 [*" "[^" "-]* | *" "-[^D]*]) real_pic=: ;;
 esac
 AT_CHECK([$real_pic || exit 77])
-AT_CHECK([test "$at_srcdir" != . || exit 77])
+AT_CHECK([test . != "$at_srcdir" || exit 77])
 
 CONFIGURE=$abs_top_srcdir/tests/demo/configure
 : ${MAKE=make}