Skip to content

fix source being ignored when using pipewire#557

Merged
karlstav merged 1 commit into
karlstav:masterfrom
gardk:f-pipewire-source
Mar 29, 2024
Merged

fix source being ignored when using pipewire#557
karlstav merged 1 commit into
karlstav:masterfrom
gardk:f-pipewire-source

Conversation

@gardk

@gardk gardk commented Mar 29, 2024

Copy link
Copy Markdown
Contributor

I found that whatever I put in the source field of [input] was always ignored by cava when using PipeWire.

This PR fixes that issue, but it also brings out another issue that is actually reproducible without this patch. Described in #556.

Haven't been able to track down why that's happening.

@karlstav karlstav merged commit dc80096 into karlstav:master Mar 29, 2024
@karlstav

Copy link
Copy Markdown
Owner

hi @gardk,

I always found these settings a bit flimsy in pipewire. In my case now the source is ignored both with and without your commit. Maybe it is related to the fact that I am on ubuntu and stuck with pipewire 0.3.79

@karlstav karlstav mentioned this pull request Mar 29, 2024
@gardk gardk deleted the f-pipewire-source branch March 29, 2024 23:18
@karlstav

Copy link
Copy Markdown
Owner

there it worked, not sure why. maybe my pipewire was in some faulty state,

@gardk

gardk commented Apr 1, 2024

Copy link
Copy Markdown
Contributor Author

Nice, so you got it tested that it did what it says it did? I was a bit worried when you merged and it wasn't working for you :P

@karlstav

karlstav commented Apr 1, 2024

Copy link
Copy Markdown
Owner

yes, I can now specify a client like spotify and pause the music to reproduce the issue. What is strange is that if i specify some random string as the source it will still use the default output. Might just be a feature in pipewire.

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.

2 participants