[Bug 372161] Review Request: gnome-themes-extras - extra themes for gnome

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gnome-themes-extras - extra themes for gnome


https://bugzilla.redhat.com/show_bug.cgi?id=372161





------- Additional Comments From alexl@xxxxxxxxxxxxxxxxxxxxx  2008-01-09 07:16 EST -------
MUST FIX items:

1) OK, I think that we should go back to the monolithic package for the moment,
to avoid duplication of docs and to make sure upgrades from the old
gnome-themes-extras goes smoothly.

The good news is that I did a koji build on rawhide that worked:

http://koji.fedoraproject.org/koji/taskinfo?taskID=336678

Also there are some (mostly) small issues with running rpmlint on the packages

$ rpmlint *.rpm

gnome-themes-extras.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab:
line 20)
gnome-themes-extras-darklooks.noarch: W: summary-not-capitalized darklooks theme
gnome-themes-extras-darklooks.noarch: E: description-line-too-long Darklooks is
a meta-theme that is part of the gnome-themes-extras package. This package
gnome-themes-extras-foxtrot.noarch: E: description-line-too-long Foxtrot is a
meta-theme that is part of the gnome-themes-extras package. This package
gnome-themes-extras-gion.noarch: E: description-line-too-long Gion is a
meta-theme that is part of the gnome-themes-extras package.  This package
gnome-themes-extras-neu.noarch: W: spelling-error-in-description pacakge package
gnome-themes-extras-neu.noarch: E: description-line-too-long Neu is a meta-theme
that is part of the gnome-themes-extras pacakge. This package

2) Remember, Description lines should be no longer than 80 chars, and that
Summary should start with a capital letter.

3) You are also mixing tabs and spaces in the spec file which makes the spec
file look odd in different editors, for example in Emacs it looks like this:

%package		foxtrot
Requires:		gnome-icon-theme, gnome-themes
Group:		User Interface/Desktops
Summary:		Foxtrot metatheme
%description	foxtrot
Foxtrot is a meta-theme that is part of the gnome-themes-extras package. This
package requires that you use a Gnome 2.2 release or newer.

4) * Mon Jan 7 2008 Marc Wiriadisastra <marc@xxxxxxxxxxxxxx> - 2.20-3
- Added patches to remove non-licensed themes

isn't true any more, so should be removed/updated.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]