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

Automatic install script is broken #343

Closed
atruskie opened this issue Jul 17, 2020 · 0 comments · Fixed by #430
Closed

Automatic install script is broken #343

atruskie opened this issue Jul 17, 2020 · 0 comments · Fixed by #430

Comments

@atruskie
Copy link
Member

Describe the bug

The automatic install script no longer works. It was built using the old release system and doesn't know about the new download packages.

Expected behavior

The script should download the right package for any system.

@atruskie atruskie changed the title Autoamtic install script is broken Automatic install script is broken Jul 17, 2020
atruskie added a commit that referenced this issue Jan 29, 2021
- now installs, checks, or uninstalls AP
- powershell script is cross-platform
- properly documented commandlet
- GA will test script on Windows/Linux/MacOS
- Adds install directory to PATH on Windows, symlinks the binary on Linux/MacOS
- Adds an alias (simply AP) for analysis programs!
- All installs are local and should not need admin permissions
- the exception being Linux, which will install sox if it needs to (sudo required)

Fixes #343
atruskie added a commit that referenced this issue Feb 1, 2021
- now installs, checks, or uninstalls AP
- powershell script is cross-platform
- properly documented commandlet
- GA will test script on Windows/Linux/MacOS
- Adds install directory to PATH on Windows, symlinks the binary on Linux/MacOS
- Adds an alias (simply AP) for analysis programs!
- All installs are local and should not need admin permissions
- the exception being Linux, which will install sox if it needs to (sudo required)

Fixes #343

Fix CI problems

Also added -Force to test scripts because the CI is non-interactive

Fix matrix variable reference

Fix script name missing

Also change condition for testing AP alias removed

Fix up install_dir var

I intended to use it as bin_dir but used the wrong concept

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

Successfully merging a pull request may close this issue.

1 participant