Bug 484423 - Review Request: netbook-launcher - A clutter-based desktop launcher
Summary: Review Request: netbook-launcher - A clutter-based desktop launcher
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Christoph Wickert
QA Contact: Fedora Extras Quality Assurance
: 451772 (view as bug list)
Depends On:
Blocks: FE-DEADREVIEW FedoraMini
TreeView+ depends on / blocked
Reported: 2009-02-06 17:54 UTC by Michel Alexandre Salim
Modified: 2010-07-16 18:39 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2010-07-16 18:39:12 UTC
Type: ---

Attachments (Terms of Use)
replaces non working dialog with gnome defaults. (2.63 KB, patch)
2009-03-23 16:07 UTC, Traxtopel
no flags Details | Diff
Update logout/etc patch (3.83 KB, patch)
2009-04-29 14:19 UTC, Traxtopel
no flags Details | Diff

System ID Private Priority Status Summary Last Updated
Launchpad 369942 0 None None None Never

Description Michel Alexandre Salim 2009-02-06 17:54:52 UTC
Spec URL: http://salimma.fedorapeople.org/specs/gnome/netbook-launcher.spec
SRPM URL: http://salimma.fedorapeople.org/specs/gnome/netbook-launcher-1.2-1.fc11.src.rpm
Netbook Launcher is a desktop launcher that uses the clutter UI
library. It is commonly being used on netbook desktops with a
resolution of 1024x600 pixels and also supposed to support usage on
touchscreens. It follows the xdg spec standards from freedesktop.org
for the Desktop menu layout.

Comment 1 Michel Alexandre Salim 2009-02-06 18:01:59 UTC
*** Bug 451772 has been marked as a duplicate of this bug. ***

Comment 2 Michel Alexandre Salim 2009-02-06 18:04:47 UTC
F10 Koji build:

It currently does not build on Rawhide due to some Dwarf error:

