-
Notifications
You must be signed in to change notification settings - Fork 62
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
Version flag #105
Comments
I need to think a bit about tracking a version number, I'm not opposed to it though. |
The recent sokol-shdc update to stay in sync with the webGPU changes is a good example of why a version flag would be useful: we updated to the latest sokol_gfx, and now all previous versions of sokol-shdc cause builds to fail because they continue to output if you're opposed to manually maintaining a version number, any type of identifier would do: the commit id from which the binary was built, or a timestamp of the build time. |
The lack of versions is getting increasingly annoying. It's hard to know what version of shdc you have, or what version of sokol_gfx.h you have, or if they are compatible with each other, or how to find one or the other that is compatible with what you already have. In situations that you don't want to update your code to account for changes made in sokol_gfx.h, getting the most recent of both |
Agreed. For sokol-shdc I have a fix in mind (basically adding a For the sokol headers I still don't have a good idea how to deal with what would need to be one version number per header. |
It would be useful if sokol-shdc had a
--version
/-v
flag to print its version number and exit.background:
I've integrated sokol-shdc into the build system for our project and it relies on the developer having sokol-shdc installed. I want the build script to check that it's at least some minimum version so that developers are building with the most recent version that we've tested with. For release builds we might even force it to be a specific version.
The text was updated successfully, but these errors were encountered: