fyi, scratch build kicked off:
http://koji.fedoraproject.org/koji/taskinfo?taskID=251238

resulting x86_64 pkgs:
$ $rpmlint *.rpm
libzip.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/zipcmp ['/usr/lib64']
libzip.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/zipmerge 
['/usr/lib64']
libzip-devel.x86_64: W: no-documentation

crap.

Well, before tackling that:
1.  MUST: -devel needs
Requires: pkgconfig

I'll review this.

The man pages that correspond with API description should better
be in -devel. In main package:
%{_mandir}/man1/*zip*
in -devel:
%{_mandir}/man3/*zip*

I think it is better to keep timestamps, this should be achieved with
make install DESTDIR=$RPM_BUILD_ROOT INSTALL='install -p'

I think it is a bit unfortunate to have the header file
named simply zip.h. Maybe it could be in a libzip subdir? This maybe 
could be achieved by passing
--includedir=%{_includedir}/libzip

the license looks like a BSD one (3 clauses) to me.

> --includedir=%{_includedir}/libzip

let's not go inventing problems that don't exist (yet).  When/if it ever 
becomes an issue, we can address it then.

Otherwise, I concur with Patrice's comments wrt manpages.  MUST item 2.

I tried patching in %setup
# Avoid lib64 rpaths
%if "%{_libdir}" != "/usr/lib"
sed -i -e 's|"/lib /usr/lib|"/%{_lib} %{_libdir}|' configure
%endif
(a common rpath-munching trick), and it seems to work, see
http://koji.fedoraproject.org/koji/taskinfo?taskID=251282
call this MUST item 3.

So, otherwise, the rest is simple and looks good, address items 1,2,3, and 
I'll APPROVE this.

(almost forgot), consider also the suggesion:
make install DESTDIR=$RPM_BUILD_ROOT INSTALL='install -p'
as MUST item 4.

Thanks. I've addressed all MUST items.

Spec URL: http://svahl.fedorapeople.org/libzip/libzip.spec
SRPM URL: http://svahl.fedorapeople.org/libzip/libzip-0.8-3.fc9.src.rpm

Changelog:
- require pkgconfig in devel subpkg
- move api description to devel subpkg
- keep timestamps in %%install
- avoid lib64 rpaths

Pardon, but I consider this not to be acceptable:

# Avoid lib64 rpaths
%if "%{_libdir}" != "/usr/lib"
sed -i -e 's|"/lib /usr/lib|"/%{_lib} %{_libdir}|' configure
%endif

This is kills configure scripts.

Ralf, It works.  If you don't like it, offer a better solution, please.

Otherwise, looks good to me Sebastian, APPROVED.

Thanks, Rex.

New Package CVS Request
=======================
Package Name: libzip
Short Description: C library for reading, creating, and modifying zip archives
Owners: svahl
Branches: F-7 F-8
InitialCC: 
Cvsextras Commits: no

cvs done.

(In reply to comment #8)
> Ralf, It works.
It only looks as if. What you recommend is worst hack ever having made in
Fedora-  Next time you're going to launch a hexeditor or the like?

>  If you don't like it, offer a better solution, please.
Run autoreconf
cut diff
and patch.

> Otherwise, looks good to me Sebastian, APPROVED.
A sad day for Fedora ...

Ralf, you must understand, we're under tremendous pressure to get kde4 into
fedora rawhide asap, so I hope you can understand our not wanting to block on a
cosmetic (and currently) nonproblem.

Although I don't disagree with hacks working around upstream issues
I think that we should never ever sacrifice quality for deadlines.

If the reason for using this hack is deadlines, this is a very bad
idea, in my opinion. It adds the possibility for 'blitz' reviews of 
dubious quality and I have seen too much reviews like that, we should 
really be especially watchful and not let anything go that way. For a 
recent example, you can see how the ltsp related reviews begun, 
we should not go down that road. And for merge reviews there is also
a lot of similar issues (although some come more from ignorance than
something else).

Here it is much less extreme since the review was quite correct,
however if the hack is there because of the time it is wrong.

In rawhide now, closing.