Skip to content

feat: added runtime option for pulse backend#299

Merged
devgianlu merged 2 commits intodevgianlu:masterfrom
pyrorhythm:feature/pulse-runtime
Apr 6, 2026
Merged

feat: added runtime option for pulse backend#299
devgianlu merged 2 commits intodevgianlu:masterfrom
pyrorhythm:feature/pulse-runtime

Conversation

@pyrorhythm
Copy link
Copy Markdown
Contributor

playing with library right now and found myself in quite a situation
when trying to use pulseaudio on macos, runtime socket path is inferred from hostname and is incorrect
so i found it kind of reasonable to add a way to override ClientServerString as this option exists by itself in pulse package

@dubeyKartikay
Copy link
Copy Markdown
Contributor

@pyrorhythm I believe the project uses audio-toolbox for macOS.
pulse, as far as I know, is for linux systems.
we should test it in a linux box to make sure this does not introduce breaking changes

Comment thread output/output.go
Comment thread player/player.go
@devgianlu devgianlu merged commit 1c0adc5 into devgianlu:master Apr 6, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants