Add install script to distribution that updates PATH

petermr / ami3

Integration of cephis and normami code into a single base. Tests will be slimmed down

Apache License 2.0

17 stars 5 forks source link

Add install script to distribution that updates PATH #56

Open remkop opened 4 years ago

remkop commented 4 years ago

Something like:

check if environment variable AMI_HOME is defined
(re)define environment variable AMI_HOME to the directory of this script (dirname "$0" in *nix)
if AMI_HOME was already defined, assume we already added it to the PATH, so do nothing (echo message to user)
otherwise, add it in front of the PATH

EmanuelFaria commented 4 years ago

Hi Remko,

I just tried (wrongly, I assume) the variables you posted above, and got the following messages in terminal:

Last login: Mon Aug 17 17:44:56 on ttys000
Mannys-MacBook-Pro:~ emanuelfaria$ AMI_HOME
-bash: AMI_HOME: command not found
Mannys-MacBook-Pro:~ emanuelfaria$ AMI_HOME dirname "$0" in *nix
-bash: AMI_HOME: command not found
Mannys-MacBook-Pro:~ emanuelfaria$ dirname "$0" in *nix
dirname: illegal option -- b
usage: dirname path
Mannys-MacBook-Pro:~ emanuelfaria$ ami AMI_HOME
Unmatched argument at index 0: 'AMI_HOME'
Did you mean: metadata or generate-completion?
Usage: ami [OPTIONS] COMMAND
Try 'ami --help' for more information.
Mannys-MacBook-Pro:~ emanuelfaria$

What should i be doing instead? :-/

remkop commented 4 years ago

To display the value of an environment variable, type echo $variable. So, for AMI_HOME, that would be:

echo $AMI_HOME

remkop commented 4 years ago

But overall, this ticket doesn’t give instructions to setup these environment variables.

I wrote some instructions here: https://github.com/petermr/ami3/blob/master/INSTALL.md#setting-your-path-on-unix

EmanuelFaria commented 4 years ago

To display the value of an environment variable, type echo $variable. So, for AMI_HOME, that would be:
echo $AMI_HOME

Must be a mac thing, but when I try it, I just get an empty line after I hit return. Nothing happens

remkop commented 4 years ago

Good, that’s not a problem. An empty line just means that the AMI_HOME environment variable is not defined.

What do you see when you type

echo $PATH

in the terminal?

remkop commented 4 years ago

@mannyrules I added instructions for adding AMI to the PATH for macOS: see https://github.com/petermr/ami3/blob/master/INSTALL.md#setting-your-path-on-macos

I am still interested in seeing the output of the echo $PATH command when you execute it on your system, since something you said on Slack indicates you may still have an older version of AMI in the PATH...