Proposal: Let's see if we can programmatically fix volumeicon

johnraff · 2016-07-09 04:38:39

Head_on_a_Stick wrote:

the package has a different name from the Debian version but the binary is the same so the Debian volumeicon-alsa package must be removed before installing this version.

@HoaS You can force the uninstallation of volumeicon-alsa with these lines in debian/control:

Conflicts: volumeicon-alsa
Replaces: volumeicon-alsa

However...
I tested the new package, and results on this machine suggested there might still be a little scope for optimization:
*) Scroll to change volume does not work.
*) Volume keybinds do not work.
*) Volume notifications do not appear.

None of these issues exists with the vanilla Jessie version, making this fix perhaps the most attractive, followed by a backported Stretch pnmixer.

hhh · 2016-07-09 06:56:03

hhh wrote:

Rebooting into BL!

And promptly fell asleep. Tomorrow, fo' sho'!

Head_on_a_Stick · 2016-07-09 09:38:51

johnraff wrote:

@HoaS You can force the uninstallation of volumeicon-alsa with these lines in debian/control:
Conflicts: volumeicon-alsa
Replaces: volumeicon-alsa

Yes indeed, thanks

I was thinking that last night as I fell asleep, funnily enough

And yes, I think "pulseaudio --start && volumeicon &" should fix this for us anyway.

Head_on_a_Stick · 2016-07-09 09:40:39

johnraff wrote:

However...
I tested the new package, and results on this machine suggested there might still be a little scope for optimization:
*) Scroll to change volume does not work.
*) Volume keybinds do not work.
*) Volume notifications do not appear.

Gah! I forgot to compile it with "--enable-notify"

New version incoming...

Head_on_a_Stick · 2016-07-09 12:34:24

I have added "Conflicts" & "Replaces" and tried to add "--enable-notify" [1] but that doesn't seem to be working -- I'm not sure if this is because of a build error on my part (highly likely) or a separate bug.

The new package version should be in the repositories by now soon so just `sudo apt update && sudo apt upgrade` to get them or download the new .deb files directly

Apologies for the sloppy packaging, I haven't bothered bumping the version number or editing the changelog

[1] I added these lines to debian/rules:

override_dh_auto_configure:
        dh_auto_configure -- --enable-notify

EDIT: the build failed 'cos of that override so that can't be right

Investigating...

EDIT2: The packages now conflict with & replace volumeicon-alsa but notifications are not yet enabled.

If we decide to go with this modified package over the stock jessie version I will attempt to enable that option.

Last edited by Head_on_a_Stick (2016-07-09 13:24:02)

xaos52 · 2016-07-09 16:24:20

^ looks OK.

But will only work if that is a [tab] character in the second line, not leading spaces.

Head_on_a_Stick · 2016-07-09 16:34:16

^ Thanks!

The packages actually build fine locally with the configuration options being set manually before running `dh_make` -- the problem occurs when trying to build the packages in my OpenSUSE Build Service repository. I am using a shortcut method and just uploading the .dsc file but I think I have to upload the debian/foo files instead to customise the package.

I don't think we will be using this package in BL anyway I'm not going to bother unless somebody wants me to.

#21 2016-07-09 04:38:39

Re: Proposal: Let's see if we can programmatically fix volumeicon

#22 2016-07-09 06:56:03

Re: Proposal: Let's see if we can programmatically fix volumeicon

#23 2016-07-09 09:38:51

Re: Proposal: Let's see if we can programmatically fix volumeicon

#24 2016-07-09 09:40:39

Re: Proposal: Let's see if we can programmatically fix volumeicon

#25 2016-07-09 12:34:24

Re: Proposal: Let's see if we can programmatically fix volumeicon

#26 2016-07-09 16:24:20

Re: Proposal: Let's see if we can programmatically fix volumeicon

#27 2016-07-09 16:34:16

Re: Proposal: Let's see if we can programmatically fix volumeicon

Board footer