[Bug 238373] Review Request: libopenraw - decode camera RAW files.

[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: libopenraw - decode camera RAW files.


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





------- Additional Comments From moritz@xxxxxxxxxxxx  2007-05-02 07:44 EST -------
(In reply to comment #1)

I don't want to be picky, but:

> checking pkg-config is at least version 0.9.0... yes

Shouldn't this imply:
BuildRequires: pkgconfig >= 0.9.0
??

Perhaps you might want to version the
Requires: pkgconfig
for -devel as well, dunno. *shrug*

Note: This isn't a package specific requirement, but seems to be introduced by 
autoconf. The requirement or the versioning _may_ therefore be obsolete or 
quietly ignorable. pkgconfig interestingly _is not_ in the exception list for 
BRs: http://fedoraproject.org/wiki/Extras/FullExceptionList

Re the versioning: The Fedora packaging guidelines (http://fedoraproject.org/
wiki/Packaging/Guidelines#head-0711805dd733fe3b31741e9d5236d72941a79d94) say:
"First, if the lowest possible requirement is so old that nobody has a version 
older than that installed on any target distribution release, there's no need 
to include the version in the dependency at all." This may very much be the 
case for this pkgconfig version. :)

Moritz,
off to build his own gthumb with this lib, thanks for the package!

-- 
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]