[Bug 230071] Review Request: libkexiv2 - A library to manipulate EXIF/IPTC information

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libkexiv2 - A library to manipulate EXIF/IPTC information


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230071


wolfy@xxxxxxxxxxxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED




------- Additional Comments From wolfy@xxxxxxxxxxxxxxxxxx  2007-02-28 20:42 EST -------
First of all, it would have been nice if the URL of spec from comment #7 would
have been the modified one.
Second, I think that the Description tag in -devel should match the one from the
main package. You have modified only the main one...

As a comment prior to the review: examining the build log and then the sources,
I have noticed that %configure looks for doxygen and some doxygen related
scripts seem to exist in the admin directory. Unfortunately my experience with
doxygen is limited to installing it via yum. Just adding doxygen as BR makes
configure pick it, but nothing seems to be changed afterwards: nothing is built
in the admin directory andthe build log has some failures which _seem_ to look
like missing QT requirements. Explicitely requiring qt-devel does not help
either and unfortunately the machine where I have done all the tests just
suffered yet another kernel crash. OTOH the admin folder does not seem to
contain anything else but a bunch of scripts so I guess there is no need to
struggle with it (could be a leftover from a more comprehensive package from
which this tar has been separated). If I am wrong here, I would appreciate if
someone would correct me.

Good:

- rpmlint checks do not return anything on the source and binary rpm. the -devel
package gives:
W: libkexiv2-devel no-documentation
which I guess that can be ignored. The doc is in the main package.
- package meets naming guidelines
- package meets packaging guidelines
- license (GPL) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream, is the latest available version, sha1sum
 7218bb8b81955fc4ef376f302bda9b94bd2b68bd  libkexiv2-0.1.1.tar.bz2
- package compiles on devel (x86_64 and i386)
- no missing BR
- no unnecessary BR
- no locales 
- not relocatable
- owns all files/directories that it creates, does not take ownership of other
files/directories
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- not a GUI so no need for .desktop file 
- devel package ok
- no .la files (correctly removed in %install)
- post/postun ldconfig ok
- devel correctly requires pkgconfig
- separation of libs between main and devel packages respects versioning (once
again, I kindly ask someone more experienced to verify and correct me if I am wrong)


SHOULD
- successfully builds in mock for fc6 and devel, i386 and x86_64
warning:  I cannot verify if the libs are actually working, nothing to test them
with
- no scriptlets except usage of ldconfig which is OK
- pkgconfig(.pc) files are in -devel
- devel requires the base package using a fully versioned dependency.

What I would like to see
- including of AUTHORS, ChangeLog and maybe even the RELEASE.rev files as %doc
- the same Description in -devel as in main
Both are minor and non blockers, they can be fixed before uploading to cvs.

>From my point of view the package is OK (minus the minor caveats I have
mentioned ). However I kindly ask someone more experienced to have a second
look, therefore I will leave this bug in REVIEW state for another day or two.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]