falconindy / ponymix

CLI volume control for PulseAudio
MIT License
177 stars 27 forks source link

latest master not working with ubuntu 14.10 #41

Closed simao closed 8 years ago

simao commented 8 years ago

The latest master does not work with ubuntu 14.10.

I get the following output:

~   ponymix set-volume 50
ponymix: no match found for device: s
 ~ 1   ponymix set-volume 50
ponymix: no match found for device: ³ð
 ~ 1   ponymix set-volume 50
ponymix: no match found for device: Þ
 ~ 1   ponymix set-volume 50
ponymix: no match found for device: V
 ~ 1   ponymix set-volume 50
ponymix: no match found for device: ã¥
 ~ 1   ponymix set-volume 50
ponymix: no match found for device: 3
 ~ 1   ponymix set-volume 50
ponymix: no match found for device: ã                         
 ~ 1   ponymix set-volume 50
ponymix: no match found for device: 3
 ~ 1   ponymix set-volume 50
ponymix: warning: ambiguous result for '', using 'alsa_output.pci-0000_00_03.0.hdmi-stereo'
50
 ~   ponymix set-volume 50
ponymix: warning: ambiguous result for '', using 'alsa_output.pci-0000_00_03.0.hdmi-stereo'
50
 ~   ponymix set-volume 50
ponymix: warning: ambiguous result for '', using 'alsa_output.pci-0000_00_03.0.hdmi-stereo'
50

But the sound is not set at all.

I reverted to the 4 tag in this repository and everything works again.

Thank you.

Simao

falconindy commented 8 years ago

Thanks for the report. Looks like 78ef2f963dc9b causes c_str() to be called on a temporary.

falconindy commented 8 years ago

39d61e9e59dc8 should fix. Could you please check?

simao commented 8 years ago

Perfect, works great. Thanks!

Jesin commented 8 years ago

I guess this can be closed, then?