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

build-linux.sh error and workaround #493

Closed
ecv80 opened this issue Feb 4, 2019 · 3 comments
Closed

build-linux.sh error and workaround #493

ecv80 opened this issue Feb 4, 2019 · 3 comments

Comments

@ecv80
Copy link

ecv80 commented Feb 4, 2019

Describe the bug
./build-linux.sh --clean throws:
./build-linux.sh: unrecognized option '--clean'
./build-linux.sh: missing operand
Try './build-linux.sh --help' for more information.

To Reproduce
Steps to reproduce the behavior:

  1. Follow Linux install instructions up to and including this step "Run ./build-linux.sh --clean"

Expected behavior
Shouldn't throw errors and just clean.

Screenshots
n/a

Desktop (please complete the following information):

  • OS: Ubuntu 18.04.1 LTS
  • Host n/a
  • Version git master

Additional context
Another user and I got the aforementioned error when trying to build.
It's something to do with argument parsing.
I don't know how getopt works, so I worked around it using sed.
I also switched the help and verbose options since they seem to be inverted.
I was able to build but can not test it to confirm it all works as I have no vst3 host. LV2 appears to be the generally desirable option as per some discussions and popular hosts roadmaps.
Here's the modified script: https://pastebin.com/AhmwMAa6

@baconpaul
Copy link
Collaborator

baconpaul commented Feb 4, 2019

@jsakkine changed it to not use --. We need to update docs.
Run ./build-linux.sh clean.
which doc did you read?
Thanks!

@esaruoho
Copy link
Collaborator

esaruoho commented Feb 4, 2019

-- has been removed from README.md

@baconpaul
Copy link
Collaborator

Hi @ecv80 we updated the README this morning to include the latest directions on how to build. Sorry about that! You caught the package in the 12 hours between us changing the arguments and updating the docs!

If you look at the README now they work with the code at master.

I'm going to close this issue. Please reopen it if you still experience problems.

Thank you.

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

3 participants