Go environment variables should be defined only if Go is not already installed about vulsctl

[Jiab77]

@sidahmed-malaoui, I think you are right but I've spotted another issue that I need to discuss with @kotakanbe, the following lines are wrong as the file .profile is not loaded but the file .bash_profile instead:

echo "export GOROOT=/usr/local/go" >> "$HOME"/.profile;
echo "export GOPATH=$HOME/go" >> "$HOME"/.profile;
echo "export PATH=$PATH:$GOROOT/bin:$GOPATH/bin" >> "$HOME"/.profile;

and should be replaced by:

echo "export GOROOT=/usr/local/go" >> $HOME/.bash_profile
echo "export GOPATH=$HOME/go" >> $HOME/.bash_profile
echo "export PATH=$PATH:$GOROOT/bin:$GOPATH/bin" >> $HOME/.bash_profile

Another thing that should be cleaned in the script is the leading ; at the end of the lines, it is useless in bash for running a single command.

[Jiab77]

@kotakanbe, here is the patching code I'm currently testing for the install-host/upgrade.sh script:

# Dirty GO missing variables patch
if [[ -f $HOME/.profile ]]; then
	source $HOME/.profile
else
	source $HOME/.bash_profile
fi

Let me know what you prefer and I can do the required changes.

[sudnonk]

I agree with this issue.
In my case, on CentOS, I installed golang via yum and they placed go at /usr/lib/golang/bin.
However, as it has mentioned in this issue, the script changes GOROOT, then make install in later part can't find the directory /usr/local/go.
I solved simply just comment out line 9-11.