[MPD] CoverGloobus doesnt refresh music name and cover when music changes

Bug #605532 reported by Guilherme de Sousa
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
CoverGloobus
Fix Committed
Undecided
Unassigned

Bug Description

Setup: OS: ArchLinux x86_64
            Player: MPD + Sonata(GUI)

CoverGloobus doesn't refresh when song changes.
It only works correctly when it makes the first 'read' from the player(when covergloobus is opened), or if I change to another player in the preferences, save, then go back to MPD again in the preferences.

I'm running the latest version: CoverGloobus 1.6 RevNo: 253

best regards

EDIT2: I've noticed now that it doesnt refresh it this ocasion:
1.A music that has album cover is playing
2.I change to a music that doesnt have album cover
3. it doesnt change the name to the new music and hangs there.

if a music with album cover is playing and I change to another music that also has album cover, CoverGloobus works normally changing the music name and the cover.

regards

-----edit1 (offtopic - not a bug, just a recommendation)

it would be great that covergloobus dint need a composite manager running for it to show up.
since the most common use is stick in the desktop without 'on top', it could use fake transparency that would make the same effect.

description: updated
description: updated
Revision history for this message
Alexandr Grigorcea (cahr-gr) wrote :

works fine for me
could you please run
$ covergloobus --debug
from console and post here errors that will appear (if will appear)
thanks

Revision history for this message
Guilherme de Sousa (guisacouto) wrote :

Cant enter in debug mode:

__________________________________________________________
guilherme:~$ covergloobus
** Message: pygobject_register_sinkfunc is deprecated (GtkWindow)
** Message: pygobject_register_sinkfunc is deprecated (GtkInvisible)
** Message: pygobject_register_sinkfunc is deprecated (GtkObject)
[INFO] UI: Theme: iSticky
[INFO] Song Changed:
guilherme:~$ covergloobus --debug
** Message: pygobject_register_sinkfunc is deprecated (GtkWindow)
** Message: pygobject_register_sinkfunc is deprecated (GtkInvisible)
** Message: pygobject_register_sinkfunc is deprecated (GtkObject)
[DEBUG] Installed themes:
45Controls , BadChoice , BadChoice2 , BoxOfTricks , BoxOfTricksMod , corner , Coversutra , Dirty , Ibex , iSticky , LucidDark , Polaroid , Postcard , Postcard2 , simple , ToolTip , Vinyl ,
[DEBUG] Installed players:
['Amarok2', 'Audacious2', 'Banshee', 'Clementine', 'DeaDBeeF', 'Exaile', 'Exaile3', 'Guayadeque', 'Listen', 'MOC', 'MPD', 'QuodLibet', 'Rhythmbox', 'Songbird', 'Spotify', 'Totem', 'VLC', 'gmusicbrowser']
[DEBUG] Installed lyrics search engines:
Traceback (most recent call last):
  File "/usr/share/covergloobus/covergloobus.py", line 42, in <module>
    import docky
  File "/usr/share/covergloobus/docky.py", line 25, in <module>
    import common
  File "/usr/share/covergloobus/common.py", line 309, in <module>
    print get_lyric_engines()
NameError: name 'get_lyric_engines' is not defined
guilherme:~$

__________________________________________________________

running without --debug works great, but with debug it doesnt

regards

Revision history for this message
Alexandr Grigorcea (cahr-gr) wrote : Re: [Bug 605532] Re: [MPD] CoverGloobus doesnt refresh music name and cover when music changes
Download full text (3.2 KiB)

any strange messages when running cg from console without --debug?

On Wed, Jul 14, 2010 at 11:16 PM, Guilherme de Sousa <<email address hidden>
> wrote:

> Cant enter in debug mode:
>
> __________________________________________________________
> guilherme:~$ covergloobus
> ** Message: pygobject_register_sinkfunc is deprecated (GtkWindow)
> ** Message: pygobject_register_sinkfunc is deprecated (GtkInvisible)
> ** Message: pygobject_register_sinkfunc is deprecated (GtkObject)
> [INFO] UI: Theme: iSticky
> [INFO] Song Changed:
> guilherme:~$ covergloobus --debug
> ** Message: pygobject_register_sinkfunc is deprecated (GtkWindow)
> ** Message: pygobject_register_sinkfunc is deprecated (GtkInvisible)
> ** Message: pygobject_register_sinkfunc is deprecated (GtkObject)
> [DEBUG] Installed themes:
> 45Controls , BadChoice , BadChoice2 , BoxOfTricks , BoxOfTricksMod , corner
> , Coversutra , Dirty , Ibex , iSticky , LucidDark , Polaroid , Postcard ,
> Postcard2 , simple , ToolTip , Vinyl ,
> [DEBUG] Installed players:
> ['Amarok2', 'Audacious2', 'Banshee', 'Clementine', 'DeaDBeeF', 'Exaile',
> 'Exaile3', 'Guayadeque', 'Listen', 'MOC', 'MPD', 'QuodLibet', 'Rhythmbox',
> 'Songbird', 'Spotify', 'Totem', 'VLC', 'gmusicbrowser']
> [DEBUG] Installed lyrics search engines:
> Traceback (most recent call last):
> File "/usr/share/covergloobus/covergloobus.py", line 42, in <module>
> import docky
> File "/usr/share/covergloobus/docky.py", line 25, in <module>
> import common
> File "/usr/share/covergloobus/common.py", line 309, in <module>
> print get_lyric_engines()
> NameError: name 'get_lyric_engines' is not defined
> guilherme:~$
>
> __________________________________________________________
>
> running without --debug works great, but with debug it doesnt
>
> regards
>
> --
> [MPD] CoverGloobus doesnt refresh music name and cover when music changes
> https://bugs.launchpad.net/bugs/605532
> You received this bug notification because you are a member of Gloobus
> Developers, which is the registrant for CoverGloobus.
>
> Status in CoverGloobus: New
>
> Bug description:
> Setup: OS: ArchLinux x86_64
> Player: MPD + Sonata(GUI)
>
> CoverGloobus doesn't refresh when song changes.
> It only works correctly when it makes the first 'read' from the player(when
> covergloobus is opened), or if I change to another player in the
> preferences, save, then go back to MPD again in the preferences.
>
> I'm running the latest version: CoverGloobus 1.6 RevNo: 253
>
> best regards
>
> EDIT2: I've noticed now that it doesnt refresh it this ocasion:
> 1.A music that has album cover is playing
> 2.I change to a music that doesnt have album cover
> 3. it doesnt change the name to the new music and hangs there.
>
> if a music with album cover is playing and I change to another music that
> also has album cover, CoverGloobus works normally changing the music name
> and the cover.
>
> regards
>
> -----edit1 (offtopic - not a bug, just a recommendation)
>
> it would be great that covergloobus dint need a composite manager running
> for it to show up.
> since the most common use is stick in the desktop without 'on top', it
> could use fake transparency that...

