Fix goenv install for 1.3.1 and later

[alext]

There was a problem with the vercomp function that caused it to echo
more than once for some comparisons. This would appear to be a side
effect of changing the stackoverflow function to echo instead of using
return value.

Comparing 1.2 and 1.3.1 is one of the cases that was echoing more than
once. This then caused the test for "$rtn" == "1" to be false, which
meant that it was trying to use the old source URL.

[meatballhat]

👏

(assuming it works! 😆)

[alext]

@Solistra I can't reproduce that issue. It downloaded 1.0.3 for me correctly. Additionally, I extracted the function into this test script:

#! /usr/bin/env bash

function vercomp () {
    # http://stackoverflow.com/questions/4023830
    # 0: '='
    # 1: '>'
    # 2: '<'
    if [[ $1 == $2 ]]
    then
        echo 0
        return
    fi
    local IFS=.
    local i ver1=($1) ver2=($2)
    # fill empty fields in ver1 with zeros
    for ((i=${#ver1[@]}; i<${#ver2[@]}; i++))
    do
        ver1[i]=0
    done
    for ((i=0; i<${#ver1[@]}; i++))
    do
        if [[ -z ${ver2[i]} ]]
        then
            # fill empty fields in ver2 with zeros
            ver2[i]=0
        fi
        if ((10#${ver1[i]} > 10#${ver2[i]}))
        then
            echo 1
            return
        fi
        if ((10#${ver1[i]} < 10#${ver2[i]}))
        then
            echo 2
            return
        fi
    done
#    echo 0
}

for ver in 1.0 1.0.3 1.1 1.1.1 1.2 1.2.1 1.3 1.3.1; do
  echo "${ver}         $(vercomp ${ver} 1.2)"
done

and it returned the values expected:

1.0       2
1.0.3       2
1.1       2
1.1.1       2
1.2       0
1.2.1       1
1.3       1
1.3.1       1

[jszwedko]

👍 any news on this @wfarr ?