Bug 781649 - New upstream release
Summary: New upstream release
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: volumeicon
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
Assignee: Christoph Wickert
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-14 01:52 UTC by Raphael Groner
Modified: 2012-02-13 00:24 UTC (History)
1 user (show)

Fixed In Version: volumeicon-0.4.5-3.fc16
Clone Of:
Environment:
Last Closed: 2012-02-13 00:24:25 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
doubled notification about volume level (27.22 KB, image/png)
2012-01-15 18:35 UTC, Raphael Groner
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Xfce 8359 0 None None None Never

Description Raphael Groner 2012-01-14 01:52:04 UTC
Description of problem:
The version of volumeicon package is outdated. 
Danger of potential bugs, that are already fixed upstream. :)

Version-Release number of selected component (if applicable):
volumeicon.x86_64 0:0.4.1-3.fc16

How reproducible:
yes

Steps to Reproduce:
1. yum install volumeicon
2.
3.
  
Actual results:
Version 0.4.1-3

Expected results:
Version 0.4.5 or later.

Additional info:
http://softwarebakery.com/maato/volumeicon.html
 -> Changes

Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3663868

Comment 1 Christoph Wickert 2012-01-14 18:39:21 UTC
Thanks for reporting this issue. However I'm not yet sure if I'll push this into F16 (the version you reported this bug against) because it conflicts with our update guidelines (see https://fedoraproject.org/wiki/Updates_Policy) as there are slight changes in the user experience.

And please note that this request does not qualify as urgent because it doesn't meet any of the criteria outlined in https://bugzilla.redhat.com/page.cgi?id=fields.html#bug_severity.

Comment 2 Raphael Groner 2012-01-14 18:53:50 UTC
D'accord with rawhide only. I am locally using a fc16 scratch build, done by myself.

Comment 3 Christoph Wickert 2012-01-14 19:50:41 UTC
I'd like to update, but it doesn't build on rawhide, see
http://koji.fedoraproject.org/koji/taskinfo?taskID=3682328

Comment 4 Raphael Groner 2012-01-14 20:40:40 UTC
What about the older build?
https://koji.fedoraproject.org/koji/taskinfo?taskID=3682103

Something has to be changed, either in your configuration or at koji. Maybe report a bug against mockbuild?

Comment 5 Raphael Groner 2012-01-15 11:13:11 UTC
Build of old version 0.4.1 fails currently also in f17 due to mass rebuild.

http://koji.fedoraproject.org/koji/buildinfo?buildID=291758

Comment 6 Raphael Groner 2012-01-15 12:20:36 UTC
There are two notifications about volume level when I put the mouse pointer over the systray icon and scroll with the wheel. It's only one popup notification when the slider is used to adjust volume level.

Comment 7 Christoph Wickert 2012-01-15 18:12:06 UTC
I have no idea what knd of notifications you are talking about. Tooltips? Notifications through libnotify? If so, what notification daemon are you using?

Comment 8 Raphael Groner 2012-01-15 18:35:39 UTC
Created attachment 555369 [details]
doubled notification about volume level

Name       : xfce4-notifyd
Architektur : x86_64
Version    : 0.2.2
Ausgabe    : 1.fc16
Größe : 266 k
Repo        : installed
Aus repo    : fedora

Comment 9 Christoph Wickert 2012-01-15 20:13:57 UTC
And what version of volumeicon are you using? 0.4.5-1.fc16 I built yesterday? Can you show me your ~/.config/volumeicon/volumeicon? Do you have anything else running, e.g. xfce4-volumed?

Comment 10 Raphael Groner 2012-01-15 20:30:53 UTC
> And what version of volumeicon are you using? 0.4.5-1.fc16 I built yesterday?

Yeah. Your spec and patch files from git.

$ rpm -q volumeicon
volumeicon-0.4.5-1.fc16.x86_64

> Can you show me your ~/.config/volumeicon/volumeicon?

$ cat .config/volumeicon/volumeicon 

[StatusIcon]
stepsize=5
lmb_slider=true
mmb_mute=false
use_horizontal_slider=true
show_sound_level=true
onclick=xfce4-mixer
theme=tango

[Hotkeys]
up_enabled=false
down_enabled=false
mute_enabled=false
up=XF86_AudioRaiseVolume
down=XF86_AudioLowerVolume
mute=XF86AudioMute

[Alsa]
card=hw:0

> Do you have anything else running, e.g. xfce4-volumed?

$ ps aux |grep volumed
raphael   1647  0.0  0.4 856464 10028 ?        Ssl  14:11   0:04 xfce4-volumed
raphael  12331  0.0  0.0 111300   904 pts/0    S+   21:29   0:00 grep --color=auto volumed

Comment 11 Christoph Wickert 2012-01-16 18:19:21 UTC
I guess one notification is from xfce4-volumed and the other from volumeicon then. Please don't run both if you don't want to have two notifications.

Comment 12 Raphael Groner 2012-01-17 17:18:29 UTC
> Please don't run both if you don't want to have two notifications.

Without volumed running. The keyboard shortcuts do not work (and no notification), but there's only one notification popup for volumeicon. On the other hand, I don't get a notification any more for xfce4-mixer panel plugin. 

How can I get all to work properly with only one notification popup for each of the following possibilities?
a. shortcut keys
b. volumeicon wheel scrolling
c. xfce4-mixer panel plugin wheel scrolling
It isn't IMHO necessary to show the notification popup when adjustments to volume level will be done by slinder of either volumeicon or xfce4-mixer dialog.

Possible solution: Maybe disable libnotify in volumeicon when volumed is running? But I am not sure how that could be achieved. Is there a configuration setting somewhere to modify?

Comment 13 Christoph Wickert 2012-01-17 18:05:54 UTC
(In reply to comment #12)
> > Please don't run both if you don't want to have two notifications.
> 
> Without volumed running. The keyboard shortcuts do not work 

They should be working because we already handle them through xfwm4, xfce4-volumed is only required for the notifications.

> How can I get all to work properly with only one notification popup for each of
> the following possibilities?
> a. shortcut keys

Check that everything is configured correctly in xfce4-keyboard-settings.

> b. volumeicon wheel scrolling
> c. xfce4-mixer panel plugin wheel scrolling

Both already support scrolling, but I wonder why you need two mixer icons in the panel.

> It isn't IMHO necessary to show the notification popup when adjustments to
> volume level will be done by slinder of either volumeicon or xfce4-mixer
> dialog.

While I agree this makes some sense, this is a feature request that should be discussed with the upstream developers. The problem however remains: If you use two different icons, how should one know the volume was changed through the slider or the mousewheel of the *other* icon?

> Possible solution: Maybe disable libnotify in volumeicon when volumed is
> running? But I am not sure how that could be achieved. Is there a configuration
> setting somewhere to modify?

No, it's a compile time option and it doesn't involve any checks if any arbitrary other programs are running, too.

Comment 14 Raphael Groner 2012-01-17 18:55:36 UTC
(In reply to comment #13)
> > Without volumed running. The keyboard shortcuts do not work 
>
> They should be working because we already handle them through xfwm4,

Well, I forgot to enable the three checkboxes in volumeicon's configuration. With those settings, volumeicon can do the shortcut handling, too. :)

> xfce4-volumed is only required for the notifications.

A little bit too much overload for only a popup window, eh? Please see my (off-topic) omment below.

> Check that everything is configured correctly in xfce4-keyboard-settings.

Not needed (any more), if volumeicon is used with enabled shortcuts. At least on my box, it works.

> The problem however remains: If you use
> two different icons, how should one know the volume was changed through the
> slider or the mousewheel of the *other* icon?

AFAIK This is the job of xfce4-volumed to detect general changes. You won't need it if you have only one slider control with own libnotify support.

>> But I am not sure how that could be achieved. Is there a configuration
>> setting somewhere to modify?
>
> No, it's a compile time option and it doesn't involve any checks if any
> arbitrary other programs are running, too.

I agree, this should go to upstream as a feature request for a run-time option to disable libnotify temporarily.
Also, xfce4-mixer should integrate libnotify support instead of relaying on xfce4-volumed. (off-topic)

My decision is to go with volumeicon and get rid of xfce4-volumed and xfce4-mixer panel plugin. Thanks.

Comment 15 Raphael Groner 2012-01-18 19:10:05 UTC
(In reply to comment #14)
> Also, xfce4-mixer should integrate libnotify support instead of relaying on
> xfce4-volumed. (off-topic)

Reported to upstream as a feature request.

ttps://bugzilla.xfce.org/show_bug.cgi?id=8359

Comment 16 Raphael Groner 2012-01-18 19:10:27 UTC
(In reply to comment #14)
> Also, xfce4-mixer should integrate libnotify support instead of relaying on
> xfce4-volumed. (off-topic)

Reported to upstream as a feature request.

https://bugzilla.xfce.org/show_bug.cgi?id=8359

Comment 17 Fedora Update System 2012-01-28 13:22:40 UTC
volumeicon-0.4.5-3.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/volumeicon-0.4.5-3.fc16

Comment 18 Fedora Update System 2012-01-28 20:57:57 UTC
Package volumeicon-0.4.5-3.fc16:
* should fix your issue,
* was pushed to the Fedora 16 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing volumeicon-0.4.5-3.fc16'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-1017/volumeicon-0.4.5-3.fc16
then log in and leave karma (feedback).

Comment 19 Fedora Update System 2012-02-12 22:47:49 UTC
volumeicon-0.4.5-3.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 20 Raphael Groner 2012-02-12 23:35:40 UTC
What about upstream version 0.4.6? ;)

""
0.4.6	- Changed the default card that is used from "hw:0" to "default" as pointed out by Christoph Wickert.
""

Comment 21 Christoph Wickert 2012-02-13 00:24:25 UTC
Raphael, please THINK and LOOK before reopening this bug.

0.4.6 is already in rawhide and.
0.4.6 is just 0.4.5 plus the patch I wrote and that is included in Fedora's 0.4.5 already.


Note You need to log in before you can comment on or make changes to this bug.