Merge r335,653,676 from libarchive.googlecode.com: Instead of

conditioning tests on HAVE_ZLIB, etc, just ask libarchive for the
service and handle the failure coming back from libarchive.  This
gives us better test coverage of common client usage where clients
simply try to use libarchive services and handle the errors coming
back instead of trying to second-guess which libarchive services are
compiled in.

1 files changed, 11 insertions, 5 deletions

diff --git a/lib/libarchive/test/test_read_format_gtar_gz.c b/lib/libarchive/test/test_read_format_gtar_gz.c
index afced28..6d3b66d 100644
--- a/lib/libarchive/test/test_read_format_gtar_gz.c
+++ b/lib/libarchive/test/test_read_format_gtar_gz.c
@@ -34,25 +34,31 @@ static unsigned char archive[] = {
 
 DEFINE_TEST(test_read_format_gtar_gz)
 {
-#if HAVE_ZLIB_H
 	struct archive_entry *ae;
 	struct archive *a;
+	int r;
+
 	assert((a = archive_read_new()) != NULL);
 	assert(0 == archive_read_support_compression_all(a));
 	assert(0 == archive_read_support_format_all(a));
 	assert(0 == archive_read_open_memory(a, archive, sizeof(archive)));
-	assert(0 == archive_read_next_header(a, &ae));
+	r = archive_read_next_header(a, &ae);
+	if (UnsupportedCompress(r, a)) {
+		skipping("Skipping GZIP compression check: "
+		    "This version of libarchive was compiled "
+		    "without gzip support");
+		goto finish;
+	}
+	assert(0 == r);
 	assert(archive_compression(a) == ARCHIVE_COMPRESSION_GZIP);
 	assert(archive_format(a) == ARCHIVE_FORMAT_TAR_GNUTAR);
 	assert(0 == archive_read_close(a));
+finish:
 #if ARCHIVE_VERSION_NUMBER < 2000000
 	archive_read_finish(a);
 #else
 	assert(0 == archive_read_finish(a));
 #endif
-#else
-	skipping("Need zlib");
-#endif
 }