Bug 305731

Summary: Review Request: fedora-gnome-theme - Fedora GNOME theme
Product: [Fedora] Fedora Reporter: Ray Strode [halfline] <rstrode>
Component: Package ReviewAssignee: Matthias Clasen <mclasen>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: low    
Version: rawhideCC: fedora-package-review, notting
Target Milestone: ---Flags: mclasen: fedora-review+
notting: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-10-11 15:58:37 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 305441    

Description Ray Strode [halfline] 2007-09-25 17:42:06 UTC
Spec URL:
http://rstrode.fedorapeople.org/fedora-gnome-theme/checkout/fedora-gnome-theme.spec

SRPM URL:
http://rstrode.fedorapeople.org/fedora-gnome-theme/checkout/fedora-gnome-theme-1.0.0-1.fc8.src.rpm

Description:
This package contains the fedora gnome meta theme.

Comment 1 Ray Strode [halfline] 2007-09-25 17:47:25 UTC
Rex, Than,

Sorry for the bug spam, I didn't mean to CC you guys on this package.

Comment 2 Matthias Clasen 2007-09-25 19:32:38 UTC
License field invalid
Source needs to be a url or have a comment
find_lang construct could do with a comment

So, this package now contains the default system gtkrc
but the gconf settings for the default theme still live somewhere else ?

Requires seem somewhat incongruent,
Requires: nodoka-theme-gnome
Requires: fedora-icon-theme
Requires: fedora-screensaver-theme
Requires: fedorainfinity-screensaver-theme

Since nodoka-theme-gnome is already a "meta" package. shouldn't we
explicitly list all the components ?

Comment 3 Ray Strode [halfline] 2007-09-26 21:32:40 UTC
fixed

Comment 4 Matthias Clasen 2007-09-27 18:34:33 UTC
All looks fine now, approved.

Comment 5 Ray Strode [halfline] 2007-10-09 16:08:36 UTC
New Package CVS Request
=======================
Package Name: fedora-gnome-theme
Short Description: Fedora GNOME theme
Owners: rstrode,davidz
Branches:
InitialCC:
Cvsextras Commits: no

Comment 6 Bill Nottingham 2007-10-09 21:34:39 UTC
done