Page MenuHomeFreeBSD

audio/alsa-plugins: convert options to subpackages
Needs ReviewPublic

Authored by jbeich on Jan 30 2024, 4:26 PM.
Tags
None
Referenced Files
F105569202: D43667.diff
Tue, Dec 17, 6:46 PM
Unknown Object (File)
Tue, Dec 3, 12:19 AM
Unknown Object (File)
Tue, Nov 26, 4:36 PM
Unknown Object (File)
Oct 24 2024, 8:17 AM
Unknown Object (File)
Oct 2 2024, 5:22 AM
Unknown Object (File)
Sep 30 2024, 11:34 PM
Unknown Object (File)
Sep 25 2024, 11:36 PM
Unknown Object (File)
Sep 25 2024, 4:28 AM
Subscribers

Details

Summary

Avoid heavy pulseaudio dependency by default. Trivial case: no USE* and no subpackage consumers (everything wants only OSS plugin).

Test Plan

Runtime tested inside jail via aplay.

$ poudriere testport -j 132amd64 audio/alsa-plugins
[...]
=>> Checking shared library dependencies
 0x0000000000000001 NEEDED               Shared library: [libasound.so.2]
 0x0000000000000001 NEEDED               Shared library: [libc.so.7]
 0x0000000000000001 NEEDED               Shared library: [libthr.so.3]

$ poudriere testport -j 132amd64 audio/alsa-plugins~ffmpeg
[...]
=>> Checking shared library dependencies
 0x0000000000000001 NEEDED               Shared library: [libasound.so.2]
 0x0000000000000001 NEEDED               Shared library: [libavcodec.so.60]
 0x0000000000000001 NEEDED               Shared library: [libavutil.so.58]
 0x0000000000000001 NEEDED               Shared library: [libc.so.7]
 0x0000000000000001 NEEDED               Shared library: [libswresample.so.4]

$ poudriere testport -j 132amd64 audio/alsa-plugins~jack
[...]
=>> Checking shared library dependencies
 0x0000000000000001 NEEDED               Shared library: [libasound.so.2]
 0x0000000000000001 NEEDED               Shared library: [libc.so.7]
 0x0000000000000001 NEEDED               Shared library: [libjack.so.0]
 0x0000000000000001 NEEDED               Shared library: [libthr.so.3]

$ poudriere testport -j 132amd64 audio/alsa-plugins~pulseaudio
[...]
=>> Checking shared library dependencies
 0x0000000000000001 NEEDED               Shared library: [libasound.so.2]
 0x0000000000000001 NEEDED               Shared library: [libc.so.7]
 0x0000000000000001 NEEDED               Shared library: [libpulse.so.0]
 0x0000000000000001 NEEDED               Shared library: [libthr.so.3]

$ poudriere testport -j 132amd64 audio/alsa-plugins~samplerate
[...]
=>> Checking shared library dependencies
 0x0000000000000001 NEEDED               Shared library: [libasound.so.2]
 0x0000000000000001 NEEDED               Shared library: [libc.so.7]
 0x0000000000000001 NEEDED               Shared library: [libsamplerate.so.0]

$ poudriere testport -j 132amd64 audio/alsa-plugins~speex
[...]
=>> Checking shared library dependencies
 0x0000000000000001 NEEDED               Shared library: [libasound.so.2]
 0x0000000000000001 NEEDED               Shared library: [libc.so.7]
 0x0000000000000001 NEEDED               Shared library: [libspeexdsp.so.1]

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

jbeich created this revision.
jbeich edited the test plan for this revision. (Show Details)
  • Rebase after c402d06aa792
  • Add SELF_DEPENDS.<subpkg>=main due to libasound.so.2 (see NEEDED)
  • Add pkg-descr.<subpkg> as suggested in D43576
  • Revert SELF_DEPENDS as libasound.so is part of alsa-lib, not alsa-plugins
  • Move one more file to jack subpackage (missed during 609ec25809c7 rebase)
  • Lowercase plugin names in pkg-descr.<subpkg>
  • Rebase after 14b194a6952f
  • Depend on alsa-lib in subpackages
  • Tested alsa-plugins-jack runtime via pcm.!default pcm.plug:jack in asound.conf
mat added a subscriber: mat.

This seems ok, if you have actually tested things still work after the conversion, go ahead.

This revision was not accepted when it landed; it landed in state Needs Review.Feb 2 2024, 1:46 PM
This revision was automatically updated to reflect the committed changes.