/usr/bin/ld: Dwarf Error: Offset (5317) greater than or equal to .debug_str size (4647).
main.o: In function `main':
/home/michel/rpmbuild/BUILD/netbook-launcher-1.2/src/main.c:118: undefined reference to `libgnomeui_module_info_get'
/home/michel/rpmbuild/BUILD/netbook-launcher-1.2/src/main.c:118: undefined reference to `gnome_program_init'

... (the list goes on)

Comment 3 Michel Alexandre Salim 2009-03-02 21:04:26 UTC
OK, fixed now. The new versions are listed on a different page.

Spec URL: http://salimma.fedorapeople.org/specs/gnome/netbook-launcher.spec

Rawhide build:

Comment 4 Christoph Wickert 2009-03-10 03:26:50 UTC
If this is the successor of ume-launcher, it should certainly obsolete it.

Comment 5 Michel Alexandre Salim 2009-03-10 13:08:18 UTC
 It is indeed. Upstream documentation is a bit sparse, I (or someone else) will need to observe how Ubuntu switches between maximus/netbook-launcher and the normal GNOME stack.

Currently, installing this and maximus means that they both get started on your next GNOME seession. Not much fun.

Comment 6 Christoph Wickert 2009-03-10 13:20:24 UTC
Another issue that comes to my mind is the name. "Netbook" is a tradmark of Psion and the lawsuit Dell, Intel and others vs. Psion is not finished yet.

Comment 7 Traxtopel 2009-03-23 14:36:28 UTC
Did you find a workaround for the gdm_socket, this software still uses

  "/var/run/gdm_socket" G_SEARCHPATH_SEPARATOR_S \

Without this being updated you cannot logout using the panel.

Should also be pre/post script added to spec.

if [ "$1" -gt 1 ]; then
    export GCONF_CONFIG_SOURCE=`gconftool-2 --get-default-source`
    gconftool-2 --makefile-uninstall-rule \
      %{_sysconfdir}/gconf/schemas/%{name}.schemas >/dev/null || :
    killall -HUP gconfd-2 || :

export GCONF_CONFIG_SOURCE=`gconftool-2 --get-default-source`
gconftool-2 --makefile-install-rule \
  %{_sysconfdir}/gconf/schemas/%{name}.schemas > /dev/null || :
killall -HUP gconfd-2 || :

if [ "$1" -eq 0 ]; then
    export GCONF_CONFIG_SOURCE=`gconftool-2 --get-default-source`
    gconftool-2 --makefile-uninstall-rule \
      %{_sysconfdir}/gconf/schemas/%{name}.schemas > /dev/null || :
    killall -HUP gconfd-2 || :

Comment 8 Traxtopel 2009-03-23 16:07:23 UTC
Created attachment 336312 [details]
replaces non working dialog with gnome defaults.

Patched for 1.6.16

Lock Screen
Defaults to gnome applets.

Comment 9 Michel Alexandre Salim 2009-04-29 12:45:06 UTC
Thanks! And sorry for the delay. I've been running 1.6.21 and most things work (apart from, strangely, the Log Out button).


Should we ask the legal team about the netbook trademark?

Comment 10 Traxtopel 2009-04-29 14:18:33 UTC
The logout button will not work since it prereqs an older gdm (needs socket), needs a rewrite. The patch I proposed adds functionality.

Comment 11 Traxtopel 2009-04-29 14:19:32 UTC
Created attachment 341758 [details]
Update logout/etc patch

Workaround for logout button issue.

Comment 12 Michel Alexandre Salim 2009-04-30 17:23:00 UTC
Thanks. I've stayed away from the UI changes -- on a netbook, the right-hand side is cluttered enough already with my bookmarks that adding any more default buttons makes the interface less usable.

I've fixed the GDM calls for logout, shutdown and restart:



and send the patches upstream (see external bug report link)

Would you like to do a formal review? The only unpatched thing right now is the Ubuntu icon -- I'd perhaps ask the art team to take a look.

Comment 13 Christoph Wickert 2009-06-08 13:56:11 UTC
Ok, now that the lawsuit is settled, I'm going to review this. Stay tuned.

Comment 14 Christoph Wickert 2009-06-12 02:10:06 UTC
OK, just a short message I'm still working on this. From a packaging POV most things look good (but there are some problems), but the app itself needs some love, especially the patch:

- Logout dialog is huge, at least when using German localization
- The dialog is not closed after switching user or locking screen
- Icons standby and hibernate are not the right ones, should be the ones from g-p-m
- Icon for reboot should be xfsm-reboot from xfce-session I guess
- The Ubuntu artwork should be replaced with Fedora or removed
- Change the colors to something more Fedora-like?

I will look into these this weekend.

BTW: The package does not build in F-12 due to missing BRs. I guess this a problem with pkg-config evaluating Requires.private in a pkgconfig file that is missing a requirement, but I haven't looked what is actually missing.

Comment 15 Didier 2009-07-29 10:22:14 UTC
FYI : version 2.0.0 was released last week (July, 22nd).

Comment 16 Peter Robinson 2010-01-25 21:32:59 UTC
Is there still issue in this, or should the review be closed?

Comment 17 Christoph Wickert 2010-01-25 21:50:27 UTC
I still like to see this in Fedora, so I suggest to update to the latest release and then I will review it.

Comment 18 Peter Robinson 2010-01-25 22:36:58 UTC
Michael: please update the package for review

Comment 19 Juan Manuel Rodriguez 2010-05-26 22:54:22 UTC
Hey Michel, if you're no longer interested, would you mind if I took the package instead? 

I'll take radio silence by Friday, May 28 as a 'yes' :)

Comment 20 Michel Alexandre Salim 2010-05-27 12:17:47 UTC
Certainly, do go ahead. I ended up using gnome-shell as my netbook's primary interface, and there were several time-consuming changes that still needed to be made to netbook-launcher at the time (user interface icons, etc.) that someone who uses it more should probably take over.

Apologies for the delay!

Comment 21 Christoph Wickert 2010-07-16 18:39:12 UTC
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews I'm now closing this bug. 

@Juan, if you are interested in this package, please open a new review request. Close this one a duplicate of the new one and CC me when you do so.

Note You need to log in before you can comment on or make changes to this bug.