bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH 2/2] tests: drop unused link dependency


From: Eric Blake
Subject: [PATCH 2/2] tests: drop unused link dependency
Date: Tue, 26 Apr 2011 22:22:11 -0600

Ever since commit 2392b18099, openat and fdopendir no longer drag in
xalloc-die through indirect dependencies, so these tests no longer
need $(LIBINTL).

* modules/areadlinkat-tests (Makefile.am): Drop stale LDADD.
* modules/dirent-safer-tests (Makefile.am): Likewise.
* modules/fdopendir-tests (Makefile.am): Likewise.
* modules/hash-tests (Makefile.am): Likewise.
* modules/linkat-tests (Makefile.am): Likewise.
* modules/mkfifoat-tests (Makefile.am): Likewise.
* modules/openat-safer-tests (Makefile.am): Likewise.
* modules/openat-tests (Makefile.am): Likewise.
* modules/readlinkat-tests (Makefile.am): Likewise.
* modules/symlinkat-tests (Makefile.am): Likewise.
* modules/fdutimensat-tests (test_fdutimensat_LDADD): Drop unused
LIBINTL.
* modules/utimensat-tests (test_utimensat_LDADD): Likewise.
* tests/test-hash.c: Drop unused include.

Signed-off-by: Eric Blake <address@hidden>
---

