8000 LinuxVST: add missing CMakeLists.txt, remove superfluous CMakeCache.txt, change README by magnetophon · Pull Request #5 · airwindows/airwindows · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

LinuxVST: add missing CMakeLists.txt, remove superfluous CMakeCache.txt, change README #5

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

magnetophon
Copy link
@magnetophon magnetophon commented Aug 21, 2018

Hi Chris,

Many thanks for releasing these great plugins under a free licence!

I had some trouble building the LinuxVST's; these changes fix that.
I also added @ECH2's note about your builds and made the instructions in the readme more copy paste.

@magnetophon magnetophon changed the title add missing CMakeLists.txt, remove superfluous CMakeCache.txt. LinuxVST: add missing CMakeLists.txt, remove superfluous CMakeCache.txt, change README Aug 21, 2018
Then, build and install:
```
cd airwindows/plugins/LinuxVST/
cmake

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be:

cd airwindows/plugins/LinuxVST/build
cmake ..

cmake
make
mkdir -p /usr/local/lib/lxvst
cp *.so /usr/local/lib/lxvst

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most Linux VST hosts actually look for LinuxVSTs in /usr/lib/vst and many distributions install them there.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some distributions install the vst in /isr/lib64/vst ... Using cmake variable LIBDIR.

@magnetophon
Copy link
Author
magnetophon commented Sep 30, 2019

@SpotlightKid Thanks for your feedback.
I'm not sure if it's worth putting more effort into this, since the maintainers have not reacted yet, in over a year... :(

@SpotlightKid
Copy link

@magnetophon Your PR was very helpful in getting the plugins compiled on Linux. Maybe my tiny corrections will help others too.

@magnetophon
Copy link
Author

Fair enough! Thanks again!

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

Successfully merging this pull request may close these issues.

3 participants
0