Red Hat Bugzilla – Bug 215158
Review Request: xfce4-eyes-plugin - Eyes for the Xfce panel
Last modified: 2014-09-23 08:04:02 EDT
Spec URL: http://home.arcor.de/christoph.wickert/fedora/extras/review/SPECS/xfce4-eyes-plugin.spec
SRPM URL: http://home.arcor.de/christoph.wickert/fedora/extras/review/SRPMS/xfce4-eyes-plugin-126.96.36.199-1.fc7.src.rpm
Description: A xfce4 panel plugin that adds eyes which watch your every step. Scary!
In XFCE 4.2 this plugin used to be part of the xfce-toys, in 4.4 it's seperate package.
It is possible to remove everything that owns /usr/share/xfce4/
and xfce4-eyes-plugin can still be installed. It is not
obvious that xfce4-eyes-plugin should depend on a package
which owns /usr/share/xfce4/, however maybe xfce4-panel should.
(In reply to comment #1)
> It is possible to remove everything that owns /usr/share/xfce4/
> and xfce4-eyes-plugin can still be installed.
Really? The package depends on xfce4-panel, which owns
> It is not
> obvious that xfce4-eyes-plugin should depend on a package
> which owns /usr/share/xfce4/, however maybe xfce4-panel should.
I have been talking about that with Kevin before. CC'ing him.
$ rpm -qf /usr/share/xfce4
After removing those packages and the dependent packages, xfce
isn't functional anymore, but there is still certainly a packaging
mistake somewhere since xfce4-panel is still there.
Of course you are correct.
How about this: xfce4-panel should depend on xfce4-icon-theme, since the "add
new starter"-dialog uses the default xfce icons. xfce4-icon-theme also owns
(In reply to comment #4)
> Of course you are correct.
> How about this: xfce4-panel should depend on xfce4-icon-theme, since the "add
> new starter"-dialog uses the default xfce icons. xfce4-icon-theme also owns
Right. I used a similar argument to convince Kevin to have Thunar depend
* follow packaging guidelines
* rpmlint is silent
* licence GPL match package, included
* match upstream
* sane provides
* build and works in devel
* %files right except that /usr/share/xfce4/ is unowned, but it is
also the case for xfce4-panel which is the culprit here.
* %doc not runtime
$ ldd -u -r /usr/libexec/xfce4/panel-plugins/xfce4-eyes-plugin
Unused direct dependencies:
Except for /lib/libdl.so.2, this should certainly be solved to avoid
unneeded rebuild when those soname changes. This is not a blocker,
but certainly shows some issues somewhere (upstream, or in the
devel packages corresponding with the sonames).
In reply to comment #5:
Yeah, I guess xfce4-panel should Requires: xfce4-icon-theme. That would fix this
issue up unless I am missing something. I will queue that change for the next
time I update xfce4-panel.
Thanks for the review, Patrice.
(In reply to comment #6)
> $ ldd -u -r /usr/libexec/xfce4/panel-plugins/xfce4-eyes-plugin
> Unused direct dependencies:
> Except for /lib/libdl.so.2, this should certainly be solved to avoid
> unneeded rebuild when those soname changes. This is not a blocker,
> but certainly shows some issues somewhere (upstream, or in the
> devel packages corresponding with the sonames).
I'll have to take a closer look at this, it's nearly the same for many other
Imported into CVS, sync fore Core 6 is requested, built for devel and added to
Package Change Request
Package Name: xfce4-eyes-plugin
New Branches: epel7
Git done (by process-git-requests).