Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at

Bug 221045

Summary: Review Request: gtk-rezlooks-engine - Rezlooks GTK2 engine
Product: [Fedora] Fedora Reporter: Mads Villadsen <maxx>
Component: Package ReviewAssignee: Mamoru TASAKA <mtasaka>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: rds204
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-01-26 22:19:38 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 163779    

Description Mads Villadsen 2006-12-31 02:28:36 UTC
Spec URL:
Description: Rezlooks is a gtk2 theme engine based on clearlooks.

This is my second package (first one is #220759), and I need a sponsor.

Comment 1 Mads Villadsen 2006-12-31 02:29:56 UTC

Comment 2 Robert Spanton 2006-12-31 14:32:39 UTC
What's the license of the "Source1?" files?  Those archives don't appear to
contain a license.

But otherwise:
 * rpmlint output is clean
 * source in srpm is as spec file specifies
 * builds on i386
 * installs and works on i386

Comment 3 Mads Villadsen 2006-12-31 15:28:59 UTC
I have found that unfortunately themes rarely contain a license. However
Gilouche and Graphite are linked to from the main page for the
theme and there the license is specified to be GPL.  The rest of the themes are
from the same author and all appear next to each other on his website

So I assumed GPL. I will try to get in touch with him to clear up any licensing

Comment 4 Mads Villadsen 2006-12-31 20:13:30 UTC
I have just received an e-mail from the author confirming that the themes are
under the GPL license.

So everything is okay there as well.

Comment 5 Mads Villadsen 2007-01-01 21:52:41 UTC
Removed the Rezlooks-Aerials theme since it required an extra metacity theme and
icon theme.

New spec url is the same as the old one.
New SRPM url:

Comment 6 Mads Villadsen 2007-01-13 22:41:20 UTC
I have uploaded a new version which adds COPYING file.


The new spec file has the same url as the old one.

Comment 7 Mamoru TASAKA 2007-01-19 11:06:55 UTC
Removing NEEDSPONSOR (bug 220759)

Comment 8 Mamoru TASAKA 2007-01-24 18:06:40 UTC
* Firstly, Fedora's packaging policy by default is "all or
  nothing" unless there is a certain reason for conditional
  dependency (check: "Conditional dependencies" in )

  What I mean by here is: I browsed under , there are 6 themes
  named "Rezlooks-XXX.tar.gz", while you seem to have chosen
  4 of them and 2 erased by the some reason.

  If you have a reason why you didn't include 2 themes, please
  comment in the spec file why (sorry if it was only because there
  was only 4 themes when you packaged this srpm). Otherwise, 
  you must include "all" themes or include "nothing".

  * rpmlint for srpm:
W: gtk-rezlooks-engine strange-permission Rezlooks-Snow.tar.gz 0600
W: gtk-rezlooks-engine strange-permission Rezlooks-Gilouche.tar.gz 0600
W: gtk-rezlooks-engine strange-permission gtk-rezlooks-engine.spec 0600
W: gtk-rezlooks-engine strange-permission Rezlooks-graphite.tar.gz 0600
W: gtk-rezlooks-engine strange-permission Rezlooks-candy.tar.gz 0600
W: gtk-rezlooks-engine strange-permission 39179-rezlooks-0.6.tar.gz 0600
   Please change the permission of files included in srpm
   to 0644.

   * Documentation
     /usr/share/doc/gtk-rezlooks-engine-0.6/NEWS says:
See Changelog for news.
     This file is useless and please include Changelog instead.

Comment 9 Mads Villadsen 2007-01-24 22:15:14 UTC
I have uploaded a new version which removes NEWS file and adds Changelog file.

I have also fixed the permissions, and included the Rezlooks-dark theme.

I have not included Rezlooks-Arials since it requires the nuoveXT-1.6 icon
theme, and contains a theme file which specifies the wrong controls and window
border themes. This is now also noted in the spec file.


The new spec file has the same url as the old one.

Comment 10 Mamoru TASAKA 2007-01-25 16:41:19 UTC

* All sources coincides with what I downloaded on md5sum.
* Packaging issues are all okay.

   This package (gtk-rezlooks-engine) is APPROVED by me