After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 685893 - rhythmbox lyrics plugin: some service no longer work
rhythmbox lyrics plugin: some service no longer work
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: Plugins (other)
unspecified
Other Linux
: Normal normal
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-10-10 14:52 UTC by Leo Iannacone
Modified: 2012-10-14 06:55 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Disable some lyrics service (1.57 KB, patch)
2012-10-10 14:53 UTC, Leo Iannacone
none Details | Review

Description Leo Iannacone 2012-10-10 14:52:57 UTC
Some service provided with lyrics plugin no longer work:

 * If service Astraweb is enabled in Lyric plugin you will get this error:

Traceback (most recent call last):
  • File "/usr/lib/rhythmbox/plugins/rb/Loader.py", line 47 in _contents_cb
    (ok, contents, etag) = file.load_contents_finish(result)
  • File "/usr/lib/python2.7/dist-packages/gi/types.py", line 47 in function
    return info.invoke(*args, **kwargs)
gi._glib.GError: Cannot connect to destination (search.lyrics.astraweb.com)



 * If you enable winampnc.com in lyrics plugin, you will never get info from winampnc.com

It seems service is down.

This is a simple search string that should give "Metallica - One" lyric built by plugin:
http://www.winampcn.com/lyrictransfer/get.aspx?song=one&artist=metallica&lsong=one&Datetime=20060601




I would suggest to remove them from plugin installation folder and configuration.


See attachment.


NOTE: I'm writing two new parsers replacing them.
Comment 1 Leo Iannacone 2012-10-10 14:53:19 UTC
Created attachment 226179 [details] [review]
Disable some lyrics service
Comment 2 Leo Iannacone 2012-10-10 14:54:57 UTC
The patch also updates TerraParser link (and id) according with the new URL.
Comment 3 Jonathan Matthew 2012-10-14 06:55:40 UTC
I've fixed the winampcn.co parser in commit e403650. Commit 4865e34 disables the astraweb parser. The changes to the Terra parser should go with your other patch relating to that code.