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

Arch: pulseaudio (17.0+r43+g3e2bb8a1e-1) breaks dependency 'pulseaudio<=17.0' required by qubes-vm-pulseaudio #9660

Open
colons opened this issue Dec 19, 2024 · 4 comments
Labels
affects-4.2 This issue affects Qubes OS 4.2. affects-4.3 This issue affects Qubes OS 4.3. C: Arch Linux The Arch Linux template C: audio needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.

Comments

@colons
Copy link

colons commented Dec 19, 2024

Qubes OS release

4.2

Brief summary

like #8875, but with a newer pulseaudio release

Steps to reproduce

try to update an arch template

Expected behavior

the updates get applied

Actual behavior

Updating [template name]
Refreshing package info
:: Synchronizing package databases...
 qubes-r4.2-current-testing downloading...
 qubes-r4.2-current downloading...
 core downloading...
 extra downloading...
 multilib downloading...
:: Starting full system upgrade...
resolving dependencies...
looking for conflicting packages...
:: installing pulseaudio (17.0+r43+g3e2bb8a1e-1) breaks dependency 'pulseaudio<=17.0' required by qubes-vm-pulseaudio
error: failed to prepare transaction (could not satisfy dependencies)
Installed packages: None
Updated packages: None
@colons colons added P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. labels Dec 19, 2024
@alimirjamali
Copy link

Also affects r4.3

Unlike #8875, we do not have a major version change this time

@marmarek
Copy link
Member

What is 17.0+r43+g3e2bb8a1e-1? I mean, I see what that is, but why is it a released as a package into Arch? Are there no more normal Pulseaudio releases?

Anyway, what matters is whether module-related headers have changed since 17.0. If not, simply bumping the dependency should be enough. But if they did, we need to import them into pulse/pulsecore-17.0+... dir in gui-agent-linux, and make sure Makefile chooses the right one.

All that said, isn't Arch using Pipewire already? Maybe qubes-vm-pulseaudio shouldn't be needed at all anymore?

@andrewdavidwong andrewdavidwong added C: Arch Linux The Arch Linux template needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. C: audio affects-4.2 This issue affects Qubes OS 4.2. affects-4.3 This issue affects Qubes OS 4.3. labels Dec 20, 2024
@k4z4n0v4
Copy link

While there is a pipewire-qubes package for Archlinux, it is set to replace qubes-vm-pulseaudio<=4.1.26-10 so installing it now will not result in removing qubes-pulse integration in Arch. I think we can just fix the dependencies and push people to stop using pulse on Arch alltogether, if the pipewire integration works.

@lubellier
Copy link

What is 17.0+r43+g3e2bb8a1e-1? I mean, I see what that is, but why is it a released as a package into Arch? Are there no more normal Pulseaudio releases?

The ArchLinux package does NOT change the PulseAudio API, it takes the PulseAudio code from the 17.0 tag up to the last commit in the master branch. PulseAudio didn't push a new release (since 17.0).
The ArchLinux package includes also the 0001-alsa-ucm-Fix-segfault-from-recursion-due-to-too-many.patch patch.

The PulseAudio 17.0..master change didn't contain a header change, same for the 0001-alsa-ucm-Fix-segfault-from-recursion-due-to-too-many.patch patch.

@marmarek : So from my Point Of View, the QubesOS required update is only a fix of the 17.x dependency pattern.

Information sources:
The ArchLinux package change between 17.0-3 and 17.0+r43+g3e2bb8a1e-1 .
The Pulseaudio change between 17.0 tag and the commit included in the ArchLinux Package

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-4.2 This issue affects Qubes OS 4.2. affects-4.3 This issue affects Qubes OS 4.3. C: Arch Linux The Arch Linux template C: audio needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.
Projects
None yet
Development

No branches or pull requests

6 participants