search

fossfreedom / rhythmweb

rhythmbox v2.9x/v3.x GTK3 port of the original rhythmweb GTK2 plugin created previously for rhythmbox v0.1x
GNU General Public License v3.0
23 stars 8 forks source link

Plugin fails to load on fedora 15 with rhythmbox 2.90.1 #14

Closed fapestniegd closed 12 years ago

fapestniegd commented 12 years ago

Not sure what's going on...

Details follow:

(rhythmbox:11752): Rhythmbox-WARNING : Unable to grab media player keys: GDBus.Error:org.freedesktop.DBus.Error.UnknownMethod: No such interface `org.gnome.SettingsDaemon.MediaKeys' on object at path /org/gnome/SettingsDaemon/MediaKeys Traceback (most recent call last): File "/home/jameswhite/.local/share/rhythmbox/plugins/rhythmweb/rhythmweb.py", line 48, in class RhythmwebPlugin(GObject.GObject, Peas.Activatable): File "/home/jameswhite/.local/share/rhythmbox/plugins/rhythmweb/rhythmweb.py", line 50, in RhythmwebPlugin object = GObject.Property(type=GObject.GObject) File "/usr/lib/python2.7/site-packages/gi/module.py", line 234, in getattr attr = super(DynamicGObjectModule, self).getattr(name) File "/usr/lib/python2.7/site-packages/gi/module.py", line 101, in getattr self.name**, name)) AttributeError: 'gi.repository.GObject' object has no attribute 'Property'

(rhythmbox:11752): libpeas-WARNING **: Error loading plugin 'rhythmweb'

fossfreedom commented 12 years ago

v2.90 of rhythmbox does not support the code syntax currently implemented in this version of the plugin.

This plugin only works with v2.96 and above - I think you'll find that in a newer version of fedora - or you might want to consider installing v2.96 or v2.97 from another repository.

fapestniegd commented 12 years ago

Thanks for the info. I got it working without the full upgrade (which I'll have to do later)

changing: object = GObject.Property(type=GObject.GObject) to object = GObject.property(type=GObject.GObject) in /home/jameswhite/.local/share/rhythmbox/plugins/rhythmweb/rhythmweb.py

was sufficient to get the module to load and control rhythmbox.

fossfreedom commented 12 years ago

Thanks for the info. Dont know why the rhythmbox devs made this "subtle" change.

If you have any other issues or suggestions, just create an issue. Thanks

fossfreedom commented 12 years ago

@fapestniegd - ok - the .property syntax (lowercase) seems to also work in 2.96 as well - so I've committed that change

thanks again for the report.