Bug 566695

Summary:	Review Request: goddard-kde-theme - Goddard KDE Theme
Product:	[Fedora] Fedora	Reporter:	Jaroslav Reznik <jreznik>
Component:	Package Review	Assignee:	Rex Dieter <rdieter>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	fedora-package-review, notting, rdieter
Target Milestone:	---	Flags:	rdieter: fedora-review+ j: 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:	2010-02-25 16:20:49 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:	566290
Bug Blocks:

Description Jaroslav Reznik 2010-02-19 13:26:15 UTC

Spec URL: http://rezza.hofyland.cz/fedora/packages/goddard-kde-theme/goddard-kde-theme.spec
SRPM URL: http://rezza.hofyland.cz/fedora/packages/goddard-kde-theme/goddard-kde-theme-12.91.0-1.fc12.src.rpm
Description: This is Goddard KDE Theme Artwork containing KDM and KSplash theme.

Comment 1 Rex Dieter 2010-02-20 10:07:44 UTC

1.  Should add some Provides similar to what constantine-kde-theme has:

%if 0%{?fedora} == 13
Provides:       system-kde-theme = %{version}-%{release}
Provides:       system-kdm-theme = %{version}-%{release}
Provides:       system-ksplash-theme = %{version}-%{release}
%endif

(This can be addressed post review)

2. rpmlint:

goddard-kde-theme.src: W: spelling-error %description -l en_US KSplash -> K Splash, KS plash, KS-plash
goddard-kde-theme.noarch: W: spelling-error %description -l en_US KSplash -> K Splash, KS plash, KS-plash
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/ksplash/Themes/Goddard/1400x1050/logo.png ../../../../../../pixmaps/system-logo-white.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/system-logo-white.png ../../../../../pixmaps/system-logo-white.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1280x768.png ../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1024x768.png ../../../../../backgrounds/goddard/default/standard/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1152x720.png ../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1440x900.png ../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1366x768.png ../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1280x720.png ../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/ksplash/Themes/Goddard/1280x800/goddard.png ../../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-640x480.png ../../../../../backgrounds/goddard/default/standard/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1920x1200.png ../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1200x900.png ../../../../../backgrounds/goddard/default/standard/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1920x1080.png ../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1280x960.png ../../../../../backgrounds/goddard/default/standard/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1920x1440.png ../../../../../backgrounds/goddard/default/standard/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/ksplash/Themes/Goddard/1400x1050/goddard.png ../../../../../../backgrounds/goddard/default/standard/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1024x600.png ../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1152x864.png ../../../../../backgrounds/goddard/default/standard/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1440x1080.png ../../../../../backgrounds/goddard/default/standard/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-800x600.png ../../../../../backgrounds/goddard/default/standard/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1600x1200.png ../../../../../backgrounds/goddard/default/standard/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1680x1050.png ../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard.png ../../../../../backgrounds/goddard/default/standard/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-1280x800.png ../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-800x480.png ../../../../../backgrounds/goddard/default/wide/goddard.png
goddard-kde-theme.noarch: W: dangling-relative-symlink /usr/share/kde4/apps/kdm/themes/Goddard/goddard-2048x1536.png ../../../../../backgrounds/goddard/default/standard/goddard.png
2 packages and 0 specfiles checked; 0 errors, 28 warnings.

Comment 2 Rex Dieter 2010-02-20 10:17:14 UTC

Functionality OK

Licensing OK

macros  OK, consistent

%files  OK (simple)

Source tarball doesn't match upstrea
in src.rpm:
$ md5sum *.bz2
9e88306f9216a59eb603edcd90dceb20  goddard-kde-theme-12.91.0.tar.bz2

downloaded from fedorahosted:
$ md5sum *.bz2
126e870673e448b4481cec8cefd62ad8  goddard-kde-theme-12.91.0.tar.bz2

Looks like the only difference in contant are the embedded .spec file, mostly harmless. :)


Looks good, APPROVED.

Comment 3 Jaroslav Reznik 2010-02-20 10:42:32 UTC

Thanks Rex.

I'll add correct provides and update correct tarball (it's going to change).

Comment 4 Jaroslav Reznik 2010-02-20 10:45:02 UTC

New Package CVS Request
=======================
Package Name: goddard-kde-theme
Short Description: Goddard KDE Theme
Owners: jreznik than rdieter kkofler ltinkl
Branches: F-13
InitialCC:

Comment 5 Jason Tibbitts 2010-02-20 18:03:12 UTC

CVS done (by process-cvs-requests.py).

Comment 6 Rex Dieter 2010-02-25 16:20:49 UTC

imported.