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

Curl install script improvements #1341

Closed
10 tasks done
derekbekoe opened this issue Nov 16, 2016 · 2 comments
Closed
10 tasks done

Curl install script improvements #1341

derekbekoe opened this issue Nov 16, 2016 · 2 comments
Assignees
Labels

Comments

@derekbekoe
Copy link
Member

derekbekoe commented Nov 16, 2016

  • Change default install location to a user location (e.g. ~/az) instead of global location (usr/local/az)

  • Prompt user if they want the script to modify their profile/rc file to update $PATH.

  • Fail install with appropriate message if custom install path contains space. Provide user friendly error message to address Installer resolves links and does not handle spaces in paths #1070

  • Allow custom profile/rc file to be provided. By defaulting to a user location rather than global, we address Installing on OSX tries to modify root's bash completion, not the user's #1009 as sudo will no longer be required. This work item extends this further so a user provided file can be used.

  • Save tab completion script to install location and provide instructions on how user can enable tab completion later (addresses [Install] Download tab completion script even when user turns down tab completion #939).

  • After install, check $PATH for az and warn user if another az exists already on the system (e.g. from a pip install or apt-get install). We can warn the user and list the other azs found.

  • If installation already exists in target directory, prompt user so we delete the old install directory.

  • Curl script should also show a warning/error message if the required native dependencies are not installed.

  • Verify Python version (>= 2.7) before proceeding with curl install

Once the above changes have been made:

  • Publish updated scripts to storage account so they're publicly available.
@derekbekoe derekbekoe self-assigned this Nov 16, 2016
@derekbekoe
Copy link
Member Author

#1400 should be part of this change also.

@derekbekoe
Copy link
Member Author

Curl script should also show a warning/error message if the required native dependencies are not installed.
e.g.

Ubuntu 14.04

$ curl -L https://aka.ms/InstallAzureCli | bash
Native dependencies are required that are not currently installed. Please install them with the command below then try again.
“sudo apt-get install -y libssl-dev libffi-dev python-dev”

CentOS

$ curl -L https://aka.ms/InstallAzureCli | bash
Native dependencies are required that are not currently installed. Please install them with the command below then try again.
“sudo yum install -y gcc libffi-devel python-devel openssl-devel”

SUSE OpenSUSE

$ curl -L https://aka.ms/InstallAzureCli | bash
Native dependencies are required that are not currently installed. Please install them with the command below then try again.
“sudo zypper --non-interactive install gcc libffi-devel python-devel openssl-devel”

It can also provide a prompt and then attempt to install the native dependencies.

cc: @johanste, @yugangw-msft, @brendandburns

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

No branches or pull requests

2 participants