Updated right kvm.sh and link in readme

Author: Suhas Joshi

README.md

@@ -52,7 +52,7 @@ To install KVM and the correct version of Mono on OS X using [Homebrew](http://b
 
  * Install [Homebrew](http://brew.sh) if it is not already installed.
  * Run command `brew tap aspnet/k` to tap the ASP.NET vNext related git repositories. If you had already tapped the repo for previous releases, run `brew untap aspnet/k` to delete the old commands and tap again to get the updated brew scripts.
- * Run command `brew install kvm` to install KVM. This also automatically install the latest KRE package from https://www.myget.org/f/aspnetmaster/api/v2 feed.
+ * Run command `brew install kvm` to install KVM. This also automatically install the latest KRE package from https://www.nuget.org/api/v2 feed.
  * Run command `source kvm.sh` on your terminal if your terminal cannot understand kvm. 
 
 #### Linux:

kvm.sh

@@ -14,23 +14,19 @@ if [ -z "$KRE_USER_HOME" ]; then
 fi
 
 KRE_USER_PACKAGES="$KRE_USER_HOME/packages"
-KRE_MONO45=
-KRE_X86=
-KRE_X64=
 if [ -z "$KRE_FEED" ]; then
     KRE_FEED="https://www.myget.org/F/aspnetmaster/api/v2"
 fi
 
 _kvm_find_latest() {
-    local platform="mono45"
-    local architecture="x86"
+    local platform="Mono"
 
     if ! _kvm_has "curl"; then
         echo 'KVM Needs curl to proceed.' >&2;
         return 1
     fi
 
-    local url="$KRE_FEED/GetUpdates()?packageIds=%27KRE-$platform-$architecture%27&versions=%270.0%27&includePrerelease=true&includeAllVersions=false"
+    local url="$KRE_FEED/GetUpdates()?packageIds=%27KRE-$platform%27&versions=%270.0%27&includePrerelease=true&includeAllVersions=false"
     xml="$(curl $url 2>/dev/null)"
     echo $xml | grep \<[a-zA-Z]:Version\>* >> /dev/null || return 1
     version="$(echo $xml | sed 's/.*<[a-zA-Z]:Version>\([^<]*\).*/\1/')"
@@ -61,7 +57,7 @@ _kvm_package_name() {
 
 _kvm_package_runtime() {
     local kreFullName="$1"
-    echo "$kreFullName" | sed "s/KRE-\([^-]*\).*/\1/"
+    echo "$kreFullName" | sed "s/KRE-\([^.-]*\).*/\1/"
 }
 
 _kvm_download() {
@@ -93,6 +89,7 @@ _kvm_download() {
     [[ $httpResult != "302" && $httpResult != "200" ]] && echo "HTTP Error $httpResult fetching $kreFullName from $KRE_FEED" && return 1
 
     _kvm_unpack $kreFile $kreFolder
+    return  $? 
 }
 
 _kvm_unpack() {
@@ -119,38 +116,20 @@ _kvm_unpack() {
         -exec sh -c "head -c 11 {} | grep '/bin/bash' > /dev/null"  \; -print | xargs chmod 775
 }
 
-# This is not currently required. Placeholder for the case when we have multiple platforms  (ie if we bundle mono)
-_kvm_requested_platform() {
-    local default=$1
-    [[ -z $KRE_MONO45 ]] && echo "mono45" && return
-    echo $default
-}
-
-# This is not currently required. Placeholder for the case where we have multiple architectures (ie if we bundle mono)
-_kvm_requested_architecture() {
-    local default=$1
-
-    [[ -n $KRE_X86 && -n $KRE_X64 ]] && echo "This command cannot accept both -x86 and -x64" && return 1
-    [[ -z $KRE_X86 ]] && echo "x86" && return
-    [[ -z $KRE_X64 ]] && echo "x64" && return
-    echo $default
-}
-
 _kvm_requested_version_or_alias() {
     local versionOrAlias="$1"
 
     if [ -e "$KRE_USER_HOME/alias/$versionOrAlias.alias" ]; then
         local kreFullName=$(cat "$KRE_USER_HOME/alias/$versionOrAlias.alias")
         local pkgName=$(echo $kreFullName | sed "s/\([^.]*\).*/\1/")
         local pkgVersion=$(echo $kreFullName | sed "s/[^.]*.\(.*\)/\1/")
-        local pkgPlatform=$(_kvm_requested_platform $(echo "$pkgName" | sed "s/KRE-\([^-]*\).*/\1/"))
-        local pkgArchitecture=$(_kvm_requested_architecture $(echo "$pkgName" | sed "s/.*-.*-\([^-]*\).*/\1/"))
+        local pkgPlatform=$(echo "$pkgName" | sed "s/KRE-\([^.-]*\).*/\1/")
     else
         local pkgVersion=$versionOrAlias
-        local pkgPlatform=$(_kvm_requested_platform "mono45")
-        local pkgArchitecture=$(_kvm_requested_architecture "x86")
+        local pkgPlatform="Mono"
     fi
-    echo "KRE-$pkgPlatform-$pkgArchitecture.$pkgVersion"
+
+    echo "KRE-$pkgPlatform.$pkgVersion"
 }
 
 # This will be more relevant if we support global installs
@@ -169,7 +148,7 @@ kvm()
     case $1 in
         "help" )
             echo ""
-            echo "K Runtime Environment Version Manager - Build 10002"
+            echo "K Runtime Environment Version Manager - Build 10017"
             echo ""
             echo "USAGE: kvm <command> [options]"
             echo ""
@@ -198,12 +177,14 @@ kvm()
             echo "list KRE aliases which have been defined"
             echo ""
             echo "kvm alias <alias>"
-            echo "display value of named alias"
+            echo "display value of the specified alias"
             echo ""
             echo "kvm alias <alias> <semver>"
             echo "<alias>            The name of the alias to set"
             echo "<semver>|<alias>   The KRE version to set the alias to. Alternatively use the version of the specified alias"
             echo ""
+            echo "kvm unalias <alias>"
+            echo "remove the specified alias"
             echo ""
         ;;
 
@@ -233,7 +214,7 @@ kvm()
             done
             if [[ "$versionOrAlias" == "latest" ]]; then
                 echo "Determining latest version"
-                versionOrAlias=$(_kvm_find_latest mono45 x86)
+                versionOrAlias=$(_kvm_find_latest)
                 [[ $? == 1 ]] && echo "Error: Could not find latest version from feed $KRE_FEED" && return 1
                 echo "Latest version is $versionOrAlias"
             fi
@@ -249,14 +230,15 @@ kvm()
                   mkdir "$kreFolder" > /dev/null 2>&1
                   cp -a "$versionOrAlias" "$kreFile"
                   _kvm_unpack "$kreFile" "$kreFolder"
+                  [[ $? == 1 ]] && return 1
                 fi
                 kvm use "$kreVersion" "$persistent"
                 [[ -n $alias ]] && kvm alias "$alias" "$kreVersion"
             else
                 local kreFullName="$(_kvm_requested_version_or_alias $versionOrAlias)"
                 local kreFolder="$KRE_USER_PACKAGES/$kreFullName"
                 _kvm_download "$kreFullName" "$kreFolder"
-                [[ $? == 1 ]] && return
+                [[ $? == 1 ]] && return 1
                 kvm use "$versionOrAlias" "$persistent"
                 [[ -n $alias ]] && kvm alias "$alias" "$versionOrAlias"
             fi
@@ -339,12 +321,23 @@ kvm()
 
             local kreFullName=$(_kvm_requested_version_or_alias "$3")
 
-            [[ ! -d "$KRE_USER_PACKAGES/$kreFullName" ]] && echo "$kreFullName is not an installed KRE version." && return 1
-
-            echo "Setting alias '$name' to '$kreFullName'"
+            [[ ! -d "$KRE_USER_PACKAGES/$kreFullName" ]] && echo "$kreFullName is not an installed KRE version" && return 1
 
+            local action="Setting"
+            [[ -e "$KRE_USER_HOME/alias/$name.alias" ]] && action="Updating"
+            echo "$action alias '$name' to '$kreFullName'"
             echo "$kreFullName" > "$KRE_USER_HOME/alias/$name.alias"
         ;;
+        
+        "unalias" )
+            [[ $# -ne 2 ]] && kvm help && return
+            
+            local name=$2
+            local aliasPath="$KRE_USER_HOME/alias/$name.alias"
+            [[ ! -e  "$aliasPath" ]] && echo "Cannot remove alias, '$name' is not a valid alias name" && return 1
+            echo "Removing alias $name"
+            rm "$aliasPath" >> /dev/null 2>&1
+        ;;
 
         "list" )
             [[ $# -gt 2 ]] && kvm help && return
@@ -357,15 +350,38 @@ kvm()
                 local searchGlob=$(_kvm_requested_version_or_alias "$versionOrAlias")
             fi
             echo ""
-            local formatString="%-6s %-20s %-7s %-12s %s\n"
-            printf "$formatString" "Active" "Version" "Runtime" "Architecture" "Location"
-            printf "$formatString" "------" "-------" "-------" "------------" "--------"
+            
+            local arr=()            
+            local i=0
+            local format="%-20s %s\n"
+            for _kvm_file in $(find "$KRE_USER_HOME/alias" -name *.alias); do
+                arr[$i]="$(basename $_kvm_file | sed 's/.alias//')/$(cat $_kvm_file)"
+                let i+=1
+            done
+    
+            local formatString="%-6s %-20s %-7s %-20s %s\n"
+            printf "$formatString" "Active" "Version" "Runtime" "Location" "Alias"
+            printf "$formatString" "------" "-------" "-------" "--------" "-----"
+            
+            local formattedHome=`(echo $KRE_USER_PACKAGES | sed s=$HOME=~=g)`
             for f in $(find $KRE_USER_PACKAGES/* -name "$searchGlob" -type d -prune -exec basename {} \;); do
                 local active=""
                 [[ $PATH == *"$KRE_USER_PACKAGES/$f/bin"* ]] && local active="  *"
                 local pkgName=$(_kvm_package_runtime "$f")
                 local pkgVersion=$(_kvm_package_version "$f")
-                printf "$formatString" "$active" "$pkgVersion" "$pkgName" "x86" "$KRE_USER_PACKAGES"
+
+                local alias=""
+                local delim=""
+                for i in "${arr[@]}"; do    
+                    temp="KRE-$pkgName.$pkgVersion"
+                    temp2="KRE-$pkgName-x86.$pkgVersion"
+                    if [[ ${i#*/} == $temp || ${i#*/} == $temp2 ]]; then
+                        alias+="$delim${i%/*}"
+                        delim=", "
+                    fi
+                done
+
+                printf "$formatString" "$active" "$pkgVersion" "$pkgName" "$formattedHome" "$alias"
                 [[ $# == 2 ]] && echo "" &&  return 0
             done
 
@@ -377,6 +393,8 @@ kvm()
             echo "Unknown command $1"
             return 1
     esac
+    
+    return 0
 }
 
-kvm list default >/dev/null && kvm use default >/dev/null || true
+kvm list default >/dev/null && kvm use default >/dev/null || true