[Bug 476357] Review Request: libicns - Library for manipulating Macintosh icns 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.


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





--- Comment #2 from Andrea Musuruane <musuruan@xxxxxxxxx>  2008-12-18 06:14:46 EDT ---
> - [ ??? ] MUST: The package must meet the Packaging Guidelines.
> Why -utils subpackage?

Well, the library can be used without the utilities - therefore the split. The
utilities are end user utilities that use this library to convert icns to png
and vice versa.

> - [ BAD ] MUST: The sources used to build the package must match the
> upstream source, as provided in the spec URL. Reviewers should use
> md5sum for this task. If no upstream URL can be specified for this
> package, please see the Source URL Guidelines for how to deal with this.
> 
> md5sum = f6ab74c2cfd4ab5b11d1cb9114743c90
> 
> I can't actually download this from the given URL -- sourceforge gives
> me a bunch of redirects and then a 404.

I just followed the packaging guideline to provide a source URL that is on
Sourceforge:
https://fedoraproject.org/wiki/Packaging/SourceURL#Sourceforge.net

To check the md5, you can use a mirror.

> - [ BAD ] MUST: Packages containing pkgconfig(.pc) files must
> 'Requires: pkgconfig' (for directory ownership and usability).

I'll fix this.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

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