New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add native support for pulseaudio. #283

Merged
merged 2 commits into from Jun 24, 2016

Conversation

Projects
None yet
3 participants
@Lompik
Contributor

Lompik commented Jun 14, 2016

This adds ability to get Pulseaudio current volume without calling an
external program. Also it provides a few other parameters like card's
current profile similarly to pavucontrol. Currently it only queries the
default sink (and associated card).

Add native support for pulseaudio.
This adds ability to get Pulseaudio current volume without calling an
external program. Also it provides a few other parameters like card's
current profile similarly to pavucontrol. Currently it only queries the
default sink (and associated card).
@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Jun 14, 2016

Thank you for making this, it was on my "wanted feature" list.
(not tested)

edit: don't forget to update print_version

ghost commented Jun 14, 2016

Thank you for making this, it was on my "wanted feature" list.
(not tested)

edit: don't forget to update print_version

Switch to pulse async API.
This us by far the most efficient as conky subscribe to a few pulseaudio
notifications. Whenever pulseaudio server changes (volume or default
sink or default card), conky's callback updates the results.
@brndnmtthws

This comment has been minimized.

Show comment
Hide comment
@brndnmtthws

brndnmtthws Jun 24, 2016

Owner

Looks great, thanks!

Owner

brndnmtthws commented Jun 24, 2016

Looks great, thanks!

@brndnmtthws brndnmtthws merged commit 587631a into brndnmtthws:master Jun 24, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@TELunus

This comment has been minimized.

Show comment
Hide comment
@TELunus

TELunus Feb 2, 2017

Can we get some documentation on how to use this? I notice that https://github.com/brndnmtthws/conky/wiki/Configuration-Settings and https://github.com/brndnmtthws/conky/wiki/Configuration-Variables don't mention pulse audio, and most information on the web still suggests Conky doesn't play nice with pulse audio.

TELunus commented Feb 2, 2017

Can we get some documentation on how to use this? I notice that https://github.com/brndnmtthws/conky/wiki/Configuration-Settings and https://github.com/brndnmtthws/conky/wiki/Configuration-Variables don't mention pulse audio, and most information on the web still suggests Conky doesn't play nice with pulse audio.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment