-
Notifications
You must be signed in to change notification settings - Fork 35
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
Investigate including VST3 SDK as a submodule #91
Comments
Unfortunately, we still need to support VST3 in order to do this (#89), as even the earliest tagged VST3 SDK releases on Github don't include any of the VST2 SDK code. |
If this isn't possible then a "hack" would be to in a build setup use |
Indeed, we should try to see if that works with this archive link that we currently link to in our docs, and if so, we could provide (platform-specific?) script(s) to set it up more easily. |
submodules are notoriously hard to work with, and I would recommend against using them unless it provides a lot of benefit over something else. It's the ugly duckling that was added to Git simply so KDE could convert their SVN repo that used svn:external heavily, but then nobody really gave it the love that was needed... It's not really something that should have been in Git in the first place IMO. |
Yeah. For windows it would be possible to just include wget/curl and unzip inside a |
@kusma In this case it might be ok if it's only to fetch the code and never update it, otherwise I agree |
another problem tho with submodules is that everyone forgets to add |
@emoon: It's still pretty awkward to set up the project if you're not online or if github has problems, even if you have both repos already cloned. Submodules don't really do the "distributed" part of "distributed version control"... |
Right |
And yeah, I have opted to move away from submodules for most/all of my projects because in the end they are a PITA to work with. |
agreed about submodules; a custom script is likely much easier to use/work with. |
It would be possible to use |
Like this: #92 :) |
Fixed in #92 |
This doesn't count as distribution as far as I'm aware, so long as our source releases don't actually include this source, we should be able to do this, which simplifies BYOSDK greatly.
The text was updated successfully, but these errors were encountered: