Skip to content

Move miniaudio and stb_vorbis dependencies to build.zig.zon#10

Merged
IntegratedQuantum merged 10 commits intoPixelGuys:masterfrom
MiniBaker007:miniaudio_stbvorbis
Dec 23, 2025
Merged

Move miniaudio and stb_vorbis dependencies to build.zig.zon#10
IntegratedQuantum merged 10 commits intoPixelGuys:masterfrom
MiniBaker007:miniaudio_stbvorbis

Conversation

@MiniBaker007
Copy link
Copy Markdown
Contributor

Progress towards PixelGuys/Cubyz#2346

The stb_vorbis lib is included with the official miniaudio repo, so I include it from there.

Also, miniaudio internally uses stb_vorbis for decoding ogg vorbis files, but the stb_vorbis header must be defined first to enable that, so a small wrapper file is created on the fly. This is necessary if we want to be using miniaudio's high level API (without rewriting all the interfaces for a custom decoder.)

Comment thread build.zig Outdated
Comment thread build.zig Outdated
@Argmaster Argmaster moved this to Easy to Review in PRs to review Dec 20, 2025
@Argmaster Argmaster moved this from Easy to Review to Low Priority in PRs to review Dec 20, 2025
@Argmaster Argmaster moved this from Low Priority to High Priority in PRs to review Dec 20, 2025
Comment thread build.zig Outdated
Comment thread build.zig Outdated
Comment thread build.zig Outdated
@IntegratedQuantum IntegratedQuantum moved this from High Priority to In review in PRs to review Dec 20, 2025
Comment thread build.zig Outdated
Comment thread build.zig Outdated
Comment thread build.zig Outdated
Comment thread build.zig Outdated
Comment thread lib/miniaudio_stbvorbis.c Outdated
Copy link
Copy Markdown
Member

@IntegratedQuantum IntegratedQuantum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want me to make a release of Cubyz-libs with these changes right away?

@IntegratedQuantum IntegratedQuantum merged commit f19a4c5 into PixelGuys:master Dec 23, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants