date:20180626

Re: Pulseaudio module migration gconf => gsettings

2018-06-26 Thread Martin Steigerwald

Hi David.

Many thanks for your prompt response.

David Rosca - 26.06.18, 08:23:
> On Mon, Jun 25, 2018 at 11:01 PM Martin Steigerwald 
 wrote:
> > Hi!
> > 
> > The unfixed bug
> > 
> > Bug 386665 - Drop dependency on pulseaudio-gconf
> > 
> > makes plasma-pa uninstallable in the development versions of Debian
> > and Arch.
> > 
> > We would really appreciate an upstream fix for that one. The comment
> > by Luigi points out commits for paprefs gconf => gsettings
> > migration. Maybe something similar would work with Plasma PA?
> > 
> > https://bugs.kde.org/show_bug.cgi?id=386665#c13
> > 
> > Would someone help David Rosca (in Cc) in case he is unavailable or
> > has no time to attend to the issue?
> 
> We are aware of the issue and are planning to port to gsettings.
> Actually, it may be added as alternative to gconf as otherwise it will
> break the functionality for distros that doesn't have PulseAudio 12
> yet. For now, I'll make the gconf dependency optional.
> 
> Also, PulseAudio 12 was released less then one week ago, and you
> immediately dropped gconf package, would be cool if you gived us a
> little more time.

Pulseaudio package maintainer did so. I bet Qt/KDE team would also liked 
to have more time.

On the other hand the deprecation of gconf module in Pulseaudio is quite 
old I believe.

What is the functionality that will break without the gconf or gsettings 
module? I think building without gconf may be good enough as an interim 
solution if not too much stuff breaks for our Debian Unstable / Testing 
users.

Thansk,
-- 
Martin

Re: Pulseaudio module migration gconf => gsettings

2018-06-26 Thread Maximiliano Curia

¡Hola David!

El 2018-06-26 a las 08:23 +0200, David Rosca escribió:

Hi,

On Mon, Jun 25, 2018 at 11:01 PM Martin Steigerwald wrote:

Hi!

The unfixed bug

Bug 386665 - Drop dependency on pulseaudio-gconf

makes plasma-pa uninstallable in the development versions of Debian and
Arch.

We would really appreciate an upstream fix for that one. The comment by
Luigi points out commits for paprefs gconf => gsettings migration. Maybe
something similar would work with Plasma PA?

https://bugs.kde.org/show_bug.cgi?id=386665#c13

Would someone help David Rosca (in Cc) in case he is unavailable or has
no time to attend to the issue?

We are aware of the issue and are planning to port to gsettings.
Actually, it may be added as alternative to gconf as otherwise it will
break the functionality for distros that doesn't have PulseAudio 12
yet. For now, I'll make the gconf dependency optional.