fdutimensat-tests still pulls in xalloc, via utimecmp (which
calls xalloc_die if it can't manage a hash table).  So this
patch isn't quite perfect yet, but getting closer.  Any ideas
on whether it would be possible to make utimecmp work without
xalloc?

 ChangeLog                  |   16 ++++++++++++++++
 modules/areadlinkat-tests  |    1 -
 modules/dirent-safer-tests |    3 ---
 modules/fdopendir-tests    |    1 -
 modules/fdutimensat-tests  |    2 +-
 modules/hash-tests         |    2 --
 modules/linkat-tests       |    1 -
 modules/mkfifoat-tests     |    1 -
 modules/openat-safer-tests |    1 -
 modules/openat-tests       |    5 -----
 modules/readlinkat-tests   |    1 -
 modules/symlinkat-tests    |    1 -
 modules/utimensat-tests    |    2 +-
 tests/test-hash.c          |    1 -
 14 files changed, 18 insertions(+), 20 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index a610358..1b9c522 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,21 @@
 2011-04-26  Eric Blake  <address@hidden>

+       tests: drop unused link dependency
+       * modules/areadlinkat-tests (Makefile.am): Drop stale LDADD.
+       * modules/dirent-safer-tests (Makefile.am): Likewise.
+       * modules/fdopendir-tests (Makefile.am): Likewise.
+       * modules/hash-tests (Makefile.am): Likewise.
+       * modules/linkat-tests (Makefile.am): Likewise.
+       * modules/mkfifoat-tests (Makefile.am): Likewise.
+       * modules/openat-safer-tests (Makefile.am): Likewise.
+       * modules/openat-tests (Makefile.am): Likewise.
+       * modules/readlinkat-tests (Makefile.am): Likewise.
+       * modules/symlinkat-tests (Makefile.am): Likewise.
+       * modules/fdutimensat-tests (test_fdutimensat_LDADD): Drop unused
+       LIBINTL.
+       * modules/utimensat-tests (test_utimensat_LDADD): Likewise.
+       * tests/test-hash.c: Drop unused include.
+
        hash, mgetgroups: drop xalloc dependency
        * lib/hash.c (hash_oversized): Copy from xalloc.h.
        * lib/mgetgroups.c (mgetgroups_oversized): Likewise.
diff --git a/modules/areadlinkat-tests b/modules/areadlinkat-tests
index 422ac61..23e5278 100644
--- a/modules/areadlinkat-tests
+++ b/modules/areadlinkat-tests
@@ -13,4 +13,3 @@ configure.ac:
 Makefile.am:
 TESTS += test-areadlinkat
 check_PROGRAMS += test-areadlinkat
-test_areadlinkat_LDADD = $(LDADD) @LIBINTL@
diff --git a/modules/dirent-safer-tests b/modules/dirent-safer-tests
index 172c662..2bc9593 100644
--- a/modules/dirent-safer-tests
+++ b/modules/dirent-safer-tests
@@ -10,6 +10,3 @@ configure.ac:
 Makefile.am:
 TESTS += test-dirent-safer
 check_PROGRAMS += test-dirent-safer
-# Link with libintl when needed. dirent-safer uses fdopendir if it is present,
-# and fdopendir indirectly depends on xgetcwd -> xalloc-die -> gettext-h.
-test_dirent_safer_LDADD = $(LDADD) $(LIBINTL)
diff --git a/modules/fdopendir-tests b/modules/fdopendir-tests
index bee2df7..39575ba 100644
--- a/modules/fdopendir-tests
+++ b/modules/fdopendir-tests
@@ -12,4 +12,3 @@ configure.ac:
 Makefile.am:
 TESTS += test-fdopendir
 check_PROGRAMS += test-fdopendir
-test_fdopendir_LDADD = $(LDADD) @LIBINTL@
diff --git a/modules/fdutimensat-tests b/modules/fdutimensat-tests
index 48ea78d..1d107ce 100644
--- a/modules/fdutimensat-tests
+++ b/modules/fdutimensat-tests
@@ -19,4 +19,4 @@ configure.ac:
 Makefile.am:
 TESTS += test-fdutimensat
 check_PROGRAMS += test-fdutimensat
-test_fdutimensat_LDADD = $(LDADD) $(LIB_CLOCK_GETTIME) @LIBINTL@
+test_fdutimensat_LDADD = $(LDADD) $(LIB_CLOCK_GETTIME)
diff --git a/modules/hash-tests b/modules/hash-tests
index a77bfe7..b3f814b 100644
--- a/modules/hash-tests
+++ b/modules/hash-tests
@@ -7,11 +7,9 @@ hash-pjw
 inttostr
 progname
 stdbool
-xalloc

 configure.ac:

 Makefile.am:
 TESTS += test-hash
 check_PROGRAMS += test-hash
-test_hash_LDADD = $(LDADD) @LIBINTL@
diff --git a/modules/linkat-tests b/modules/linkat-tests
index c49c1b3..91e70c0 100644
--- a/modules/linkat-tests
+++ b/modules/linkat-tests
@@ -17,4 +17,3 @@ configure.ac:
 Makefile.am:
 TESTS += test-linkat
 check_PROGRAMS += test-linkat
-test_linkat_LDADD = $(LDADD) @LIBINTL@
diff --git a/modules/mkfifoat-tests b/modules/mkfifoat-tests
index 8be33df..a1e33c9 100644
--- a/modules/mkfifoat-tests
+++ b/modules/mkfifoat-tests
@@ -13,4 +13,3 @@ configure.ac:
 Makefile.am:
 TESTS += test-mkfifoat
 check_PROGRAMS += test-mkfifoat
-test_mkfifoat_LDADD = $(LDADD) @LIBINTL@
diff --git a/modules/openat-safer-tests b/modules/openat-safer-tests
index 1f0b158..a33d2c6 100644
--- a/modules/openat-safer-tests
+++ b/modules/openat-safer-tests
@@ -9,4 +9,3 @@ configure.ac:
 Makefile.am:
 TESTS += test-openat-safer
 check_PROGRAMS += test-openat-safer
-test_openat_safer_LDADD = $(LDADD) @LIBINTL@
diff --git a/modules/openat-tests b/modules/openat-tests
index 250a574..959d882 100644
--- a/modules/openat-tests
+++ b/modules/openat-tests
@@ -31,8 +31,3 @@ Makefile.am:
 TESTS += test-fchownat test-fstatat test-mkdirat test-openat test-unlinkat
 check_PROGRAMS += test-fchownat test-fstatat test-mkdirat test-openat \
   test-unlinkat
-test_fchownat_LDADD = $(LDADD) @LIBINTL@
-test_fstatat_LDADD = $(LDADD) @LIBINTL@
-test_mkdirat_LDADD = $(LDADD) @LIBINTL@
-test_openat_LDADD = $(LDADD) @LIBINTL@
-test_unlinkat_LDADD = $(LDADD) @LIBINTL@
diff --git a/modules/readlinkat-tests b/modules/readlinkat-tests
index de23df8..96faa56 100644
--- a/modules/readlinkat-tests
+++ b/modules/readlinkat-tests
@@ -13,4 +13,3 @@ configure.ac:
 Makefile.am:
 TESTS += test-readlinkat
 check_PROGRAMS += test-readlinkat
-test_readlinkat_LDADD = $(LDADD) @LIBINTL@
diff --git a/modules/symlinkat-tests b/modules/symlinkat-tests
index 9033149..13ae9bd 100644
--- a/modules/symlinkat-tests
+++ b/modules/symlinkat-tests
@@ -12,4 +12,3 @@ configure.ac:
 Makefile.am:
 TESTS += test-symlinkat
 check_PROGRAMS += test-symlinkat
-test_symlinkat_LDADD = $(LDADD) @LIBINTL@
diff --git a/modules/utimensat-tests b/modules/utimensat-tests
index 26dbac6..987a86a 100644
--- a/modules/utimensat-tests
+++ b/modules/utimensat-tests
@@ -19,4 +19,4 @@ configure.ac:
 Makefile.am:
 TESTS += test-utimensat
 check_PROGRAMS += test-utimensat
-test_utimensat_LDADD = $(LDADD) $(LIB_CLOCK_GETTIME) @LIBINTL@
+test_utimensat_LDADD = $(LDADD) $(LIB_CLOCK_GETTIME)
diff --git a/tests/test-hash.c b/tests/test-hash.c
index 108daef..d07682b 100644
--- a/tests/test-hash.c
+++ b/tests/test-hash.c
@@ -20,7 +20,6 @@
 #include "hash.h"
 #include "hash-pjw.h"
 #include "inttostr.h"
-#include "xalloc.h"

 #include <stdio.h>
 #include <stdlib.h>
-- 
1.7.4.4




reply via email to

[Prev in Thread] Current Thread [Next in Thread]