Skip to content

GitLab

Closed
Created by João Sequeira@iquasereOwner

Several suggested improvements to install.bash

Created by: bwu62

Things I think you should consider changing:

  1. Remove Xming install line at beginning of file. This seems to be specifically a Windows program and should not be hard coded into the automated install file.
  2. As of writing this, installing conda and then trying to conda install anything runs into SSL errors due to a change in pyOpenSSL. You need to upgrade it to at least 16.2.0 with python3 -m easy_install --upgrade pyOpenSSL==16.2.0
  3. Conda installs should have an additional -y flag to suppress confirmation
  4. Lines 23,31 you shouldn't hard code your own home directory path.
  5. Line 25,31 please prepend any comments with a #
  6. Line 35 I don't think you need to install aptitude just to install build-essential. Why not add build-essential to line 36 and do both at once?
  7. Lines 73,74 are python code and do not belong in a bash file