Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 986550 - Review Request: python-xmp-toolkit - Python toolkit for working with XMP metadata
Summary: Review Request: python-xmp-toolkit - Python toolkit for working with XMP meta...
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2013-07-20 14:07 UTC by Lars Kiesow
Modified: 2017-04-17 14:09 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2017-04-17 13:15:19 UTC

Attachments (Terms of Use)

Description Lars Kiesow 2013-07-20 14:07:15 UTC
Spec URL:
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 ( for a while now.

Comment 1 Mario Blättermann 2013-07-20 20:27:09 UTC
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 for more info.

BuildRequires: python-devel
is deprecated, you have to use python2-devel instead:

Comment 2 Lars Kiesow 2013-07-20 22:31:09 UTC
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:

Comment 3 Christopher Meng 2013-07-20 22:55:42 UTC
And please remove the dot at the end of summary.

Comment 4 Christopher Meng 2013-07-20 23:32:03 UTC
Also,  please scratch builds for rawhide and f19 via Koji, because f17 is nearly EOL.


Comment 6 Christopher Meng 2013-07-22 00:52:58 UTC

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


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

Comment 7 Martin Gieseking 2013-07-22 06:47:45 UTC
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/

Also, remove file .buildinfo from docs/html.

Comment 9 Mario Blättermann 2013-08-18 18:29:29 UTC
The tarball contains a bundled egg, please remove it before building the package.
See 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

Comment 10 Lars Kiesow 2013-09-22 17:12:05 UTC
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:

And here are new scratch builds:

Comment 11 Parag AN(पराग) 2015-08-22 07:08:50 UTC
Hi Lars,
   We have this process 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


3) To find the packages already submitted for review, check

4) and is useful while doing package reviews.

5) 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

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 :)

Comment 12 Parag AN(पराग) 2015-08-24 03:10:22 UTC
I tried today to review your package but links are not working. Please upload the spec and srpm links.

Comment 13 Parag AN(पराग) 2015-09-23 06:37:17 UTC
Still the links are not working.

Note You need to log in before you can comment on or make changes to this bug.