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-1.fc17.src.rpm Description: Python XMP Toolkit is a library for working with XMP metadata, as well as reading/writing XMP metadata stored in many different file formats. Fedora Account System Username: lkiesow This is the first package I'm submitting to Fedora. Thus I'm seeking a sponsor. I have been a packager for the Opencast Matterhorn project (opencast.org) for a while now.
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.