Page MenuHomeFreeBSD
Differential D50827

devel/aws-sdk-cpp: make PulseAudio dependency optional
Needs ReviewPublic
Actions

Authored by olgeni on Jun 13 2025, 6:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jul 21, 2:12 PM
Unknown Object (File)
Sun, Jul 19, 9:02 PM
Unknown Object (File)
Sat, Jul 18, 6:43 PM
Unknown Object (File)
Sat, Jul 18, 6:24 PM
Unknown Object (File)
Mon, Jul 13, 5:58 AM
Unknown Object (File)
Fri, Jul 10, 4:53 PM
Unknown Object (File)
Thu, Jul 2, 5:14 AM
Unknown Object (File)
Tue, Jun 30, 9:12 AM
View All 64 Files
Subscribers
patmaddox

Details

Reviewers
eduardo
Summary
  • Add PULSEAUDIO option (enabled by default) for text-to-speech audio output
  • Move libpulse.so dependency from mandatory to optional based on PULSEAUDIO option
  • Add patch to respect FORCE_NO_PULSE CMake flag to disable PulseAudio detection
  • Allows building without audio dependencies when text-to-speech audio output is not needed

I made this because I have server builds on poudriere with no pulseaudio and noticed it sneaking in :)

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 64861
Build 61744: arc lint + arc unit

Event Timeline

olgeni requested review of this revision.Jun 13 2025, 6:13 AM
olgeni created this revision.
Harbormaster completed remote builds in B64840: Diff 156955.Jun 13 2025, 6:13 AM
olgeni edited the summary of this revision. (Show Details)Jun 13 2025, 6:14 AM
eduardo added a comment.Jun 13 2025, 12:08 PM

Hello,

Is there an upstream PR open about this changes?

eduardo added a comment.Jun 13 2025, 8:25 PM

Also, there is a open bugzilla PR: 287370
Can you link summary to it?

Cheers

patmaddox added a subscriber: patmaddox.Jun 14 2025, 2:35 AM
olgeni updated this revision to Diff 157016.Jun 14 2025, 9:22 AM

Bit better variable name.

Harbormaster completed remote builds in B64861: Diff 157016.Jun 14 2025, 9:22 AM
olgeni added a comment.Jun 14 2025, 9:24 AM
In D50827#1160217, @eduardo wrote:

Hello,

Is there an upstream PR open about this changes?

Not yet, let me give it a try and see if it flies.

Revision Contents
Changeset List

PathSize
devel/
aws-sdk-cpp/
10 lines
files/
15 lines

Diff 157016

View Options

devel/aws-sdk-cpp/Makefile

Loading...
View Options

devel/aws-sdk-cpp/files/patch-src_aws-cpp-sdk-text-to-speech_CMakeLists.txt

Loading...