test: the instdir-ltlib test is expected to fail.

Discussed in https://bugs.gnu.org/71596.

* t/list-of-tests.mk (handwritten_TESTS): move t/instdir-ltlib.sh
(XFAIL_TESTS): to here. It previously worked by accident
because lib_LTLIBRARIES was misspelled as lib_LTIBRARIES;
Paul Eggert fixed that on June 7,
commit 1d35638b23e95fe6f41c828a3442f6d7f242f4c4, and the
test started failing as a result.

* t/instdir-ltlib.sh: add some doc.
* t/all.sh: likewise. (Another xfail test, which is mysterious to me.)

diff --git a/t/all.sh b/t/all.sh
index 526abebd82531cd2bf6bb04361f688c906c3c728..faae8ada21d6b58cf3f584a06865d5994208533f 100644 (file)
--- a/t/all.sh
+++ b/t/all.sh
@@ -27,6 +27,10 @@ $AUTOMAKE
 
 for target in $targets; do
   grep "${target}-local" Makefile.in # For debugging.
+  
+  # This grep fails, thus the test is listed in XFAIL.
+  # We're checking that (e.g.) all-am does not depend on all-local,
+  # but why?
   grep "${target}-am:.*${target}-local" Makefile.in
 done
 

diff --git a/t/instdir-ltlib.sh b/t/instdir-ltlib.sh
index 7744fd6e6f421cd357d01323656e2d54398d4533..23fe9c12dfd094e76e8a97df1bf93a414f415f43 100644 (file)
--- a/t/instdir-ltlib.sh
+++ b/t/instdir-ltlib.sh
@@ -15,12 +15,20 @@
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
 # If $(libdir) or $(pyexecdir) is the empty string, then nothing should
-# be installed there.
-# This test exercises the libtool code paths.
+# be installed there, and in fact libtool will refuse to link due to the
+# missing argument for -rpath:
+#   /bin/sh ./libtool [...] -rpath  libfoo.lo  
+#   libtool:   error: only absolute run-paths are allowed
+# (Hopefully the error message will be improved.)
+# Thus this test is expected to fail.
+# 
+# This test exercises some of the libtool code paths.
 
 required='cc libtoolize'
 . test-init.sh
 
+# Although LT_INIT is preferred nowadays, keep using AC_PROG_LIBTOOL
+# to help make sure it's still supported.
 cat >>configure.ac <<'END'
 AC_PROG_CC
 AM_PROG_AR

diff --git a/t/list-of-tests.mk b/t/list-of-tests.mk
index 9f6a0868510c9a372ed02bcc3bb734e33dc6ffd9..3761d4ad3585a057c21a589d38a2047e18bf1fe2 100644 (file)
--- a/t/list-of-tests.mk
+++ b/t/list-of-tests.mk
@@ -36,6 +36,7 @@ t/override-conditional-2.sh \
 t/override-conditional-pr13940.sh \
 t/dist-pr109765.sh \
 t/instdir-cond2.sh \
+t/instdir-ltlib.sh \
 t/java-nobase.sh \
 t/objext-pr10128.sh \
 t/remake-timing-bug-pr8365.sh \
@@ -545,7 +546,6 @@ t/instdir-cond2.sh \
 t/instdir-no-empty.sh \
 t/instdir-java.sh \
 t/instdir-lisp.sh \
-t/instdir-ltlib.sh \
 t/instdir-prog.sh \
 t/instdir-python.sh \
 t/instdir-texi.sh \