Page MenuHomeFreeBSD

audio/virtual_oss: Add sndstat(4) support
ClosedPublic

Authored by khng on Mar 24 2021, 3:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 21, 8:31 PM
Unknown Object (File)
Mar 13 2024, 4:04 PM
Unknown Object (File)
Mar 13 2024, 4:04 PM
Unknown Object (File)
Mar 13 2024, 4:04 PM
Unknown Object (File)
Mar 10 2024, 10:54 AM
Unknown Object (File)
Feb 21 2024, 11:22 PM
Unknown Object (File)
Jan 22 2024, 10:45 PM
Unknown Object (File)
Jan 10 2024, 1:45 AM
Subscribers

Details

Summary

Fix "FATAL: Makefile: [37]: use a tab (not space) after a variable name"
as well.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 38048
Build 34937: arc lint + arc unit

Event Timeline

khng requested review of this revision.Mar 24 2021, 3:58 PM
This revision is now accepted and ready to land.Mar 24 2021, 4:03 PM

Approved by: philip (mentor)

But just wondering: how does this software behave when it is built on a system where sndstat.h is available but subsequently run on a system where it is not? i.e. what happens if we build a binary package on a new package builder and install the package on an older (but still supported) version of FreeBSD?

Approved by: philip (mentor)

But just wondering: how does this software behave when it is built on a system where sndstat.h is available but subsequently run on a system where it is not? i.e. what happens if we build a binary package on a new package builder and install the package on an older (but still supported) version of FreeBSD?

The following line shall fail:
https://github.com/hselasky/virtual_oss/blob/main/virtual_main.c#L1811

The consequence of this ioctl call failing is that no virtual_oss devices will be listed in /dev/sndstat, despite the node created by virtual_oss is still presented. For the later question, our current builder will run with the lowest supported release version, so that should not happen in practise.

This revision was automatically updated to reflect the committed changes.