-
Notifications
You must be signed in to change notification settings - Fork 34
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
Lock down versioning/release process #65
Comments
We probably also want some way of determining tool/VST versions so it's easy for users to check their compatibility. |
Note that when distributing plugins, it's crucial that we adhere to the VST SDK license terms, whatever those may be (this requires some investigation). The tools themselves might also have licenses we need to adhere to (eg. the converter tests at least use a third-party JSON codec lib). |
OK, so TL;DR the VST SDK terms are really awful, see https://developer.steinberg.help/display/VST/VST+3+Licensing. If I'm interpreting them correctly, then TL;DR we would have to:
Alternatively we might be able to find an alternative framework (eg. iPlug2, which is not yet ready for prime-time) or reimplement this API ourselves (the API surface is fairly small, but can we do this "clean-room" enough etc? Will plugin hosts drop VST2 support any time soon?); neither are very attractive right now. Ugh. Note that we should still be able to version/ship source-only releases, which is likely sufficient for what I really want here (which is to nail down proper versions so we can start making potentially-breaking changes). |
I've looked into/thought a bit more about this, and I'm currently thinking the following (note that the usual IANAL/this is not legal advice etc apply):
So, I think the next step is to look into reimplementing the parts of the VST2 plugin API that we actually want/need to support, using vst-rs as a reference/model impl. |
After looking a bit into vst-rs and talking to the folks behind it, it still seems some of this stuff operates in a legal grey-area, which I'm not entirely thrilled about. Another option came up, which was to migrate to VST 3 proper, still BYOSDK for developers, and sign the VST 3 proprietary agreement myself in order to ship binary builds on behalf of the project (at least covering the main repo; we might need to be very explicit that other folks couldn't distribute the same plugs under these terms). As far as I can tell, this avoids any legal grey-areas and allows us to be fully compliant with the license terms, but could actually be considered a breaking change, as it locks out older DAWs that don't support VST 3. One such DAW is Live 9, which we've used for years and currently have test projects for in the converter test suite. I'm not entirely sure if this matters tho; afaik most folks are on Live 10 or even 11 by now, but still. All of this is still fairly unsatisfactory, but I realized another important point - shipping binary plug builds and starting to have versioned/tagged "releases" are two separate issues, and we can have one without the other. So, I'm going to spawn another issue (Edit: #83) to handle the binary build work (and defer it probably) and instead shrink-wrap today's codebase as a source release and go from there. |
Since we don't currently build any binaries or anything, I think we can simply draft tagged source releases via the Github UI for now, and properly document/detail a process (likely including a proper changelog too) later for future versions. |
I really want to do this ASAP, both to make it easier for users to get up and running without necessarily building anything (assuming we'd ship pre-built VSTs and perhaps also libs/converter tools as part of the releases) but also as I think WS could be made a lot better with a few fundamental changes/fixes, and I'd rather start doing those after we get a release of WS in its current state made, so users can in theory always have something to go back to for old projects if it's important to them (though I suspect users are almost always happier with new toys than ways to keep the old ones working).
The text was updated successfully, but these errors were encountered: