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

debian/rules: Use default portaudio sound #3380

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

leamas
Copy link
Collaborator

@leamas leamas commented Aug 3, 2023

Always use the default portaudio sound system, also on arm architectures. Makes for consistent sound usage, avoiding the problems related to using external programs to run sounds.

While on it, remove an unused if branch.

Closes: #3285

EDIT: Updated bug reference.

Alec Leamas added 2 commits August 3, 2023 21:09
Always use the default portaudio sound system, also on arm
architectures. Makes for consistent sound usage, avoiding the
problems related to using external programs to run sounds.

Closes: OpenCPN#3333
@leamas leamas changed the title Debian sound 3333 debian/rules: Use default portaudio sound Aug 3, 2023
@bdbcat
Copy link
Member

bdbcat commented Aug 3, 2023

I will need to test this on Ubuntu PPA build workflow for all linux platforms. Please stand by.

@leamas
Copy link
Collaborator Author

leamas commented Aug 3, 2023

Sure, no hurry.

@as365n4
Copy link

as365n4 commented Oct 8, 2023

I opened #3454 against this PR as it seems related to me.

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.

Linux PPA package: use of non-optimal SystemCmdSound
3 participants