Bug 13805 - K4Guitune: wrong entry in desktop file
Summary: K4Guitune: wrong entry in desktop file
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 5
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard: MGA4TOO advisory MGA5-64-OK MGA4-32-OK
Keywords: validated_update
Depends on:
Blocks:
 
Reported: 2014-07-26 14:26 CEST by Marc Lattemann
Modified: 2015-09-17 20:03 CEST (History)
5 users (show)

See Also:
Source RPM: k4guitune-1.1.1-4.mga4.src.rpm
CVE:
Status comment:


Attachments

Description Marc Lattemann 2014-07-26 14:26:44 CEST
K4guitune does not start via the KDE menu: you will get error 'Unable to open audio device /dev/dsp'

K4guitar needs to be run with the aoss wrapper: 'aoss K4guitune' from konsole works fine.

this is due to a wrong entry in in the .desktop file (/usr/share/applications/kde4/k4guitune.desktop) which needs to be change to 
'Exec=aoss k4guitune'

Please refer to GtkGuitune desktop entry which starts without problems from the KDE-menu


Steps to Reproduce:
1. Install K4guitune
2. Try to start K4guitune > fails with error message 'Unable to open audio device /dev/dsp'
3. Update .desktop file to Exec=aoss k4guitune
4. K4guitune starts from menu


Reproducible: 

Steps to Reproduce:
Jani Välimaa 2014-07-26 16:47:11 CEST

Keywords: (none) => Junior_job

Comment 1 Florian Hubold 2014-07-26 17:26:15 CEST
@Colin:
Are there any differences to /usr/bin/aoss and /usr/bin/padsp and any preference? Probably padsp should be used ...

CC: (none) => doktor5000, mageia

Comment 2 Colin Guthrie 2014-07-26 18:33:57 CEST
@Florian. Actually neither should be used directly in .desktop files. We generally prefer to use "soundwrapper" which will run the correct program for you.

In actual fact, these days, osspd is preferred over any kind of LD_PRELOAD hacks. osspd uses cuse (character device fuse). Basically the userspace component (rather than kernel) will crate the /dev/dsp (and friends) character devices and a daemon listens on it and will proxy the data appropriately to either PulseAudio or Alsa. This approach has several advantages over the previous systems so for legacy sound I'd always recommend that.

If osspd is used, you don't actually have to do anything to the .desktop file, but you should probably still use soundwrapper as it can know whether or not osspd is in use and simple be a no-op if it is or defer to padsp or aoss as appropriate otherwise.
Comment 3 Yann Cantin 2015-08-04 21:15:37 CEST
Updated .desktop file to Exec=soundwrapper k4guitune

- k4guitune-1.1.1-6.2.mga5
- k4guitune-1.1.1-8.mga6

Please test.

CC: (none) => yann.cantin

Comment 4 Rémi Verschelde 2015-08-05 09:45:10 CEST
Thanks Yann. When pushing an update candidate, please assign the bug report to the QA team (qa-bugs@ml.mageia.org) and provide an advisory [1] that describes what the update does in plain English (intended for end users).

[1] https://wiki.mageia.org/en/Update_Advisory_Announcement_Example
Comment 5 Yann Cantin 2015-08-06 00:24:42 CEST
Uploaded patched package for Mageia 5 and Cauldron.

The application should start when launched via menu now, please test.

Advisory:
========================

Use soundwrapper in k4guitune desktop file.

========================

Updated packages in core/updates_testing:
========================
k4guitune-1.1.1-6.2.mga5
k4guitune-1.1.1-8.mga6

From source RPMs:
k4guitune-1.1.1-6.2.mga5.src.rpm
k4guitune-1.1.1-8.mga6.src.rpm

Assignee: bugsquad => qa-bugs
Whiteboard: (none) => advisory

Comment 6 Marc Lattemann 2015-08-06 23:36:15 CEST
tested in MGA5 42Bit VBox: installation works: soundwrapper will be installed as dependency and K4guitune starts now from menu. 
It does not not get any sound input, however it seems that it is due to the virtual machine, since pavucontrol shows no input sound as well. Will try tomorrow on real hw.
Comment 7 Lewis Smith 2015-08-10 21:12:17 CEST
This bug is listed for Mageia 4 rather than Mageia 5. Can this be corrected? (I prefer not to do it since it is not 'my' bug; and I may be wrong).

@Marc
There has been a lot of talk about sound not working with some Mageia host/guest combinations under VirtualBox.

Version: 4 => 3
CC: (none) => lewyssmith

