audio: beginnings of audio system #372

Merged
alichraghi merged 4 commits from main into main 2022-06-29 04:57:13 +00:00
alichraghi commented 2022-06-26 20:06:12 +00:00 (Migrated from github.com)
  • By selecting this checkbox, I agree to license my contributions to this project under the license(s) described in the LICENSE file, and I have the right to do so or have received permission to do so by an employer or client I am producing work for whom has this right.
- [x] By selecting this checkbox, I agree to license my contributions to this project under the license(s) described in the LICENSE file, and I have the right to do so or have received permission to do so by an employer or client I am producing work for whom has this right.
emidoots (Migrated from github.com) approved these changes 2022-06-27 06:15:51 +00:00
emidoots (Migrated from github.com) left a comment

Looks like a great starting point, thanks!

Looks like a great starting point, thanks!
alichraghi commented 2022-06-27 08:08:30 +00:00 (Migrated from github.com)

a lot of changes coming... :)
sorry i could not get enough time and delayed you guys, so I think it would be better to merge this now. i will write the rest in another PR.

~~a lot of changes coming... :)~~ sorry i could not get enough time and delayed you guys, so I think it would be better to merge this now. i will write the rest in another PR.
emidoots (Migrated from github.com) reviewed 2022-06-29 04:55:20 +00:00
@ -0,0 +6,4 @@
pub const pkg = std.build.Pkg{
.name = "soundio",
.source = .{ .path = thisDir() ++ "/soundio/main.zig" },
};
emidoots (Migrated from github.com) commented 2022-06-29 04:55:19 +00:00

this should point to audio/main.zig right? We can do this in another PR, just making a note

this should point to `audio/main.zig` right? We can do this in another PR, just making a note
emidoots (Migrated from github.com) reviewed 2022-06-29 04:55:40 +00:00
@ -0,0 +14,4 @@
alsa: bool = false,
coreaudio: bool = false,
wasapi: bool = false,
};
emidoots (Migrated from github.com) commented 2022-06-29 04:55:39 +00:00

I'm not sure we should make these options public at all.

I'm not sure we should make these options public at all.
emidoots (Migrated from github.com) reviewed 2022-06-29 04:55:55 +00:00
@ -0,0 +34,4 @@
step.addIncludePath(soundio_path);
}
pub fn buildSoundIo(b: *Builder, step: *std.build.LibExeObjStep, options: SoundIoOptions) *std.build.LibExeObjStep {
emidoots (Migrated from github.com) commented 2022-06-29 04:55:55 +00:00

I'd suggest we also do not make this public.

I'd suggest we also do not make this public.
emidoots (Migrated from github.com) approved these changes 2022-06-29 04:56:48 +00:00
emidoots (Migrated from github.com) left a comment

Nice work!

Nice work!
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
hexops/mach!372
No description provided.