After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 694678 - Seahorse should depend on GnuPG for testing and documentation
Seahorse should depend on GnuPG for testing and documentation
Status: RESOLVED FIXED
Product: seahorse
Classification: Applications
Component: general
3.7.x
Other Linux
: High critical
: ---
Assigned To: Seahorse Maintainer
Seahorse Maintainer
Depends on:
Blocks:
 
 
Reported: 2013-02-25 15:00 UTC by Jim Campbell
Modified: 2013-03-04 19:55 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
seahorse preferences menu (4.25 KB, image/png)
2013-02-25 15:00 UTC, Jim Campbell
  Details
3.8: seahorse should be built with gnupg and gpgme (1.80 KB, patch)
2013-02-26 09:12 UTC, Stef Walter
committed Details | Review
prefs: Don't show preferences when no preferences are available (9.87 KB, patch)
2013-02-26 09:16 UTC, Stef Walter
committed Details | Review

Description Jim Campbell 2013-02-25 15:00:44 UTC
Created attachment 237358 [details]
seahorse preferences menu

When I select Edit > Preferences in a jhbuild from 2012-02-24, I only get options for Help or Close.

The other preference options should be available.
Comment 1 Stef Walter 2013-02-26 09:12:52 UTC
Created attachment 237414 [details] [review]
3.8: seahorse should be built with gnupg and gpgme

This is a patch against jhbuild.

GnuPG is an optional dependency of seahorse only because some downstreams
want to use seahorse without GnuPG. In GNOME we should always build it
with GnuPG, test and document it that way.
Comment 2 Stef Walter 2013-02-26 09:14:40 UTC
Colin, could you do me a favor and look over the attached jhbuild patch before I push it to jhbuild? This is the first time I messed with sysdeps, so want to make sure I get this right. This is also a heads up that we should probably include gnupg and gpgme as deps of seahorse in gnome-ostree.
Comment 3 Stef Walter 2013-02-26 09:16:54 UTC
Created attachment 237415 [details] [review]
prefs: Don't show preferences when no preferences are available

This is a patch against seahorse.

If seahorse is built without GnuPG support (not recommended, but
possible) then there are currently no preferences available.

Hide the Preferences menu item in these cases, so it doesn't show
a confusing empty window.

In addition fix some assertions and cleanup some unused code
related to this.
Comment 4 Stef Walter 2013-03-04 19:50:19 UTC
Comment on attachment 237414 [details] [review]
3.8: seahorse should be built with gnupg and gpgme

Pushed to jhbuild master.
Comment 5 Stef Walter 2013-03-04 19:55:34 UTC
Attachment 237415 [details] pushed as 8ca9513 - prefs: Don't show preferences when no preferences are available