-
Notifications
You must be signed in to change notification settings - Fork 33
Audio Support (v1) #51
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
base: main
Are you sure you want to change the base?
Conversation
Mesa DescriptionTL;DRAdds comprehensive audio support to the chromium-headful image for Docker and Unikraft deployments, enabling live view with audio and enhancing security by running audio services as a non-root user. Why we made these changesTo enable full audio functionality for the Chromium-headful environment in both Docker containers and Unikraft deployments, allowing for a complete 'live view with audio' experience. This also improves security by configuring audio services to run as a non-root user. What changed?Core Audio System & Security:
D-Bus Integration:
Input/Output & UX:
Deployment (Minor):
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Performed full review of fe02e69...84c77a5
9 files reviewed | 0 comments | Review on Mesa | Edit Reviewer Settings
1048c39 to
44c33b7
Compare
|
| exit 1 | ||
| fi | ||
| sleep 0.5 | ||
| done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: PulseAudio Readiness Check Fails Due to Missing pactl
The PulseAudio readiness check in wrapper.sh fails because pactl is not installed. The script attempts to use pactl info to verify PulseAudio startup, but pactl (provided by pulseaudio-utils) is missing, causing the script to exit and prevent container launch. Install pulseaudio-utils or use an alternative check.
|
Heya @raiden-staging - sorry for the delay here. We have some deeper changes in flight (#57). Can we hold off on these for a bit, thanks! |
sure - will port ^ support on the #57 merge 👍 |
[ Issue #47 ] [ @juecd @rgarcia @Sayan- ]
Audio drivers + audio sockets support | In docker + unikraft | Live view with audio
kernel_audio_support_unikraft.mp4
Tests
Notes
main:5d05b0cd18ebafb8db3bf4ee1c3a9d3e3741386d) as base while fixing audio. The fix was then merged with the latestkernel-imageson this PR. If someone can run a test build to confirm / debug would be cool 👍