Bug 551265 - Review Request: gnome-xcf-thumbnailer - Thumbnailer for XCF files
Summary: Review Request: gnome-xcf-thumbnailer - Thumbnailer for XCF files
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Nicoleau Fabien
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-12-29 17:54 UTC by Mohamed El Morabity
Modified: 2010-02-18 22:24 UTC (History)
3 users (show)

Fixed In Version: gnome-xcf-thumbnailer-1.0-2.fc12
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-02-18 22:21:18 UTC
Type: ---
Embargoed:
nicoleau.fabien: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Mohamed El Morabity 2009-12-29 17:54:39 UTC
Spec URL: http://melmorabity.fedorapeople.org/packages/gnome-xcf-thumbnailer/gnome-xcf-thumbnailer.spec
SRPM URL: http://melmorabity.fedorapeople.org/packages/gnome-xcf-thumbnailer/gnome-xcf-thumbnailer-1.0-1.fc12.src.rpm
Description: Thumbnailer for GIMP's own format, XCF files

mock is OK, rpmlint is silent both with the SRPM and the debug RPM, not on the binary RPM itself:
gnome-xcf-thumbnailer.x86_64: W: conffile-without-noreplace-flag /etc/gconf/schemas/gnome-xcf-thumbnailer.schemas
A configuration file is stored in your package without the noreplace flag. A
way to resolve this is to put the following in your SPEC file:
%config(noreplace) /etc/your_config_file_here
Anyway, this can be ignored for GConf files.

Note that I need a sponsor, although it's not my first package (see my other
review requests 551027 and 551042).

Comment 1 Mohamed El Morabity 2010-01-26 07:56:57 UTC
Some minor fix (replaced in %files section %{bindir}/* by %{bindir}/gnome-xcf-thumbailer) :

Spec URL : http://melmorabity.fedorapeople.org/packages/gnome-xcf-thumbnailer/gnome-xcf-thumbnailer.spec
SRPM URL:
http://melmorabity.fedorapeople.org/packages/gnome-xcf-thumbnailer/gnome-xcf-thumbnailer-1.0-2.fc12.src.rpm

Comment 2 Mohamed El Morabity 2010-01-26 07:57:24 UTC
Some minor fix (replaced in %files section %{bindir}/* by %{bindir}/gnome-xcf-thumbailer) :

Spec URL : http://melmorabity.fedorapeople.org/packages/gnome-xcf-thumbnailer/gnome-xcf-thumbnailer.spec
SRPM URL:
http://melmorabity.fedorapeople.org/packages/gnome-xcf-thumbnailer/gnome-xcf-thumbnailer-1.0-2.fc12.src.rpm

Comment 3 Nicoleau Fabien 2010-02-02 21:23:38 UTC
I'll review this.

Comment 4 Nicoleau Fabien 2010-02-02 22:32:16 UTC
Package revienw : 

 [OK]  Package is named according to the Package Naming Guidelines
 [OK]  Specfile name matches %{name}.spec
 [OK]  Package seems to meet Packaging Guidelines
 [OK]  Package successfully compiles and builds into binary RPMs on at least one
       supported architecture : Builds on koji
 [OK]  Rpmlint output: the warning about "noreplace" config file can be ignored
 [OK]  Package is not relocatable.
 [OK]  License in specfile matches actual License and meets Licensing Guidelines
 [OK]  License file is included in %doc.
 [OK]  Specfile is legible and written in AE
 [OK]  Sourcefile in the Package is the same as provided in the mentioned Source
 [OK]  Package compiles successfully
 [OK]  All build dependencies are listed in BuildRequires
 [N/A] Specfile handles locales properly
 [N/A] ldconfig called in %post and %postun if required
 [N/A] Package owns directorys it creates
 [OK]  Package requires other packages for directories it uses.
 [OK] Package does not list a file more than once in the %files listing
 [OK] %files section includes %defattr and permissions are set properly
 [OK] %clean section is there and contains rm -rf %{buildroot}
 [OK] Macros are consistently used
 [OK] Package contains code, or permissable content.
 [N/A] Large documentation files are in a -doc subpackage
 [OK] Program runs properly without files listed in %doc
 [N/A] Header files are in a -devel package
 [N/A] Static libraries are in a -static package
 [N/A] Package requires pkgconfig if .pc files are present
 [N/A] .so-files are put into a -devel subpackage
 [N/A] Subpackages include fully versioned dependency for the base package
 [N/A] Any libtool archives (*.la) are removed
 [N/A] contains desktop file (%{name}.desktop) if it is a GUI application
 [OK] Package does not own files or directories owned by other packages.
 [OK] %{buildroot} is removed at beginning of %install
 [N/A] Filenames are encoded in UTF-8

Also GConf rules are respected : http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#GConf

This package is APPROVED

Comment 5 Mohamed El Morabity 2010-02-03 04:07:46 UTC
New Package CVS Request
=======================
Package Name: gnome-xcf-thumbnailer
Short Description: Thumbnailer for GIMP's own format, XCF files
Owners: melmorabity
Branches: F-11 F-12
InitialCC:

Comment 6 Kevin Fenzi 2010-02-03 04:12:30 UTC
CVS done (by process-cvs-requests.py).

Comment 7 Fedora Update System 2010-02-03 04:43:18 UTC
gnome-xcf-thumbnailer-1.0-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/gnome-xcf-thumbnailer-1.0-2.fc11

Comment 8 Fedora Update System 2010-02-03 04:43:59 UTC
gnome-xcf-thumbnailer-1.0-2.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/gnome-xcf-thumbnailer-1.0-2.fc12

Comment 9 Fedora Update System 2010-02-05 01:18:50 UTC
gnome-xcf-thumbnailer-1.0-2.fc12 has been pushed to the Fedora 12 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 gnome-xcf-thumbnailer'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2010-1434

Comment 10 Fedora Update System 2010-02-05 01:50:41 UTC
gnome-xcf-thumbnailer-1.0-2.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 gnome-xcf-thumbnailer'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2010-1511

Comment 11 Fedora Update System 2010-02-18 22:21:12 UTC
gnome-xcf-thumbnailer-1.0-2.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Fedora Update System 2010-02-18 22:24:35 UTC
gnome-xcf-thumbnailer-1.0-2.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.


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