Merge lp://qastaging/~scopes-hackers/unity-lens-music/rb-scope into lp://qastaging/unity-lens-music

There are still a couple of rough edges (better album handling, monitoring changes to the xml file), but I think it's good to go in for beta2.

This looks pretty ok, but there's a commented out FIXME where one would launch an album, so it appears that playing an album hasn't been implemented. Did I miss something?

> This looks pretty ok, but there's a commented out FIXME where one would launch
> an album, so it appears that playing an album hasn't been implemented. Did I
> miss something?

Currently albums aren't really handled - they behave same way like the song they are "extracted" from.

For the record, Rhythmbox doesn't provide any way to launch albums.
One solution would be, on activation of an album result, to :

rhythmbox-client --play <track1 of album>
rhythmbox-client --enqueue <track2> <track3> <track4> ...

Or, to do what the python rbox scope does :

rhythmbox-client --play <track1 of album>
and "# pray for rhythmbox to be ordered by tracks"

the first solution is what's done in banshee, and what i would suggest doing.

well, not exactly what's done in banshee but that's what i would do

I took the lens for a test spin but I am not getting any results from it. Looking at the code I think everything looks alright. I am not sure what gives (and I don't get any errors on stdout).

Nice to see that all the bells and whistles of Dee is being put to use! :-D

Mikkel, are you running fixed Dee (the non-C-locale issue)? If not run the lens with LC_ALL=C ./...

Thanks Michal, that makes it work. I think that we can bump the build dep, now that Dee has been post-release-version-bumped.

Search works nicely, but it seems that the Genre filter doesn't work?

Works well for me, genres included.

Ok, on a clean jhbuild wiht everything up-to-date this is working fine. Running well here.

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

The Jenkins job https://jenkins.qa.ubuntu.com/job/automerge-unity-lens-music/10/console reported an error when processing this lp:~scopes-hackers/unity-lens-music/rb-scope branch.
Not merging it.

New build-dep added.

The Jenkins job https://jenkins.qa.ubuntu.com/job/automerge-unity-lens-music/11/console reported an error when processing this lp:~scopes-hackers/unity-lens-music/rb-scope branch.
Not merging it.