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
Add sndio sound support for OpenBSD #451
Conversation
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.
Looks good overall
@@ -11,7 +11,7 @@ LDFLAGS="" | |||
case $(uname -s) in | |||
OpenBSD) | |||
CFLAGS="$CFLAGS -I/usr/local/include" | |||
LIBS="$LIBS -lexecinfo" | |||
LIBS="$LIBS -lexecinfo -lsndio" |
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.
This should go in Makefile.inc
in the plugins dir as such:
PLIBS=-lsndio
@@ -0,0 +1,201 @@ | |||
/* sqUnixSndioSound.c -- sound module for sndio sound system | |||
* | |||
* Last edited: 2005-04-06 05:44:40 by piumarta on pauillac.hpl.hp.com |
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.
maybe adapt that one ;)
#include <stdio.h> | ||
#include <sndio.h> | ||
|
||
#define trace() fprintf(stderr, "%s:%d %s\n", __FILE__, __LINE__, __FUNCTION__) |
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.
Please add a debug guard
This commit introduces a new plugin "vm-sound-sndio" which uses the native OpenBSD sound system as backend. This commit is meant as an initial introduction of the plugin, as sound output currently blocks the whole VM. The plugin is not loaded by default and should only get built when running on OpenBSD.
Many thanks for your feedback and sorry for the late reply. I have updated the branch incorporating your suggestions. If you can find the time to have a loot at it again, I'd appreciate it. |
Looks good to me. Anyone else having a look? |
On Wed, Dec 11, 2019 at 1:56 PM Tobias Pape ***@***.***> wrote:
Looks good to me. Anyone else having a look?
I don't see anything problematic. Integrate?
… —
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#451?email_source=notifications&email_token=AIJPEW23V5BJRL3YZK6CWXLQYFOXNA5CNFSM4JYGZGYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGUWUKQ#issuecomment-564750890>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIJPEWZRBHOIIHUR6OE3FDDQYFOXNANCNFSM4JYGZGYA>
.
--
_,,,^..^,,,_
best, Eliot
|
…: ] on method [ lastPointerOfWhileSwizzling: ]
… ] on method [ lastPointerOfWhileSwizzling: ] KILLED by 1/209 test cases.
…lse] ] on method [ freeTreeNodesDo: ]
…se] ] on method [ freeTreeNodesDo: ] KILLED by 1/234 test cases.
… true ] on method [ popObjStack: ] 14 test cases.
…true ] on method [ popObjStack: ] 10/14 Test Cases are NOT EQUIVALENT
…-windows Adding Libgit2-1.4.4 in Windows
This commit introduces a new plugin "vm-sound-sndio" which uses the
native OpenBSD sound system as backend.
This commit is meant as an initial introduction of the plugin, as
sound output currently blocks the whole VM.
The plugin is not loaded by default and should only get built when
running on OpenBSD.
My intention with this pull request is to get the infrastructure for future improvements up and running, so further development is easier for me and people have a way of testing the plugin.