offmessage / pyscrobbler

Python bindings for the last.fm API

Home Page:http://github.com/offmessage/pyscrobbler

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Merging back upstream changes from Sonata project

multani opened this issue · comments

Hi,

Sonata is a Python/Gtk client to the Music Player Daemon (https://github.com/multani/sonata) which currently contains a copy of your audioscrobbler module, which has been imported circa February 2008 by the former maintainer of the project.

Since then, it has diverged a bit from your upstream version, and I would like to move those changes back from Sonata to here.

I had a quick look at the current differences, and this is what has been changed so far:

  • code cleanup (PEP8-ification, trim trailing whitespaces)
  • fix some encoding issues with non-utf8 locales
  • use a MD5 password instead of a plain text one (?)
  • store songs in a cache if they can't be send to the server (but I heard reports that it doesn't work in Sonata, though I'm not sure if the problem is in audioscrobbler.py or its usage by Sonata)
  • a Python's logging support instead of hand-made log
  • support for the 'now playing' API
  • and there's an ongoing work on supporting Python 3

And the module from Sonata misses, of course, some stuffs you added later.

if you are interested, I'm willing to open one or more pull requests on your repository so that you can merge those changes here, and we can drop the custom module from Sonata to use this one instead.

Please do open pull requests - I'd be very happy to include your code in pyscrobbler. As you've no doubt seen I've not had a lot of time to work on it (nor, to be honest, the motivation as my needs have long since passed) but I'm very happy to keep the library alive.

Send through the pull requests and I'll get them merged as soon as I get the chance