[Bug 194279] Review Request: kdeartwork: Additional artwork (themes, sound themes, ...) for KDE

[bugzilla at redhat.com]

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: kdeartwork: Additional artwork (themes, sound themes, ...) for KDE


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





------- Additional Comments From rdieter at math.unl.edu  2006-06-14 10:41 EST -------
Re: comment #2:

>To get this to build I had to remove %{_datadir}/apps/kfiresaver/ for some
> reason its not getting built/installed
Probably due to qt bug #193741
I could probably workaround that here, for now, by adding 
BR: libGL-devel libGLU-devel

> - add "BuildRequires: gettext" for translations
AFAICT, there are no translations here (those bits are in kde-i18n-* pkgs), so
gettext ought not be needed.

> - User Interface/Desktop isn't a valid group name, unless its new to FC6?
That's what Core's kdeartwork package uses.  Do you have a better suggested value?

> - Under setup the %{?beta} tag is used, is this intentional?
I define the beta macro when/if there are any alpha/beta/preleleases done by
kde.  %{?beta} evaluates to %{nil} when undefined.  I could probably remove
that, since it's hardly used anymore.

> - Do any of these apps require .desktop files?
Apps no, screensaver's yes, in %%_datadir/applnk/System/Screensavers/

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.