Bug 458140 - seahorse-plugins - Plugins and utilities for encryption in GNOME
seahorse-plugins - Plugins and utilities for encryption in GNOME
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2008-08-06 13:01 EDT by Matthias Clasen
Modified: 2008-08-07 22:26 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-08-07 22:26:07 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
walters: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Matthias Clasen 2008-08-06 13:01:18 EDT
Spec: http://people.redhat.com/mclasen/seahorse-plugins.spec
SRPM: http://people.redhat.com/mclasen/seahorse-plugins-2.23.6-1.fc10.src.rpm

Description:
The plugins and utilities in this package integrate seahorse into
the GNOME desktop environment and allow users to perform operations
from applications like nautilus or gedit.
Comment 1 Colin Walters 2008-08-06 14:25:48 EDT
Doesn't build for me; seems to install an epiphany extension that's not in %files?

error: Installed (but unpackaged) file(s) found:
   /usr/lib64/epiphany/2.22/extensions/libseahorseextension.so
   /usr/lib64/epiphany/2.22/extensions/seahorse.ephy-extension

[+] source files match upstream
[+] package meets naming and versioning guidelines
[+] specfile is properly named, is cleanly written and uses macros consistently
[+] dist tag is present.
[+] build root is correct.
 (%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) is the
recommended value, but not the only one)
[+] license field matches the actual license.
[+] license is open source-compatible.
[+] license text included in package.
[+] latest version is being packaged
[+] BuildRequires are proper
[+] %clean is present.
[] package builds in koji
[] package installs properly
[] rpmlint is silent.
[] owns the directories it creates.
[] doesn't own any directories it shouldn't.
[] no duplicates in %files.
[] file permissions are appropriate.
[] code, not content.
[+] No doc subpackage needed
[+] %doc exists
Comment 2 Colin Walters 2008-08-06 14:43:32 EDT
Hm, trying to build in koji I get this:

checking which gecko to use... 
./configure: line 22985: test: : integer expression expected
disabling epiphany plugin
./configure: line 22993: test: : integer expression expected
./configure: line 23001: test: : integer expression expected
./configure: line 23009: test: : integer expression expected
./configure: line 23028: test: : integer expression expected
Comment 3 Matthias Clasen 2008-08-06 14:51:46 EDT
Oh, sorry. I meant to investigate why it wouldn't build the ephy extension on my system, but forgot.
Comment 4 Matthias Clasen 2008-08-06 20:16:34 EDT
I've updated the spec and srpm to include the epiphany extension now.
Comment 5 Colin Walters 2008-08-07 14:08:54 EDT
Ok:

[+] package builds in koji
[+] package installs properly
[-] 1 trivial rpmlint warning about mixed spaces and tabs
  Doesn't matter.
[+] owns the directories it creates.
[+] doesn't own any directories it shouldn't.
[+] no duplicates in %files.
[+] file permissions are appropriate.

Looks good.
Comment 6 Matthias Clasen 2008-08-07 15:03:31 EDT
New Package CVS Request
=======================
Package Name: seahorse-plugins
Short Description: Plugins and utilities for encryption in GNOME
Owners: mclasen
Branches:
InitialCC:
Cvsextras Commits: yes
Comment 7 Kevin Fenzi 2008-08-07 16:57:44 EDT
cvs done.

Note You need to log in before you can comment on or make changes to this bug.