Fix bash arg quoting, remove bash/batch shortcuts on uninstall

src/functions/Chocolatey-Uninstall.ps1

@@ -24,6 +24,8 @@ param(
 		else {
 		  Write-Debug "Looking for $($package).$($versions.found)"
           $packageFolder = Join-Path $nugetLibPath "$($package).$($versions.found)" 
+          Write-host "Uninstalling from folder $packageFolder"
+          Get-ChocolateyBins $packageFolder -uninstall
           Run-ChocolateyPS1 $packageFolder $package "uninstall"
 		  Remove-Item -Recurse -Force $packageFolder
 		}

src/functions/Generate-BinFile.ps1

@@ -11,6 +11,7 @@ param(
   $path = $path.ToLower().Replace($nugetPath.ToLower(), "%DIR%..\").Replace("\\","\")
   $pathBash = $path.Replace("%DIR%..\","`$DIR/../").Replace("\","/")
   Write-Host "Adding $packageBatchFileName and pointing to `'$path`'." -ForegroundColor $Note
+  Write-Host "Adding $packageBashFileName and pointing to `'$path`'." -ForegroundColor $Note
   if ($useStart) {
     Write-Host "Setting up $name as a non-command line application."  -ForegroundColor $Note
 "@echo off

src/functions/Get-ChocolateyBins.ps1

@@ -1,6 +1,7 @@
 function Get-ChocolateyBins {
 param(
-  [string] $packageFolder
+  [string] $packageFolder,
+  [switch] $uninstall
 )
   Write-Debug "Running 'Get-ChocolateyBins' for $packageFolder";
 
@@ -17,9 +18,17 @@ Adding batch files for any executables found to a location on PATH. In other wor
       foreach ($file in $files) {
         if (!(test-path($file.FullName + '.ignore'))) {
           if (test-path($file.FullName + '.gui')) {
-            Generate-BinFile $file.Name.Replace(".exe","").Replace(".EXE","") $file.FullName -useStart
+            if ($uninstall) {
+              Remove-BinFile $file.Name.Replace(".exe","").Replace(".EXE","") $file.FullName
+            } else {
+              Generate-BinFile $file.Name.Replace(".exe","").Replace(".EXE","") $file.FullName -useStart
+            }
           } else {
-            Generate-BinFile $file.Name.Replace(".exe","").Replace(".EXE","") $file.FullName
+            if ($uninstall) {
+              Remove-BinFile $file.Name.Replace(".exe","").Replace(".EXE","") $file.FullName
+            } else {
+              Generate-BinFile $file.Name.Replace(".exe","").Replace(".EXE","") $file.FullName
+            }
           }
           $batchCreated = $true
         }

src/functions/Remove-BinFile.ps1

@@ -0,0 +1,27 @@
+function Remove-BinFile {
+param(
+  [string] $name, 
+  [string] $path
+)
+  Write-Debug "Running 'Remove-BinFile' for $name with path:`'$path`'";
+
+  $packageBatchFileName = Join-Path $nugetExePath "$name.bat"
+  $packageBashFileName = Join-Path $nugetExePath "$name"
+  $path = $path.ToLower().Replace($nugetPath.ToLower(), "%DIR%..\").Replace("\\","\")
+  $pathBash = $path.Replace("%DIR%..\","`$DIR/../").Replace("\","/")
+  Write-Debug "Attempting to remove the batch and bash shortcuts: $packageBatchFileName and $packageBashFileName"
+  if (Test-Path $packageBatchFileName) {
+    Write-Host "Removing batch file $packageBatchFileName which pointed to `'$path`'." -ForegroundColor $Note
+    Remove-Item $packageBatchFileName
+  }
+  else {
+    Write-Host "Tried to remove batch file $packageBatchFileName but it was already removed." -ForegroundColor $Note
+  }
+  if (Test-Path $packageBashFileName) {
+    Write-Host "Removing bash file $packageBashFileName which pointed to `'$path`'." -ForegroundColor $Note
+    Remove-Item $packageBashFileName
+  }
+  else {
+    Write-Host "Tried to remove bash file $packageBashFileName but it was already removed." -ForegroundColor $Note
+  }
+}