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: lxpanel - A lightweight X11 desktop panel
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219930
------- Additional Comments From fedora@deadbabylon.de 2006-12-17 05:05 EST ------- (In reply to comment #1)
(In reply to comment #0)
SRPM URL:
http://deadbabylon.de/fedora/extras/lxpanel/lxpanel-0.2.4-2.src.rpm
Error 404.
My fault. Fixed it.
I took your spec and built the package, but there are some things it don't
like:
- Source0: should not use a mirror, instead use
dl.sourceforge.net/sourceforge/lxde/%{name}-%{version}.tar.gz
Corrected.
- BR startup-notification-devel?
Added.
- In the default config two starters (firefox and pcman-fm) are not working.
Please take a look at Chung-Yen's package at ftp://cle.linux.org.tw/pub/fedora/cle/6/SRPMS/lxpanel-0.2.4-1.fc6.src.rpm for a fix.
Added the patch.
Some othe more thoughts: I think someone should also roll a pcman-fm
package, we
should package the whole lx-desktop in extras. I have an icewm package that
I
can offer. You and Chung-Yen should unite your powers on pcman-fm, lxpanel, lxsession and lxmusic. What do you think?
I didn't know that someone else ist working on it. I will ask him what he would think. Package the whole LXDE-Desktop could be interesting for others. I only use lxpanel for myself and have never used lxmusic. Will have a look on it.
As long as we don't have no pcman-fm package I suggest this starter should
be
removed from the default config.
At first I've thought to replace pcmanfm.desktop with gnome-nautilus-home.desktop. But perhaps this could irritate some users when updating the package later and replace it again with pcmanfm.desktop.
For the moment: SPEC Url: http://deadbabylon.de/fedora/extras/lxpanel/lxpanel.spec SRPM Url: http://deadbabylon.de/fedora/extras/lxpanel/lxpanel-0.2.4-3.src.rpm
Changelog: - BR: startup-notification-devel - Added Patch1 from Chung-Yen to fix wrong starters in default config