[Bug 234079] Review Request: kicker-compiz - Pager applet for kicker to make it work with compiz

[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: kicker-compiz - Pager applet for kicker to make it work with compiz


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





------- Additional Comments From cgoorah at yahoo.com.au  2007-06-28 17:06 EST -------
ok, let's add it into fedora, it might at least satisfy a user.

#01: summary
can you change the summary from
Summary:        Pager applet for kicker to make it work with compiz
to
Summary:        Makes KDE kicker Pager applet work with compiz

The idea is to pull/locate kicker-compiz via the word "KDE" into the search 
engine of pirut.

#02: Build Requirements:
libXcomposite-devel already requires libXfixes-devel

rpm -qR libXcomposite-devel
libXcomposite = 0.3-5.1
libXcomposite.so.1
libXfixes-devel
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
xorg-x11-filesystem >= 0.99.2-3
xorg-x11-proto-devel >= 7.0-10

IS there any particular reason of doxygen's presence among BRs ?

#3: Description versus README
The README file contains virtually no important information for the user, 
other than:
Once it is installed, use the kicker dialog to add a new applet, and the pager
will appear in the list (Right-click on kicker, then "add an applet to the
panel" -> "Pager - Compiz").

Hence wipe off the README file and modify your description to be:
%description
This is a modified pager applet for kicker to make it work with
compiz. More generally, it is intended to work with window 
managers that use the concept of "large desktops" instead of 
"multiple virtual desktops" as kwin does exclusively.
Once it is installed, use the kicker dialog to add a new applet,
and the pager will appear in the list (Right-click on kicker, 
then "add an applet to the panel" -> "Pager - Compiz")

#4: QT libs
These are not neccesary on fedora platforms (thanks to Rex :) )
unset QTDIR || : ; . /etc/profile.d/qt.sh
export QTLIB=${QTDIR}/lib QTINC=${QTDIR}/include



-- 
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.