Re: Review request: QBluez

2015-02-20 Thread Albert Astals Cid
El Dissabte, 21 de febrer de 2015, a les 00:24:16, David Rosca va escriure: > > Still failing here > > Oops, sorry. It's because the problem is not that you are running > Bluez 4, but because the method call is rejected (auth issue?). > > Anyway, I modified the tests again so that it now checks w

Re: Review request: QBluez

2015-02-20 Thread Albert Astals Cid
El Dimecres, 18 de febrer de 2015, a les 11:50:01, David Rosca va escriure: > > If it's an expected situation, handle the situation. > > I have modified the tests (only the ones who would fail) so they will > be skipped if Bluez 4 is detected. Still failing here 4: Test command: /home/kdeunstabl

Re: plasma-mediacenter in kdereview from extragear

2015-02-20 Thread Jonathan Riddell
I took at look at this today and discussed at the Plasma sprint.. - possible legal issues with youtube icon, now fixed - mockcpp pointer to obscure repository cmake message - qtmultimedia needs a "this is needed at runtime" cmake message - music and video icon are black on black - crashes, at tim

plasma-mediacenter in kdereview from extragear

2015-02-20 Thread Bhushan Shah
Hello, We want to move plasma-mediacenter to kde/workspace for releasing it with Plasma 5.x release cycle, Plasma media center is living room solution of plasma workspaces, it is integrated as shell package for PMC. Current code depends upon KDE Frameworks 5 (free of kdelibs4support), kfilemetada

Re: plasma-mediacenter in kdereview from extragear

2015-02-20 Thread Bhushan Shah
On Thu, Feb 19, 2015 at 8:22 PM, Bhushan Shah wrote: > Looking forward to review from community Just for record : This mail is sent to both plasma-devel and kde-core-devel, reply to kde-core-devel -- Bhushan Shah http://bhush9.github.io IRC Nick : bshah on Freenode

Re: Review request: QBluez

2015-02-20 Thread David Rosca
> If it's an expected situation, handle the situation. I have modified the tests (only the ones who would fail) so they will be skipped if Bluez 4 is detected. I have also renamed the library to BluezQt. Here is an updated documentation: http://david.rosca.cz/bluezqt-apidocs/html/ David On Wed,