Backed out 'monitor exit' during shutdown. Causing issues with stlink

Author: haneefdm

CHANGELOG.md

@@ -1,6 +1,8 @@
 # ChangeLog
 
 # V1.13.0-pre9
+* Backed out change where we try "monitor exit". See [Issue #1185](https://github.com/Marus/cortex-debug/issues/1185)
+* Increased server temout to 10 mins
 * Fix for https://github.com/Marus/cortex-debug/issues/1167 Race comdition in parallel reading of multiple symbol files
 
 # V1.13.0-pre8

package.json

@@ -1,5 +1,5 @@
 {
-    "version": "1.13.0-pre8",
+    "version": "1.13.0-pre9",
     "preview": false,
     "activationEvents": [
         "onDebugResolve:cortex-debug",

src/backend/mi2/mi2.ts

@@ -441,12 +441,6 @@ export class MI2 extends EventEmitter implements IBackend {
             // program is in paused state
             try {
                 startKillTimeout(500);
-                try {
-                    await this.sendCommand('interpreter-exec console "monitor exit"');
-                } catch (e) {
-                    // It is possible gdb server has not implemented this
-                    ServerConsoleLog('GDB "monitor exit" failed, continue to disconnect anyway', this.pid);
-                }
                 await new Promise(() => setTimeout(() => {}, 50));          // For some people delay was needed. Doesn't hurt I guess
                 if (!this.exited) {
                     await this.sendCommand('target-disconnect');            // Yes, this can fail

src/backend/server.ts

@@ -45,7 +45,7 @@ export class GDBServer extends EventEmitter {
     protected consoleSocket: net.Socket = null;
     private initResolve: (result: boolean) => void;
     private initReject: (error: any) => void;
-    public static readonly SERVER_TIMEOUT = 2 * 60 * 1000;
+    public static readonly SERVER_TIMEOUT = 10 * 60 * 1000;
     public static readonly LOCALHOST = '0.0.0.0';
     public pid: number = -1;