Makes the kill happen properly before deploying

Stops double code running, and closes #22

Author: ThadHouse

FRC-Extension/RoboRIOCode/DeployManager.cs

@@ -312,17 +312,22 @@ public async Task UploadCode(string robotName, SettingsPageGrid page, bool debug
 
             string args = GetCommandLineArguments(robotProject);
 
-            //Must be run as admin, so is seperate
-            await RoboRIOConnection.RunCommand("killall -q netconsole-host || :", ConnectionUser.Admin);
+            //Kill the currently running robot program
+            await RoboRIOConnection.RunCommand(DeployProperties.KillOnlyCommand, ConnectionUser.LvUser);
 
             //Combining all other commands, since they should be safe running together.
             List<string> commands = new List<string>();
+
+            //Write the robotCommand file
             commands.Add($"echo {deployedCmd} {args} > {DeployProperties.CommandDir}/{deployedCmdFrame}");
             if (debug)
             {
+                //If debug write the debug flag.
                 commands.AddRange(DeployProperties.DebugFlagCommand);
             }
+            //Add all commands to restart
             commands.AddRange(DeployProperties.DeployKillCommand);
+            //run all commands
             await RoboRIOConnection.RunCommands(commands.ToArray(), ConnectionUser.LvUser);
 
             //Run sync so files are written to disk.

FRC-Extension/RoboRIOCode/DeployProperties.cs

@@ -16,7 +16,7 @@ public static class DeployProperties
             ". /etc/profile.d/natinst-path.sh; /usr/local/frc/bin/frcKillRobot.sh -t -r;",
         };
 
-        public const string KillOnlyCommand = "/usr/local/frc/bin/frcKillRobot.sh -t";
+        public const string KillOnlyCommand = ". /etc/profile.d/natinst-path.sh; /usr/local/frc/bin/frcKillRobot.sh -t";
 
         public const string DebugFlagDir = "/tmp/";