Bug 986550

Your package is noarch and stores its files in %{python_sitelib}. You have removed the wrong header in the spec file (that one which refers to noarch packages).

BTW, this header in general and some other parts are obsolete if you don't want to provide your package for EPEL 5:

* the BuildRoot definition
* the initial cleaning of $RPM_BUILD_ROOT in %install
* the %clean section
* the %defattr line in %files

If you are not pointing to EPEL 5, those lines can be safely dropped. Well, you are not forced to do so due to the guidelines, but it doesn't make sense anyway to keep them. The functionality of the mentioned macros is default for EPEL >= 6 and all currently supported Fedora versions, and the spec file becomes better readable. See http://fedoraproject.org/wiki/Packaging:Python#Macros for more info.

BuildRequires: python-devel
is deprecated, you have to use python2-devel instead:
http://fedoraproject.org/wiki/Packaging:Python#BuildRequires

Thanks for the review. I removed the the obsolete headers and changed the deprecated python-devel to python2-devel. I uploaded the new spec/srpm here:

Spec URL: http://larskiesow.de/python-xmp-toolkit/python-xmp-toolkit.spec
SRPM URL: http://larskiesow.de/python-xmp-toolkit/python-xmp-toolkit-1.0.2-2.fc17.src.rpm

And please remove the dot at the end of summary.

Also,  please scratch builds for rawhide and f19 via Koji, because f17 is nearly EOL.

Thanks.

Dot removed.
And here are the scratch builds:
f18: http://koji.fedoraproject.org/koji/taskinfo?taskID=5636786
f19: http://koji.fedoraproject.org/koji/taskinfo?taskID=5636774
f20: http://koji.fedoraproject.org/koji/taskinfo?taskID=5636784

Issue:

1. Remove %defattr(-,root,root,-) in the %files section.

Suggestions:

1. SOrt things like this:

Name:           python-xmp-toolkit
Version:        1.0.2
Release:        2%{?dist}
Summary:        Python XMP Toolkit for working with metadata
Group:          Development/Libraries
License:        BSD
URL:            http://code.google.com/p/python-xmp-toolkit/
Source0:        https://pypi.python.org/packages/source/p/%{name}/%{name}-%{version}.tar.gz
BuildArch:      noarch
BuildRequires:  python2-devel
BuildRequires:  python-setuptools
Requires:       exempi

You don't need to leave too many empty lines :)

2. %setup -q -n %{name}-%{version}
 can be safely replaced by 

%setup -q

3. You can remove VERSION file in doc as it's useless.

Hi Lars,

great to see you here. :)

A few additional notes:
Please be more explicit in the %files section to prevent adding unwanted files by accident. This also helps to get an idea what's actually going into the package: 

%doc docs/html/
%{python_sitelib}/libxmp/
%{python_sitelib}/python_xmp_toolkit-*.egg-info/

Also, remove file .buildinfo from docs/html.

Suggestions included.

New files can be found here:
Spec URL: http://larskiesow.de/python-xmp-toolkit/python-xmp-toolkit.spec
SRPM URL: http://larskiesow.de/python-xmp-toolkit/python-xmp-toolkit-1.0.2-3.fc17.src.rpm

And here are new scratch builds:
f18: http://koji.fedoraproject.org/koji/taskinfo?taskID=5659205
f19: http://koji.fedoraproject.org/koji/taskinfo?taskID=5659217
f20: http://koji.fedoraproject.org/koji/taskinfo?taskID=5659235

The tarball contains a bundled egg, please remove it before building the package.
See http://fedoraproject.org/wiki/Packaging:Python_Eggs#Upstream_Eggs for more info.

(In reply to Mario Blättermann from comment #1)
> ... parts are obsolete if you don't
> want to provide your package for EPEL 5:
> ...
> * the %defattr line in %files

Sorry for the late reply. I somehow missed your answer (has been a while since something happened here).

New files can be found here:
Spec URL: http://larskiesow.de/python-xmp-toolkit/python-xmp-toolkit.spec
SRPM URL: http://larskiesow.de/python-xmp-toolkit/python-xmp-toolkit-1.0.2-4.fc18.src.rpm

And here are new scratch builds:
f18: http://koji.fedoraproject.org/koji/taskinfo?taskID=5968890
f19: http://koji.fedoraproject.org/koji/taskinfo?taskID=5968904
f20: http://koji.fedoraproject.org/koji/taskinfo?taskID=5968912

Hi Lars,
   We have this process http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group to get sponsored into the packager group. Can you either submit few more packages and/or some full detailed package reviews? This is needed to make sure package submitter understands the rpm packaging well and follows the fedora packaging guidelines.

Please go through the following links
1) http://fedoraproject.org/wiki/Package_Review_Process

2) https://fedoraproject.org/wiki/PackagingGuidelines

3) To find the packages already submitted for review, check http://fedoraproject.org/PackageReviewStatus/

4) http://fedoraproject.org/wiki/Packaging:ReviewGuidelines and http://fedoraproject.org/wiki/Package_Review_Process#Reviewer is useful while doing package reviews.

5) https://fedorahosted.org/FedoraReview/ this is fedora-review tool to help review packages in fedora. You need to use this and do un-official package reviews of packages submitted by other contributors. While doing so mention "This is un-official review of the package." at top of your review comment.

Good to review packages listed in http://fedoraproject.org/PackageReviewStatus/NEW.html

When you do full package review of some packages, provide that review comment link here so that I can look how you have reviewed those packages.

If you got any questions please ask :)

I tried today to review your package but links are not working. Please upload the spec and srpm links.

Still the links are not working.