Skip to content
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

Use pactl only for PipeWire compatibility #21

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Klok-e
Copy link

@Klok-e Klok-e commented May 9, 2021

I recently moved to pipewire and this script stopped working, so I fixed it.

I didn't test this thoroughly on pulseaudio, so it needs some testing before it can be merged.

@pzmarzly
Copy link
Owner

pzmarzly commented May 9, 2021

Thanks a lot for the code. I'm positively surprised that PipeWire's PulseAudio compatibility layer is good enough to make the script work without many changes. I'll test the code a bit next week, if that's okay for you.

Out of curiosity, does everything work on PipeWire? Is it worthwhile to switch?

@Klok-e
Copy link
Author

Klok-e commented May 9, 2021

Yeah, I'm OK with that.

Everything works so far, and it doesn't have problems I encountered on pulseaudio, like some sinks not having audio recorded by OBS, so I had to hack up an intermediate remap sink. Overall, I'm happy with the switch.

@Klok-e
Copy link
Author

Klok-e commented Aug 10, 2021

@pzmarzly Any updates?

@siddhpant
Copy link

Please have a look at https://github.com/siddhpant/broo , I added PipeWire support along with other features.

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.

None yet

3 participants