Red Hat Bugzilla – Bug 855454
gpgme basic unit tests would fail, except they are disabled
Last modified: 2012-11-02 09:11:07 EDT
Created attachment 610835 [details]
Patch to reenable unit tests in gpgme
In FC16, at least, one of the gpgme unit tests in the 'tests/gpg' directory would fail (t-encrypt-sign test), but the unit tests are disabled.
The specfile contains a note that the tests are disabled because they depend on GPG v.1. However, the tests do run in FC16, and all pass but the one, if libgpg-error-devel is rebuilt to include static libs, and the attached patch is applied.
Here is the documentation from the attached patch:
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
- quiet configure warning 'could not find g13'
Also .gitignore changes:
- ignore 'fedpkg local' build artifacts
- ignore emacs cruft
- generalize source file globs
Created attachment 610841 [details]
Fix for the failing unit test
Attaching a second patch. This patch fixes the failing unit test mentioned in the original bug report. It is self-documenting; please check the patch for more info.
I believe this should fix up the package, if the needed changes to libgpg-error are made.
Created attachment 617397 [details]
Patch against master: re-enable unit tests; fix failing test; link tests against shared libs
This patch obsoletes the previous two submitted. The owner of #855451 argued against shipping static libs in libgpg-error-devel and instead link against shared libs to fix this bug. The suggested fix is implemented here.
This patch is against master. It does not work against f16, but koji builds fine against f17, f18 and rawhide. I am bumping the product version up to f17, since, AFAIK, f16 is EOL.
kdepimlibs-4.9.1-4.fc18, gpgme-1.3.2-1.fc18 has been submitted as an update for Fedora 18.
Package kdepimlibs-4.9.1-4.fc18, gpgme-1.3.2-1.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing kdepimlibs-4.9.1-4.fc18 gpgme-1.3.2-1.fc18'
as soon as you are able to.
Please go to the following url:
then log in and leave karma (feedback).