[NowPlaying] NowPlayingScreenlet.py crashed with error in connect()

[Jon Arnold]

Binary package hint: screenlets

The NowPlaying Screenlet seems to crash pretty much all the time for me. The screenlet itself stays up but the apport message keeps popping up. There must be some time based event that is occurring because it keeps happening.

Please let me know if there is a way I can test this more productively.

ProblemType: Crash
Architecture: i386
DistroRelease: Ubuntu 8.10
ExecutablePath: /usr/share/screenlets/NowPlaying/NowPlayingScreenlet.py
InterpreterPath: /usr/bin/python2.5
Package: screenlets 0.1.2-1
PackageArchitecture: all
ProcAttrCurrent: unconfined
ProcCmdline: python -u /usr/share/screenlets/NowPlaying/NowPlayingScreenlet.py
ProcEnviron:
 PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
 LANG=en_US.UTF-8
 SHELL=/bin/bash
PythonArgs: ['/usr/share/screenlets/NowPlaying/NowPlayingScreenlet.py']
SourcePackage: screenlets
Title: NowPlayingScreenlet.py crashed with error in connect()
Uname: Linux 2.6.27-2-generic i686
UserGroups: adm admin cdrom dialout fuse lpadmin plugdev sambashare

[Kjell Braden (afflux)]

Thanks for your report. I've forwarded it to the developers of the software.

[Twig (pfirth)]

This is also happening to me with Jaunty and all updates as of 04/04/09. However, I believe the version of python is newer (2.6 rather than 2.5)

[TheDebugger (hubert-vansteenhuyse)]

I modified the screenlet,
the problem was that out of an array of players there was not tested if the player was active before making a connection.

The screenlet is available here:
http://www.geocities.com/hvansteenhuyse/index.html

[Julien Lavergne (gilir)]

Thanks for your work on it. Could you please provide a patch instead of
the complete file ? It will be easier to deal with it. You can obtain a
patch with this command : diff -Nur old_directory/ new_directory/ >
the_patch.patch

[Julien Lavergne (gilir)]

I included the patch in a package of my PPA. Can someone confirm that the bug disappear with the screenlets 0.1.2-6~ppa3 package of my PPA (https://launchpad.net/~gilir/+archive/unstable) ?

[Twig (pfirth)]

I've added your PPA and upgraded. Unfortunately, I can't now get the nowplaying screenlet to launch despite trying a few things:

Rebooted
Switched compiz off / on
reset the config for the nowplaying screenlet

I've also tried altering the view to show running screenlets and the nowplaying screenlet is never listed as running.

I'm using rhythmbox as a music player. Let me know if you need any other details.

[Julien Lavergne (gilir)]

Thanks for the test. Could you please launch in a terminal this : python /usr/share/screenlets/NowPlaying/NowPlayingScreenlet.py and paste the output of the terminal ?

[Twig (pfirth)]

Aha!

twig@twig-desktop:~$ python /usr/share/screenlets/NowPlaying/NowPlayingScreenlet.py
  File "/usr/share/screenlets/NowPlaying/NowPlayingScreenlet.py", line 324
    continue
SyntaxError: 'continue' not properly in loop

[Julien Lavergne (gilir)]

I update the package in my PPA, could you please test with the new version (screenlets_0.1.2-6~ppa5). I attache the 2 modified patches.

[Twig (pfirth)]

Works perfectly now for me. Thanks for all your work.

[Launchpad Janitor (janitor)]

This bug was fixed in the package screenlets - 0.1.2-6

---------------
screenlets (0.1.2-6) unstable; urgency=low

  [ Julien Lavergne ]
  * debian/rules: Replace simple-patchsys by quilt.
  * debian/control:
   - Build-depends on quilt.
   - Remove X{B,S}-Python-Version.
   - Bump Standards-Version to 3.8.2 (no change needed).
  * debian/pyversions : Add compatible versions.
  * debian/patches:
   - 13-opacity-option.patch: Disable the opacity option if the screen
     is not composited (Closes: #518179).
   - 14-configuration-save.patch: From Ubuntu, save and restore the
     configuration file in some cases. (LP: #345359)
   - 15-nowplaying-mpd-try.patch: Try to connect to mpd instead of forcing
     the connection. (LP: #269256)
   - 16-clearweather-catch-connection-error.patch: From Ubuntu, catch
     connection error on update (thanks Kjell Braden for the patch).
     (LP: #264809)
   - 17-nowplaying-disable-amarok.patch: Disable Amarok 1.X support.
   - 18-nowplaying-rb-playing.patch: Avoid Dbus exception when checking
     if the player is playing.
   - 19-remove-autostart-output.patch: Remove output when screenlets are
     autostarted. (LP: #318248)
   - Update patches to apply with quilt.
  * debian/README.source: Replace by the quilt one.

  [ Siegfried-Angel Gevatter Pujals ]
  * debian/screenlets.install:
   - Install the available translations (usr/share/locale) (LP: #386878).

screenlets (0.1.2-5) unstable; urgency=medium

  * Urgency to medium to fix RC bug.
  * debian/patches: Update description for all patches.
  * debian/patches/12-new-class-creation.patch:
   - Use new-style class creation. This should avoid some segfaults
   (Closes: #519664).
  * debian/control:
   - Remove python-dcop from suggests.
   - Bump the need of a compositor to recommends, because it's needed to deal
     with transparency.
   - Following python-gnome2-desktop split.
    + Remove depends on python-gnome2-desktop
    + Add python-rsvg as depends.
    + Add python-wnck as depends, needed by screenlets-manager.
    + Add python-evolution as recommends, only needed by the
      EvolutionContacts Screenlets.
    + Add python-gnomekeyring as recommends, as it's an optional feature.
   - Bump Standards-Version to 3.8.1 (no change needed).

screenlets (0.1.2-4) unstable; urgency=low

  [ Julien Lavergne ]
  * debian/10-ImageMenuItem-update.patch
   - Replace get_children() by get_child() and get_image() following pygtk
     modifications.
  * debian/11-xfce-wallpaper-setting.patch
   - Add a xfconf-query call to update desktop wallpaper (Closes: #497222).
  * debian/control
   - Suggest xfconf for WallpaperClock under XFCE, require for
     11-xfce-wallpaper-setting.patch.
   - Switch from python-central to python-support.
   - Set XS-Python-Version to >= 2.4
   - Remove duplicated section in binary.
  * debian/copyright
   - Replace (C) by ©.
  * debian/pycompat: Remove.
  * debian/rules: Switch from pycentral to pysupport.

  [ Sandro Tosi ]
  * debian/control
    - switch Vcs-Browser field to viewsvn

 -- Julien Lavergne <email address hidden> Mon, 22 Jun 2009 15:57:41 +0100

[Julien Lavergne (gilir)]

Fix committed in revision 473 of screenlets.

[Julien Lavergne (gilir)]

and amarok patch on 475.