Lewis Smith 2015-08-10 21:12:39 CEST

Version: 3 => 4

Comment 8 James Kerr 2015-08-10 21:39:10 CEST
I've changed the version to 5 and added MGA4TOO to the whiteboard since there is no indication that the bug has been fixed in mga4.

I've also removed the advisory keyword since I don't think the advisory has been uploaded yet to SVN.

Version: 4 => 5
Hardware: x86_64 => All
Whiteboard: advisory => MGA4TOO

Comment 9 Yann Cantin 2015-08-11 13:28:25 CEST
Uploaded patched package for Mageia 4, Mageia 5 and Cauldron.

The application should start when launched via menu now, please test.

New advisory:
========================

Use soundwrapper in k4guitune desktop file.

========================

Updated packages in core/updates_testing:
========================
k4guitune-1.1.1-4.1.mga4
k4guitune-1.1.1-6.2.mga5
k4guitune-1.1.1-8.mga6

From source RPMs:
k4guitune-1.1.1-4.1.mga4.src.rpm
k4guitune-1.1.1-6.2.mga5.src.rpm
k4guitune-1.1.1-8.mga6.src.rpm
Comment 10 Marc Lattemann 2015-08-11 18:38:07 CEST
Tested it on real hw MGA5 64bit and have only partially success. 
First I checked the KDE menu and the entry in exec is: soundwrapper k4guitune %i -caption "%c" 

It starts from the menu, but does not get any input from microphone by default. When trying to change the Sample frequency at Configuration --> Audio Device the program crashes with error message from #1
- starting it from Konsole --> same error message as above and it does not start.
- starting it with 'aoss k4guitune' from konsole everything works. 

to be honest: I'm not sure if it is worth fiddling around, since I (and probably no one else?) do not use this prog. I only stumbled upon it by chance last year...
But if you want to, I'm always happy to test it :)
Comment 11 Samuel Verschelde 2015-08-21 11:09:13 CEST
Adding feedback whiteboard marker, because of mixed testing results.

Whiteboard: MGA4TOO => MGA4TOO feedback

Comment 12 Yann Cantin 2015-08-25 01:57:35 CEST
mga5 64 LANG=fr_FR.UTF-8
VT8237A/VT8251 HDA Controller

Installed :
soundwrapper-1.7-5.mga5
pulseaudio-5.0-6.mga5

k4guitune-1.1.1-6.2.mga5.x86_64.rpm

No mic attached.

Starts from the menu or console : not getting input at first.
Settings --> Configure k4guitune --> Audio Device --> change Sample frequency :
got some signal (obviously electronic noise).

Update OK for me.

Whiteboard: MGA4TOO feedback => MGA4TOO feedback MGA5-64-OK

Comment 13 David Walser 2015-09-15 19:35:50 CEST
Why does this have the feedback marker?  I see an OK for Mageia 5 and nothing that indicates that something is different on Mageia 4.
Comment 14 David Walser 2015-09-16 18:12:33 CEST
OK, we discussed on IRC and the Comment 10 issue indicates that it isn't working with Pulseaudio.  Perhaps it could work with it if you fiddled with some settings.  Disabling Pulseaudio or running osspd should make this work fine with soundwrapper.  Disabling Pulseaudio will cause soundwrapper to call aoss, and osspd will have it just run it directly which should work too.  This is something we can cycle back to and see how we want to handle things going forward for Mageia 6.  Feel free to file a bug if you deem it necessary.  I've confirmed that the soundwrapper was also correctly added to the menu entry for Mageia 4, so validating this now.

New advisory:
========================

The k4guitine application uses the OSS sound API, therefore the menu entry to
launch the program needed to use soundwrapper, which it previously did not.
This has been corrected in this update.

Keywords: Junior_job => validated_update
Whiteboard: MGA4TOO feedback MGA5-64-OK => MGA4TOO MGA5-64-OK MGA4-32-OK
CC: (none) => sysadmin-bugs

Comment 15 claire robinson 2015-09-17 16:55:53 CEST
Advisory uploaded.

Whiteboard: MGA4TOO MGA5-64-OK MGA4-32-OK => MGA4TOO advisory MGA5-64-OK MGA4-32-OK

Comment 16 Mageia Robot 2015-09-17 20:03:26 CEST
An update for this issue has been pushed to Mageia Updates repository.

http://advisories.mageia.org/MGAA-2015-0126.html

Status: NEW => RESOLVED
Resolution: (none) => FIXED


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