search

fossfreedom / coverart-browser

Browse your cover-art albums in Rhythmbox v2.96 - 3.0+
http://xpressubuntu.wordpress.com/
GNU General Public License v3.0
74 stars 19 forks source link

No module named 'coverart_extdb' #234

Closed jrbastien closed 10 years ago

jrbastien commented 10 years ago

Hi Fossfreedom,

I hadn't check on this project for a while but did update my RB 3.0 installation today with the latest master. Prior to the update it was working fine, now I got an error when loading the plugin.

jrbastien@jrbastien-VirtualBox:~$ rhythmbox

(rhythmbox:3322): Gtk-CRITICAL **: gtk_css_provider_load_from_path: assertion 'path != NULL' failed

(rhythmbox:3322): GLib-GObject-CRITICAL **: Custom constructor for class SoupServer returned NULL (which is invalid). Unable to remove object from construction_objects list, so memory was probably just leaked. Please use GInitable instead. Unable to open ~/.mtpz-data for reading, MTPZ disabled.Traceback (most recent call last): File "/home/jrbastien/.local/share/rhythmbox/plugins/coverart_browser/coverart_browser.py", line 36, in from coverart_browser_source import CoverArtBrowserSource File "/home/jrbastien/.local/share/rhythmbox/plugins/coverart_browser/coverart_browser_source.py", line 44, in from coverart_artistview import ArtistView File "/home/jrbastien/.local/share/rhythmbox/plugins/coverart_browser/coverart_artistview.py", line 42, in from coverart_extdb import CoverArtExtDB ImportError: No module named 'coverart_extdb'

(rhythmbox:3322): libpeas-WARNING **: Error loading plugin 'coverart_browser'

Did I miss an installation instruction?

fossfreedom commented 10 years ago

Hi @jrbastien

seems like a long time. Hope yourself/family are looking forward to a great xmas :)

as explained here: https://github.com/fossfreedom/coverart-browser/issues/209

I have now made this a hard-dependency on the latest version of the coverart-search-providers plugin. I can't really workaround this. Please install the latest version of the coverart-search-providers plugin ... I'm ironing a few bugs out at the moment - so the version you need is on the artist branch

i.e.

git clone https://github.com/fossfreedom/coverart-search-providers -b artist
cd coverart-search-providers
./install.sh -rb3

You'll also note in the forthcoming readme file for coverart-search-providers - it requires one more package (I'm assuming you are using RB3.0.1) -

sudo apt-get install python3-gdbm

If you are extracting embedded coverart from your music files - then as explained here you need the mutagenx package

I've this morning uploaded this into my PPA - you can install the deb on Saucy (13.10) if that is the distro you are using:

If you want to keep up-to-date with the mutagenx package you can use this directly from my PPA on saucy by adding the PPA and changing the sources to trusty

jrbastien commented 10 years ago

@fossfreedom, I'm wishing a merry Christmas to you too. Enjoy the time off with your family.

Thanks for the lengthily explanation. At first sight, I can't find the problem. Everything seems to be installed as described but I will have some free time during the holiday season and will report back on my findings and further experiments.

fossfreedom commented 10 years ago

The key part of the error message is this: "ImportError: No module named 'coverart_extdb'"

you should have this file: "/home/jrbastien/.local/share/rhythmbox/plugins/coverart_search_providers/coverart_extdb.py"

This file currently is found on the artist branch of coverart_search_providers - hence, please double check you have checked out that branch:

git clone https://github.com/fossfreedom/coverart-search-providers -b artist

As I said - I'm ironing out a few bugs - when done, I'll merge the artist branch with master and delete the artist branch.

fossfreedom commented 10 years ago

artist branch is no more ... the master branch on coverart-search-providers should be all that you need now.

N.B. mutagenx has been updated in my PPA - the latest version should be installed (or run update-manager if using the PPA)

jrbastien commented 10 years ago

OK, I just tested with the updated master branch and it fixed the 'coverart_extdb' missing issue. Unfortunately, I now got another error:

jrbastien@jrbastien-VirtualBox:~$ rhythmbox

(rhythmbox:6288): Gtk-CRITICAL **: gtk_css_provider_load_from_path: assertion 'path != NULL' failed

(rhythmbox:6288): GLib-GObject-CRITICAL **: Custom constructor for class SoupServer returned NULL (which is invalid). Unable to remove object from construction_objects list, so memory was probably just leaked. Please use GInitable instead. Unable to open ~/.mtpz-data for reading, MTPZ disabled.Traceback (most recent call last): File "/home/jrbastien/.local/share/rhythmbox/plugins/coverart_search_providers/coverart_search_providers.py", line 33, in from coverart_album_search import CoverAlbumSearch File "/home/jrbastien/.local/share/rhythmbox/plugins/coverart_search_providers/coverart_album_search.py", line 41, in from coverart_search_tracks import mutagen_library File "/home/jrbastien/.local/share/rhythmbox/plugins/coverart_search_providers/coverart_search_tracks.py", line 27, in from PIL import Image ImportError: No module named 'PIL'

(rhythmbox:6288): libpeas-WARNING **: Error loading plugin 'coverart_search_providers'

I'm on Saucy and I have installed RB 3.0 following the instructions here: http://www.webupd8.org/2013/10/install-rhythmbox-30-in-ubuntu-1310-or.html

I have used your latest deb package to install mutagenx. Perhaps, the problem is there as there was a crash reported in the sofware center even if it now shows "Installed."

Not too sure about installing your PPA. Will I be dependent on it to test any update?

fossfreedom commented 10 years ago

nice catch.

The package you require is called python3-imaging - I'll add that to the readme of coverart-search-providers immediately.

The mutagenx package is only needed if you have a need for the export & embed coverart in albums. If you have no need for this, not installing mutagenx will not be required.

When I release v1.0, I will be making mutagenx a dependency in the PPA package itself.

fossfreedom commented 10 years ago

I presume this has now been dealt with?

I've uploaded a beta version of coverart browser and coverart search providers into the Trusty PPA.

Additionally, I've uploaded the 14 new translation strings into launchpad -

I'll close this for now - shout loudly if this issue hasnt been resolved. Thanks.

jrbastien commented 10 years ago

I have been away longer than I thought. Thanks for updating the readme. It is now working. My goal is to provide with with some feedback on the artist view.

For the translation, I'm on Ubuntu 12.04. Is it still supported with the master branch?

fossfreedom commented 10 years ago

Hi @jrbastien

Hope you have not been working too hard ;)

The master branch should work for all versions from 12.04 through to 14.04 - there is one caveat as detailed in the README - for example, 12.04 can not support hovering (single click play).

I've installed the french language support on my 12.04 install and RB + the plugin correctly displays the fr_CA locale - so yes, the translation is still working. I've seen one immediate error in the terminal with fr_CA - I'll raise that now as an issue

I note you've done extensive testing in another issue - I'll drill down those as soon as and give you some feedback. Thanks.