Skip to content

pulseaudio: add pkgconfig build dep#45653

Merged
wdconinc merged 1 commit intospack:developfrom
teaguesterling:packages/pulseaudio
Aug 10, 2024
Merged

pulseaudio: add pkgconfig build dep#45653
wdconinc merged 1 commit intospack:developfrom
teaguesterling:packages/pulseaudio

Conversation

@teaguesterling
Copy link
Copy Markdown
Contributor

Pulseaudio fails to build in a bare spack/ubuntu-jammy container due to pkgconfig not being present.

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Aug 9, 2024

Hi @teaguesterling! I noticed that the following package(s) don't yet have maintainers:

  • pulseaudio

Are you interested in adopting any of these package(s)? If so, simply add the following to the package class:

    maintainers("teaguesterling")

If not, could you contact the developers of this package and see if they are interested? You can quickly see who has worked on a package with spack blame:

$ spack blame pulseaudio

Thank you for your help! Please don't add maintainers without their consent.

You don't have to be a Spack expert or package developer in order to be a "maintainer," it just gives us a list of users willing to review PRs or debug issues relating to this package. A package can have multiple maintainers; just add a list of GitHub handles of anyone who wants to volunteer.

@teaguesterling
Copy link
Copy Markdown
Contributor Author

Noticed this while attempting to build #45083 in a bare spack/ubuntu-jammy container.

Copy link
Copy Markdown
Contributor

@wdconinc wdconinc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed, ref https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/blob/v13.0/configure.ac?ref_type=tags#L100.

(Out of scope) It appears that pulseaudio is now at v17 (with only meson as build system, not sure since when). I'm not sure how many users or depdendents pulseaudio has in spack, but this may be useful to consider adding newer versions for.

@wdconinc wdconinc merged commit 2dbc521 into spack:develop Aug 10, 2024
diehlpk pushed a commit to diehlpk/spack that referenced this pull request Aug 14, 2024
Chrismarsh pushed a commit to Chrismarsh/spack that referenced this pull request Aug 16, 2024
tldahlgren pushed a commit to AcriusWinter/spack that referenced this pull request Aug 20, 2024
FrederickDeny pushed a commit to FrederickDeny/spack that referenced this pull request Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants