Bug 517615

*** Bug 497614 has been marked as a duplicate of this bug. ***

1] patch have upstream ticket id? Was it reporter to upstream?

2] license...
there are files with GPLv2 and files with GPLv3
this is not good imho, could you please contact upstream and ask them to update license that only gplv3 or gplv2 will be used. otherwise it will be complicated and we would need to ask FE-LEGAL

> 1] patch have upstream ticket id? Was it reporter to upstream?
There is no upstream ticket concerning this patch. I already wrote an email to upstream but haven't got a reply yet.
 
> 2] license...
> there are files with GPLv2 and files with GPLv3
> this is not good imho, could you please contact upstream and ask them to update
> license that only gplv3 or gplv2 will be used. otherwise it will be complicated
> and we would need to ask FE-LEGAL  

Thanks for this hint. I'm going to ask upstream.

Spec URL: http://mgieseki.fedorapeople.org/icoutils/icoutils.spec
SRPM URL:
http://mgieseki.fedorapeople.org/icoutils/icoutils-0.28.0-1.fc11.src.rpm

Upstream maintainer Oskar Liljeblad updated the license to GPLv3+, adapted the comments accordingly, and applied the ppc build patch.

koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=1609039

rpmlint output is still clean.

rpmlint must be run on every package. 
- ok, rpmlint clean

The package must be named according to the Package Naming Guidelines.
- OK

The spec file name must match the base package %{name}, in the format %{name}.spec unless your package has an exemption.
- OK

The package must meet the Packaging Guidelines.
- OK

The package must be licensed with a Fedora approved license and meet the Licensing Guidelines.
- OK

The License field in the package spec file must match the actual license.
- OK

If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package must be included in %doc.
- OK

The spec file must be written in American English.
- OK

The spec file for the package MUST be legible.
- OK

The sources used to build the package must match the upstream source, as provided in the spec URL.
- OK

The package MUST successfully compile and build into binary rpms on at least one primary architecture.
- OK

ExcludeArch present.
- OK

All build dependencies must be listed in BuildRequires.
- OK

The spec file MUST handle locales properly.
- OK

Ldconfig in %post and %postun.
- not needed, OK

Relocatable package and /usr prefix.
- OK

A package must own all directories that it creates.
- OK

A Fedora package must not list a file more than once in the spec file's %files listings.
- OK

Permissions on files must be set properly.
- OK

Each package must have a correct %clean section.
- OK

Each package must consistently use macros.
- OK

The package must contain code, or permissable content.
- OK

Large documentation files must go in a -doc subpackage.
- OK, no large documentation

If a package includes something as %doc, it must not affect the runtime of the application.
- OK

Header files must be in a -devel package.
- OK

Static libraries must be in a -static package.
- OK, no static lib

Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig' .
- OK, no .pc

Library with .so suffix must be in -devel package.
- OK, no .so

In the vast majority of cases, devel packages must require the base package using a fully versioned dependency: Requires: %{name} = %{version}-%{release}
- OK, no devel

Packages must NOT contain any .la libtool archives, these must be removed in the spec if they are built.
- OK, no .la

Gui application and desktop-file-install.
- OK, no desktop

Packages must not own files or directories already owned by other packages.
- OK

At the beginning of %install, each package MUST run rm -rf %{buildroot} (or $RPM_BUILD_ROOT).
- OK

All filenames in rpm packages must be valid UTF-8.
- OK

conclusion: approved

Thanks for the review, Jan!


New Package CVS Request
=======================
Package Name: icoutils
Short Description: Utility for extracting and converting Microsoft icon and cursor files
Owners: mgieseki
Branches: F-10 F-11 F-12
InitialCC:

It's pretty early for wanting an F-12 branch of a new package, but we can create them now so I've gone ahead and done so.

CVS done.

icoutils-0.28.0-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/icoutils-0.28.0-2.fc10

icoutils-0.28.0-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/icoutils-0.28.0-1.fc11

icoutils-0.28.0-2.fc10 has been pushed to the Fedora 10 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update icoutils'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-9211

icoutils-0.28.0-1.fc11 has been pushed to the Fedora 11 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update icoutils'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-9223

icoutils-0.28.0-1.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

icoutils-0.28.0-2.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

icoutils is required to build the recent version of Wine. (bug #733801)


Package Change Request
======================
Package Name: icoutils
New Branches: el6
Owners: mgieseki

Git done (by process-git-requests).