fix[#23] detect failure and report to user

[jardon]

• repurpose SideloaderInstallDone to handle failure
• remove dedicated fail ui
  

closes #23

[jardon]

i dont think this can be done until this gets fixed. since vso always reports a success, the vte terminal status is inaccurate.

[jardon]

This may be sufficient as-is, but i need to test it more. If the exit code is reported correctly, it should work as expected.

[jardon]

@mirkobrombin do you know if the VTE terminal checks in the users $PATH for the subprocess? i have an upstream change from vso installed locally in ~/.local, but it appears its returning a 0 status code.

[mirkobrombin]

VTE itself does not, it expect PATH to be set, which I think is the case. What happen if you hard code a echo $PATH and run VTE?

[jardon]

this does behave correctly with the upstream fix. ive rebased and it should be ready to merge, but it wont display an error was detected until that vso change is included in a new release.

[mirkobrombin]

LGTM