[Bug 219930] Review Request: lxpanel - A lightweight X11 desktop panel

[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: lxpanel - A lightweight X11 desktop panel


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





------- Additional Comments From fedora at 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

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