Read more...

Revision history for this message
Guilherme de Sousa (guisacouto) wrote :

The first changes of music were between musics that already had a cover.
Then I start to change to some musics that dont have, and that error appears, making CoverGloobus to get stuck in the last music that had cover.

best regards
______________________________________________________________
guilherme:~$ covergloobus
** Message: pygobject_register_sinkfunc is deprecated (GtkWindow)
** Message: pygobject_register_sinkfunc is deprecated (GtkInvisible)
** Message: pygobject_register_sinkfunc is deprecated (GtkObject)
[INFO] UI: Theme: iSticky
[INFO] Song Changed:
[INFO] Player Active
[INFO] Song Changed: LIFE IS REAL (DJEFF AFROZILA REMIX)
[INFO] Song Changed: Serenity (Lutzenkirchen Remix) By PsyMarck
[INFO] Song Changed: Head Cleaner by PsyMarck
Lastfm
[INFO] Song Changed: Black Jesus__By PsyMarck
[WARNING] Unable to download cover from Last.fm
[WARNING] Cover not found on Amazon
Lastfm
[INFO] Song Changed: Mamoth (Popof & Noiak Sinn Remix) By PsyMarck
[WARNING] Unable to download cover from Last.fm
[WARNING] Cover not found on Amazon
[INFO] Song Changed: Serenity (Noob Remix) by PsyMarck
[WARNING] MPD: title not found in current track
[WARNING] MPD: album not found in current track
[WARNING] MPD: artist not found in current track
[WARNING] MPD: genre not found in current track
[WARNING] MPD: date not found in current track
[WARNING] MPD: track not found in current track
Traceback (most recent call last):
  File "/usr/share/covergloobus/players/MPD.py", line 78, in check_new_song
    self.state_handler(state=self.get_state(), track=True)
  File "/usr/share/covergloobus/covergloobus.py", line 430, in state_handler
    self.check_song()
  File "/usr/share/covergloobus/covergloobus.py", line 392, in check_song
    self.change_song()
  File "/usr/share/covergloobus/covergloobus.py", line 361, in change_song
    cover = self.covers.search(self.track)
  File "/usr/share/covergloobus/coversearch.py", line 78, in search
    cover = self.location_search(track)
  File "/usr/share/covergloobus/coversearch.py", line 110, in location_search
    reg = re.compile(pattern, re.I)
  File "/usr/lib/python2.6/re.py", line 190, in compile
    return _compile(pattern, flags)
  File "/usr/lib/python2.6/re.py", line 245, in _compile
    raise error, v # invalid expression
sre_constants.error: unbalanced parenthesis
______________________________________________________________

Revision history for this message
Guilherme de Sousa (guisacouto) wrote :

found the bug :)

/usr/share/covergloobus/coversearch.py
line 109:
____________________________________________________
pattern = '^(folder|cover|front|album|albumart\.(jpg|jpeg|png|gif)$'
____________________________________________________

next to albumart there is a ) missing so it should be:

pattern = '^(folder|cover|front|album|albumart)\.(jpg|jpeg|png|gif)$'

now it works gr8!

many thanks!

best regards

Revision history for this message
Guilherme de Sousa (guisacouto) wrote :

I was observing CoverGloobus output, and I saw this in continuous loop:

[WARNING] MPD: track not found in current track

CoverGloobus doesnt stop printing this, when it doesn't found the cover I guess.. shouldn't it just like try to find it, and if it couldn't just skip it?! If for example the music is paused it will continue to print this to eternity...

regards

Revision history for this message
Alexandr Grigorcea (cahr-gr) wrote :

ah, bad pattern, thanks for your help
-------------------
"track not found in current track" message means that there is no track-number tag in the file you are listening
I just have to rewrite MPD plugin a little bit, to print this message only once when song actually changes

Revision history for this message
Guilherme de Sousa (guisacouto) wrote :

tks! you can mark it solved then!

keep the good work!

best regards!

Changed in covergloobus:
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.