Skip to content
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

Closed
yupferris opened this issue Dec 30, 2021 · 14 comments
Closed

Investigate including VST3 SDK as a submodule #91

yupferris opened this issue Dec 30, 2021 · 14 comments

Comments

@yupferris
Copy link
Member

yupferris commented Dec 30, 2021

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.

@yupferris
Copy link
Member Author

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.

@emoon
Copy link

emoon commented Dec 30, 2021

If this isn't possible then a "hack" would be to in a build setup use wget/curl or something like that to fetch and unpack it.

@yupferris
Copy link
Member Author

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.

@kusma
Copy link
Contributor

kusma commented Dec 30, 2021

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.

@emoon
Copy link

emoon commented Dec 30, 2021

Yeah. For windows it would be possible to just include wget/curl and unzip inside a bin directory, and then have a setup.cmd setup.sh etc that just fetches it.

@emoon
Copy link

emoon commented Dec 30, 2021

@kusma In this case it might be ok if it's only to fetch the code and never update it, otherwise I agree

@emoon
Copy link

emoon commented Dec 30, 2021

another problem tho with submodules is that everyone forgets to add --recursive when cloning. Now not a hard problem to fix, but it will be a constant question :)

@kusma
Copy link
Contributor

kusma commented Dec 30, 2021

@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"...

@emoon
Copy link

emoon commented Dec 30, 2021

Right

@emoon
Copy link

emoon commented Dec 30, 2021

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.

@yupferris
Copy link
Member Author

agreed about submodules; a custom script is likely much easier to use/work with.

@LeStahL
Copy link
Contributor

LeStahL commented Jan 5, 2022

It would be possible to use file(DOWNLOAD ...) and file(ARCHIVE_EXTRACT ...) in the CMakeLists.txt to get and unpack the VST3 SDK automatically - that saves the trouble of using scripts (or having the users download it manually).

@LeStahL
Copy link
Contributor

LeStahL commented Jan 5, 2022

Like this: #92 :)

@yupferris
Copy link
Member Author

Fixed in #92

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants