-
Notifications
You must be signed in to change notification settings - Fork 42
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 bindings for Zig 0.11.x #242
base: master
Are you sure you want to change the base?
Conversation
@psi29a : What do you think? |
No idea, I mean... I don't think anyone here is willing to maintain Zig support, so I'm afraid that at some point, it will just break? This PR also does not have a .github action associated with it. The assumption is that the c-api is enough for literally any language to import. |
Leaving to your decision @psi29a |
The PR description literally asks for feedback on whether it needs a GitHub action. The PR is a year old at this point and went completely ignored until now. I wasn’t going to spend a bunch of time without even a nod from a maintainer that this is something they are interested in. |
@deevus if you would like to add github actions; feel free. tests if you'd like; make sure though that the actions/tests do not automatically the whole flow yet though please. :) |
Ok I'll try to circle back round to update this again for 0.13.x |
Firstly, thanks for this project. I've been working on reverse engineering a MSDOS game and I was able to deobfuscate a bunch of HMP files, and then convert them to WAV files using wildmidi. 👍
I have been writing said project in Zig, so in doing that work I wrote a binding so that I could compile wildmidi into my program. In the spirit of OSS I'm contributing that work back here, if you want it. It has been tested on Windows, Linux and MacOS.
Let me know what else you would like me to do. E.g.:
For the Zig package manager to work,
build.zig
needs to be in the root of the project I think.