Spec URL: http://fedora.danny.cz/ZipArchive.spec SRPM URL: http://fedora.danny.cz/ZipArchive-4.1.1-3.fc17.src.rpm Description: The ZipArchive library provides C++ interface to zlib.
I'd improve on the description, e.g. (adapted from http://www.artpol-software.com/ ) The ZipArchive Library can be used to add compression functionality to your software. It is written in C++ and offers the following features: * Compression, decompression and modification of zip archives. * Segmented archives support (splitting and spanning). * Zip64 format - practically no limits for sizes and the number of files in archives. * Unicode support in archives compatible with WinZip. * AES encryption - in accordance with the WinZip format. * Standard zip encryption. * BZIP2 compression algorithm - provides a better compression ratio. Since bzip2 is bundled, you need to remove the relevant sources to make sure the package uses the system version of bzip2. And you need to add BR: bzip2-devel.
The summary also needs improvement. And I think this is incorrect: install -p -m 644 %{SOURCE1} $RPM_BUILD_ROOT%{_libdir}/cmake/%{name} the installed file name should be FindZipArchive.cmake .
Jussi, 3 notes to your comments - the Zip64, AES and bzip2 features are available only in the commercial version of the library. The GPLed version provides only zlib interface - both zlib and bzip2 (in fact completely useless here) are removed in %prep - the cmake file is installed as %{_libdir}/cmake/ZipArchive/FindZipArchive.cmake, but I'll probably drop it as the pkgconfig file can be used easier in sigil/flightcrew
*** Bug 782823 has been marked as a duplicate of this bug. ***
As discussed in the sigil review I filed, see bug 772362. I'm also interested in getting sigil (and thus ZipArchive and FlightCrew) into Fedora. But I did not know that Dan had already filed a review for ZipArchive, hence I filed a review request myself in bug 782823. Luckily I was made aware of Dan's work by Dan in the sigil review so my ZipArchive work is based on top of his. Here is my version of Dan's ZipArchive package with a few small fixes: Spec URL: http://people.fedoraproject.org/~jwrdegoede/ZipArchive.spec SRPM URL: http://people.fedoraproject.org/~jwrdegoede/ZipArchive-4.1.1-4.fc15.src.rpm
I've got the entire sigil dep chain finished, as a result of that here is a new version: Spec URL: http://people.fedoraproject.org/~jwrdegoede/ZipArchive.spec SRPM URL: http://people.fedoraproject.org/~jwrdegoede/ZipArchive-4.1.1-5.fc15.src.rpm Changes: * Thu Jan 19 2012 Hans de Goede <hdegoede> - 4.1.1-5 - Drop custom cmake module, cmake using apps can use the .pc file - Fix the .pc file to properly return -lziparch for --libs Don't be fooled by the fc15 in the filenames, that is because I'm using the old makefiles from the distcvs era to build and test wip packages. The sigil packages will only work with F-16 and newer due to needing xerces-c 3.1
For the records, Hans will be the submitter and me the reviewer.
formal review is here, see the notes explaining OK* and BAD statuses below: OK source files match upstream: b81560f504917137bfa6927bb55c7eed02166fa3 ziparchive_src.zip OK package meets naming and versioning guidelines. OK* specfile is properly named, is cleanly written and uses macros consistently. OK dist tag is present. OK license field matches the actual license. OK license is open source-compatible (GPLv2+). License text included in package. OK latest version is being packaged. OK BuildRequires are proper. OK compiler flags are appropriate. OK package builds in mock (Rawhide/i386). OK debuginfo package looks complete. OK* rpmlint is silent. OK final provides and requires look sane. N/A %check is present and all tests pass. OK shared libraries are added to the regular linker search paths with correct scriptlet OK owns the directories it creates. OK doesn't own any directories it shouldn't. OK no duplicates in %files. OK file permissions are appropriate. OK correct scriptlets present. OK code, not content. OK documentation is small, so no -docs subpackage is necessary. OK %docs are not necessary for the proper functioning of the package. OK headers in devel subpackage OK pkgconfig files in devel subpackage OK no libtool .la droppings. OK not a GUI app. - description needs improvement and I'm almost sure I made one based on Jussi's comment, but can't find it now ... - rpmlint complains about incorrect FSF address, upstream will be informed The package is APPROVED.
New Package SCM Request ======================= Package Name: ZipArchive Short Description: The ZipArchive library Owners: jwrdegoede sharkcz Branches: f16 InitialCC: Danny, thanks for the review! I'll improve the description after the initial import into git.
Git done (by process-git-requests).
ZipArchive has been imported and build for Rawhide and F-16. I don't intend to do a F-16 update in bodhi until we've the entire chain (ZipArchive, FlightCrew and Sigil) and then I'll push them all as one update -> closing.