Bug 549709 - Review Request: libgnome-keyring - Framework for managing passwords and other secrets
Summary: Review Request: libgnome-keyring - Framework for managing passwords and other...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
high
Target Milestone: ---
Assignee: Matthias Clasen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-12-22 13:01 UTC by Tomáš Bžatek
Modified: 2015-03-03 22:42 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-01-07 10:25:53 UTC
Type: ---
Embargoed:
mclasen: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Tomáš Bžatek 2009-12-22 13:01:57 UTC
Spec URL: http://tbzatek.fedorapeople.org/libgnome-keyring/libgnome-keyring.spec
SRPM URL: http://tbzatek.fedorapeople.org/libgnome-keyring/libgnome-keyring-2.29.4-1.fc13.src.rpm

Description: 
gnome-keyring is a program that keep password and other secrets for
users. The library libgnome-keyring is used by applications to integrate
with the gnome-keyring system.

--
Basically, this is a split of gnome-keyring (recent upstream decision), creating separate library with the core.

Feel free to propose better description, I'm bad with that.

rpmlint passes without warning, package builds fine in koji.

Comment 1 Tomáš Bžatek 2009-12-22 14:51:28 UTC
Forgot to add that we will have to add libgnome-keyring dependency to all g-k clients and fix tons of packages in Fedora. This will effectively make decent mess once libgnome-keyring and new gnome-keyring packages will hit rawhide.

Comment 2 Bastien Nocera 2009-12-22 14:55:14 UTC
(In reply to comment #1)
> Forgot to add that we will have to add libgnome-keyring dependency to all g-k
> clients and fix tons of packages in Fedora. This will effectively make decent
> mess once libgnome-keyring and new gnome-keyring packages will hit rawhide.  

Not really. Make the library depend on gnome-keyring, voila. You'd probably have a problem with bootstrapping gnome-keyring from scratch if it depends on libgnome-keyring, but that's relatively minor.

Comment 3 Matthias Clasen 2009-12-22 15:02:10 UTC
You still need a libgnome-keyring-devel BR everywhere...

Comment 4 Tomáš Bžatek 2009-12-22 15:11:19 UTC
Right, we can add explicit Require: libgnome-keyring-devel to the gnome-keyring-devel, that might do the trick for the moment.

FYI, we have bootstrapping problems in Fedora anyway, circular BR dependency nautilus <--> gnome-disk-utility (pulled in by gvfs, which nautilus depends on), but that's another story.

Comment 5 Matthias Clasen 2009-12-22 22:31:52 UTC
Package builds fine in mock, and rpmlint is silent:

 rpmlint /var/lib/mock/fedora-rawhide-x86_64/result/libgnome-keyring-*.rpm
4 packages and 0 specfiles checked; 0 errors, 0 warnings.

Comment 6 Matthias Clasen 2009-12-22 23:06:31 UTC
package name: ok
spec name: ok
packaging guidelines: ok
license: ok
license field: ok
license file: ok
spec language: ok
spec readable: yes
upstream source: ok
buildable: yes
ExcludeArch: none, ok
BuildRequires: ok
locale handling: none, ok
ldconfig: ok
system libraries: none, ok
relocatable: no, ok
directory ownership: ok
duplicate files: ok
file permissions: ok
%clean: ok
macro use: ok
permissable content: ok
large docs: none, ok
%doc content: ok
header files: ok
static libs: none, ok
pc files: ok
shared libs: ok
devel deps: ok
libtool archives: none, ok
gui apps: no, ok
file ownership: ok, but we should figure out how to handle the conflict with the old gnome-keyring package
%install: ok
utf8 filenames: ok


Summary: all looks good, we just need to figure out the conflict with the old gnome-keyring package. Probably add something like

Conflicts: gnome-keyring < 2.29.4
Conflicts: gnome-keyring-devel < 2.29.4

Comment 7 Tomáš Bžatek 2010-01-04 16:05:38 UTC
Thanks for the review. I've added Conflicts: in libgnome-keyring.spec and explicit Requires: libgnome-keyring in gnome-keyring.spec for both gnome-keyring and gnome-keyring-devel subpackages (as discussed above). This should guarantee good upgrade path without breaking the things.

New Spec:
http://tbzatek.fedorapeople.org/libgnome-keyring/libgnome-keyring.spec
New SRPM:
http://tbzatek.fedorapeople.org/libgnome-keyring/libgnome-keyring-2.29.4-1.fc13.src.rpm

Comment 8 Matthias Clasen 2010-01-04 18:29:11 UTC
Looks good. Approved.

Comment 9 Tomáš Bžatek 2010-01-05 11:04:24 UTC
New Package CVS Request
=======================
Package Name: libgnome-keyring
Short Description: Framework for managing passwords and other secrets
Owners: tbzatek
Branches: devel
InitialCC:

Comment 10 Kevin Fenzi 2010-01-06 21:30:26 UTC
cvs done.

Comment 11 Tomáš Bžatek 2010-01-07 10:25:53 UTC
(In reply to comment #10)
> cvs done.  
Thanks!

libgnome-keyring-2.29.4-1.fc13 has been built: http://koji.fedoraproject.org/koji/buildinfo?buildID=149812


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