Attachment 610835 Details for Bug 855454 – Patch to reenable unit tests in gpgme

[patch] Patch to reenable unit tests in gpgme

gpgme-reenable-unit-tests.patch (text/plain), 5.73 KB, created by John Morris on 2012-09-07 20:03:35 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: John Morris

Created: 2012-09-07 20:03:35 UTC

Size: 5.73 KB

patch

obsolete

>commit 58d04c3292b09f72baef4a4ef6c8824e5d9eef53
>Author: John Morris <john@zultron.com>
>Date:   Fri Sep 7 14:44:39 2012 -0500
>
>    Re-enable gpgme 'gpg' unit tests, dependent on #855451
>    
>    Unit tests in the tests/gpg directory would fail except that they have
>    been disabled.  In order to disable them, libgpg-error needs to
>    re-enable building static libraries; see bug #855451.
>    
>    This patch, dependent on the above bugfix, re-enables unit testing in
>    the gpg directory:
>    
>    - specfile:  remove --disable on  %%configure line
>    - specfile:  add BuildRequires: glibc-static for tests/gpg/t-thread1
>    - config_extras patch:  fix hunks that disable
>      gpg_error_libs and libs_pthread config
>    - ImplicitDSOLinking.patch:  add test/gpg/Makefile.in
>    
>    Also:
>    - quiet configure warning 'could not find g13'
>    
>    Also .gitignore changes:
>    - ignore 'fedpkg local' build artifacts
>    - ignore emacs cruft
>    - generalize source file globs
>
>diff --git a/.gitignore b/.gitignore
>index d76f663..789221d 100644
>--- a/.gitignore
>+++ b/.gitignore
>@@ -1,4 +1,9 @@
>-gpgme-1.2.0.tar.bz2
>-gpgme-1.2.0.tar.bz2.sig
>-/gpgme-1.3.0.tar.bz2.sig
>-/gpgme-1.3.0.tar.bz2
>+# ignore sources
>+/gpgme-[0-9].[0-9].[0-9].tar.bz2
>+/gpgme-[0-9].[0-9].[0-9].tar.bz2.sig
>+# ignore fedpkg local build artifacts
>+/.build-*.log
>+/gpgme-[0-9].[0-9].[0-9]/
>+# ignore emacs cruft
>+/.#*
>+/#*
>diff --git a/gpgme-1.3.0-ImplicitDSOLinking.patch b/gpgme-1.3.0-ImplicitDSOLinking.patch
>index cd2f7a5..17b6d38 100644
>--- a/gpgme-1.3.0-ImplicitDSOLinking.patch
>+++ b/gpgme-1.3.0-ImplicitDSOLinking.patch
>@@ -22,18 +22,20 @@ diff -up gpgme-1.3.0/src/Makefile.in.ImplicitDSOLinking gpgme-1.3.0/src/Makefile
>  all: $(BUILT_SOURCES)
>  	$(MAKE) $(AM_MAKEFLAGS) all-am
>  
>-diff -up gpgme-1.3.0/tests/gpgsm/Makefile.am.ImplicitDSOLinking gpgme-1.3.0/tests/gpgsm/Makefile.am
>---- gpgme-1.3.0/tests/gpgsm/Makefile.am.ImplicitDSOLinking	2008-12-03 14:39:48.000000000 +0100
>-+++ gpgme-1.3.0/tests/gpgsm/Makefile.am	2011-02-11 17:36:32.000000000 +0100
>-@@ -32,7 +32,7 @@ EXTRA_DIST = cert_dfn_pca01.der cert_dfn
>- INCLUDES = -I$(top_builddir)/src
>+diff -up gpgme-1.3.0/tests/gpg/Makefile.in.ImplicitDSOLinking gpgme-1.3.0/tests/gpg/Makefile.in
>+--- gpgme-1.3.0/tests/gpg/Makefile.in.ImplicitDSOLinking	2010-01-11 05:06:54.000000000 -0600
>++++ gpgme-1.3.0/tests/gpg/Makefile.in	2012-09-07 13:57:08.485271081 -0500
>+@@ -377,8 +377,8 @@ EXTRA_DIST = mkdemodirs.in pubdemo.asc s
>  
>+ INCLUDES = -I$(top_builddir)/src
>  AM_CPPFLAGS = @GPG_ERROR_CFLAGS@
> -LDADD = ../../src/libgpgme.la
>+-t_thread1_LDADD = ../../src/libgpgme-pthread.la
> +LDADD = ../../src/libgpgme.la @GPG_ERROR_LIBS@
>++t_thread1_LDADD = $(LDADD) ../../src/libgpgme-pthread.la -lpthread
>+ all: all-am
>  
>- # We don't run t-genkey in the test suite, because it takes too long
>- # and needs a working pinentry.
>+ .SUFFIXES:
> diff -up gpgme-1.3.0/tests/gpgsm/Makefile.in.ImplicitDSOLinking gpgme-1.3.0/tests/gpgsm/Makefile.in
> --- gpgme-1.3.0/tests/gpgsm/Makefile.in.ImplicitDSOLinking	2010-01-11 12:06:54.000000000 +0100
> +++ gpgme-1.3.0/tests/gpgsm/Makefile.in	2011-02-11 17:36:32.000000000 +0100
>diff --git a/gpgme-1.3.0-config_extras.patch b/gpgme-1.3.0-config_extras.patch
>index 7b9a643..9e1972a 100644
>--- a/gpgme-1.3.0-config_extras.patch
>+++ b/gpgme-1.3.0-config_extras.patch
>@@ -1,7 +1,7 @@
> diff -up gpgme-1.3.0/src/gpgme-config.in.config_extras gpgme-1.3.0/src/gpgme-config.in
> --- gpgme-1.3.0/src/gpgme-config.in.config_extras	2009-10-23 08:44:56.000000000 -0500
> +++ gpgme-1.3.0/src/gpgme-config.in	2011-03-17 13:15:12.511565537 -0500
>-@@ -20,26 +20,26 @@ LC_ALL=C
>+@@ -20,8 +20,8 @@ LC_ALL=C
>  LANG=C
>  
>  # Network libraries.
>@@ -12,10 +12,7 @@ diff -up gpgme-1.3.0/src/gpgme-config.in.config_extras gpgme-1.3.0/src/gpgme-con
>  
>  # Configure libgpg-error.
>  gpg_error_cflags="@GPG_ERROR_CFLAGS@"
>--gpg_error_libs="@GPG_ERROR_LIBS@"
>-+#gpg_error_libs="@GPG_ERROR_LIBS@"
>- 
>- # Configure thread packages.
>+@@ -31,7 +31,7 @@
>  thread_modules=""
>  
>  @HAVE_PTH_TRUE@thread_modules="$thread_modules pth"
>@@ -24,8 +21,7 @@ diff -up gpgme-1.3.0/src/gpgme-config.in.config_extras gpgme-1.3.0/src/gpgme-con
>  cflags_pth="@PTH_CFLAGS@"
>  
>  @HAVE_PTHREAD_TRUE@thread_modules="$thread_modules pthread"
>--libs_pthread="-lpthread"
>-+#libs_pthread="-lpthread"
>+@@ -39,7 +39,7 @@
>  cflags_pthread=""
>  
>  # Configure glib.
>diff --git a/gpgme.spec b/gpgme.spec
>index 149b94f..135dfa4 100644
>--- a/gpgme.spec
>+++ b/gpgme.spec
>@@ -2,7 +2,7 @@
> Name:    gpgme
> Summary: GnuPG Made Easy - high level crypto API
> Version: 1.3.0
>-Release: 4%{?dist}
>+Release: 5%{?dist}
> 
> License: LGPLv2+
> Group:   Applications/System
>@@ -25,10 +25,9 @@ BuildRequires: gnupg2-smime
> BuildRequires: libgpg-error-devel
> BuildRequires: pth-devel
> BuildRequires: libassuan2-devel
>+BuildRequires: glibc-static
> 
>-# --disable-gpg-test required since 'make check' currently includes some
>-# gpg(1)-specific tests
>-%define _with_gpg --with-gpg=%{_bindir}/gpg2 --disable-gpg-test
>+%define _with_gpg --with-gpg=%{_bindir}/gpg2
> Requires: gnupg2
> 
> %description
>@@ -68,6 +67,7 @@ sed -i -e 's|^libdir=@libdir@$|libdir=@exec_prefix@/lib|g' src/gpgme-config.in
> %build
> %configure \
>   --disable-static \
>+  --without-g13 \
>   %{?_with_gpg}
> 
> make %{?_smp_mflags}
>@@ -124,6 +124,15 @@ fi
> 
> 
> %changelog
>+* Fri Sep  7 2012 John Morris <john@zultron.com> - 1.3.0-5
>+- re-enable gpg tests:
>+-   specfile:  remove --disable on  %%configure line
>+-   specfile:  add BuildRequires: glibc-static for tests/gpg/t-thread1
>+-   config_extras patch:  fix hunks that disable
>+    gpg_error_libs and libs_pthread config
>+-   ImplicitDSOLinking.patch:  add test/gpg/Makefile.in
>+- quiet configure warning 'could not find g13'
>+
> * Thu Mar 17 2011 Rex Dieter <rdieter@fedoraproject.org> - 1.3.0-4
> - gpgme-config: remove libassuan-related flags as threatened (#676954) 
> \

Actions: View | Diff

Attachments on bug 855454: 610835 | 610841 | 617397