You might want to use the frugalware patch [1] as a base for this. This patch
is mentioned in the kde bug (although it wasn't linked), I was considering
using something like that, before plasma-pa gets removed from Debian testing
(if it comes to that).

Happy hacking,

[1]:
https://github.com/frugalware/kde5/blob/master/source/plasma/plasma-pa/plasma-pa-no-GCONF-we-DONT-need-thx.patch
--
"Brilliant opportunities are cleverly disguised as insolvable problems."
-- Gardener's Philosophy

"The reverse is also true." -- Corollary
Saludos /\/\ /\ >< `/

signature.asc
Description: PGP signature

Re: Pulseaudio module migration gconf => gsettings

2018-06-26 Thread David Rosca

On Tue, Jun 26, 2018 at 10:23 AM Martin Steigerwald  wrote:
>
> Hi David.
>
> Many thanks for your prompt response.

Here's the patch: https://phabricator.kde.org/D13734

>
> David Rosca - 26.06.18, 08:23:
> > On Mon, Jun 25, 2018 at 11:01 PM Martin Steigerwald
>  wrote:
> > > Hi!
> > >
> > > The unfixed bug
> > >
> > > Bug 386665 - Drop dependency on pulseaudio-gconf
> > >
> > > makes plasma-pa uninstallable in the development versions of Debian
> > > and Arch.
> > >
> > > We would really appreciate an upstream fix for that one. The comment
> > > by Luigi points out commits for paprefs gconf => gsettings
> > > migration. Maybe something similar would work with Plasma PA?
> > >
> > > https://bugs.kde.org/show_bug.cgi?id=386665#c13
> > >
> > > Would someone help David Rosca (in Cc) in case he is unavailable or
> > > has no time to attend to the issue?
> >
> > We are aware of the issue and are planning to port to gsettings.
> > Actually, it may be added as alternative to gconf as otherwise it will
> > break the functionality for distros that doesn't have PulseAudio 12
> > yet. For now, I'll make the gconf dependency optional.
> >
> > Also, PulseAudio 12 was released less then one week ago, and you
> > immediately dropped gconf package, would be cool if you gived us a
> > little more time.
>
> Pulseaudio package maintainer did so. I bet Qt/KDE team would also liked
> to have more time.

Fair enough.

>
> On the other hand the deprecation of gconf module in Pulseaudio is quite
> old I believe.

That's true, but up until PulseAudio 12 release, there were no alternative.

>
> What is the functionality that will break without the gconf or gsettings
> module? I think building without gconf may be good enough as an interim
> solution if not too much stuff breaks for our Debian Unstable / Testing
> users.

Two checkboxes under Advanced tab in KCM:

* Add virtual output device for simultaneous output on all local sound cards
* Automatically switch all running streams when a new output becomes available

Those will be now hidden when building without GConf.

>
> Thansk,
> --
> Martin
>
>

David

Re: Pulseaudio module migration gconf => gsettings

2018-06-26 Thread Christophe Giboudeaux

Hi,

On mardi 26 juin 2018 10:23:19 CEST Martin Steigerwald wrote:
> Hi David.
> 

> What is the functionality that will break without the gconf or gsettings
> module? I think building without gconf may be good enough as an interim
> solution if not too much stuff breaks for our Debian Unstable / Testing
> users.
> 

Virtual devices. The openSUSE maintainers for pulseaudio tried removing the 
gconf plugin and it made my virtual card to have sound both on my speakers & 
USB card disappear.

Bug#886048: Bug#902181: Bug#886048: plasma-pa: Depends on gconf

2018-06-26 Thread Martin Steigerwald

Martin Steigerwald - 25.06.18, 17:39:
> Felipe Sateler - 25.06.18, 16:41:
> > > With Debian's pulseaudio 12.0 upload, pulseaudio has removed
> > > pulseaudio- module-gconf in favour of pulseaudio-module-gsettings,
> > > which would break current plasma-pa.
> > 
> > Indeed. While pulseaudio can build a gconf or a gsettings backend,
> > building both is not advisable and would lead to confusion, as
> > changes would not be synchronized between the modules. Therefore I
> > have removed the gconf module.
> > 
> > The only reverse dependencies are plasma-pa and paprefs. paprefs is
> > already fixed upstream and I'm only waiting on a release to upload
> > to debian. That would leave plasma-pa to be fixed.
> 
> Please also see upstream bug report:
> 
> [plasma-pa] [Bug 386665] Drop dependency on pulseaudio-gconf?
> https://bugs.kde.org/386665
> 
> I hope that upstream provides a fix soon.

A port before Plasma 5.14 is unlikely:

https://bugs.kde.org/show_bug.cgi?id=386665#c17

So seems to me that building without gconf module support is the best  
option for now. See:

https://bugs.kde.org/show_bug.cgi?id=386665#c16

=> 

https://build.opensuse.org/package/view_file/
home:cfeck:branches:KDE:Unstable:Frameworks/plasma5-pa/plasma-pa.diff?
expand=1

I still have no clear idea at what functionality will break by doing so. 
Asked so on kde-distro-packagers list. I now also asked on upstream bug 
report again:

https://bugs.kde.org/show_bug.cgi?id=386665#c18

Thanks,
-- 
Martin

Bug#886048: Bug#902181: Bug#886048: plasma-pa: Depends on gconf

2018-06-26 Thread Martin Steigerwald

Martin Steigerwald - 26.06.18, 12:04:
> I still have no clear idea at what functionality will break by doing
> so. Asked so on kde-distro-packagers list. I now also asked on
> upstream bug report again:
> 
> https://bugs.kde.org/show_bug.cgi?id=386665#c18

Here is the functionality that will break:

From David Rosca:
> Two checkboxes under Advanced tab in KCM:
> 
> * Add virtual output device for simultaneous output on all local sound
> cards 
> * Automatically switch all running streams when a new output
> becomes available

Those will be now hidden when building without GConf.

From Christophe Giboudeaux:
> Virtual devices. The openSUSE maintainers for pulseaudio tried
> removing the gconf plugin and it made my virtual card to have sound
> both on my speakers & USB card disappear.

To me it seems like it is dropping gconf support for *now* and readding 
gsettings support back in with Plasma 5.14. There may be some users who 
use virtual devices, but I bet most users won´t. And those may still be 
able to use paprefs as a interim solution.

Upstream patch for dropping gconf support, by David Rosca:

https://phabricator.kde.org/D13734

Thanks,
-- 
Martin

1 2 >

1 - 100 of 107 matches

Mail list logo