Bug 528841 - Review Request: sks - Synchronizing Key Server
Summary: Review Request: sks - Synchronizing Key Server
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Toshio Ernie Kuratomi
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-10-13 23:52 UTC by Nick Bebout
Modified: 2009-10-23 02:47 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-10-23 02:47:35 UTC
Type: ---
Embargoed:
a.badger: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Nick Bebout 2009-10-13 23:52:53 UTC
Spec URL: http://nb.fedorapeople.org/review/sks.spec
SRPM URL: http://nb.fedorapeople.org/review/sks-1.1.1-1.fc11.src.rpm
Koji Scratch Build: https://koji.fedoraproject.org/koji/taskinfo?taskID=1745490

Description:
SKS is a OpenPGP keyserver whose goal is to provide easy to
deploy, decentralized, and highly reliable synchronization.

Comment 1 Todd Zullinger 2009-10-14 03:37:03 UTC
> %__make install PREFIX="%{buildroot}/usr" MANDIR="%{buildroot}/usr/share/man"

This line should use macros for /usr and /usr/share/man, e.g.:

%__make install PREFIX="%{buildroot}%{_bindir}" MANDIR="%{buildroot}%{_mandir}"

> %files
> %defattr(-,root,root)
> #%doc AUTHORS LICENSE LICENSE.OpenSSL README znc.conf

This commented line ...

> %doc BUGS CHANGELOG COPYING FILES README TODO VERSION
> %{_bindir}/sks
> %{_bindir}/sks_build.sh
> %{_bindir}/sks_add_mail
> %{_mandir}/man8/sks.8.gz
> #%{_mandir}/man1/*
> #%{_libdir}/znc/
> #%{_datadir}/znc/

And these three commented lines should be removed.

Comment 2 Nick Bebout 2009-10-14 15:46:27 UTC
Fixed. Thanks!

Comment 3 Toshio Ernie Kuratomi 2009-10-14 17:37:40 UTC
Using the new srpm here: http://fedorapeople.org/~nb/review/sks-1.1.1-2.fc11.src.rpm

Good:                                                         
* Package named appropriately.  Upstream tarball and binary are sks.
* Licensed GPLv2+, sources confirm
* license file included in package
* Spec file is legible
* Source matches upstream
* No locale files
* Not a library package
* No bundled libraries
* Not relocatable
* All created directories are owned and no others.
* Proper %clean
* macros used consistently
* Code, not content
* No %doc files affect the package runtime
* Not a GUI application
* All filenames are utf-8
* Package builds forall arches in koji
* rpmlint clean
* C libraries are dynamically linked
* Files have good permissions

APPROVED

Comment 4 Nick Bebout 2009-10-15 05:13:15 UTC
New Package CVS Request
=======================
Package Name: sks
Short Description: Synchronizing Key Server 
Owners: nb
Branches: F-10 F-11 F-12 EL-4 EL-5
InitialCC:

Comment 5 Kevin Fenzi 2009-10-15 17:15:40 UTC
cvs done.


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