Optionally enable backtick command substitution for pickRemoteProcess

Extension/package.json

@@ -539,6 +539,12 @@
           "default": true,
           "description": "%c_cpp.configuration.renameRequiresIdentifier.description%",
           "scope": "application"
+        },
+        "C_Cpp.debugger.useBacktickCommandSubstitution": {
+          "type": "boolean",
+          "default": false,
+          "description": "%c_cpp.configuration.debugger.useBacktickCommandSubstitution.description%",
+          "scope": "window"
         }
       }
     },

Extension/package.nls.json

@@ -66,6 +66,7 @@
     "c_cpp.configuration.enhancedColorization.description": "If enabled, code is colorized based on IntelliSense. This setting has no effect if IntelliSense is disabled or if using the Default High Contrast theme.",
     "c_cpp.configuration.vcpkg.enabled.markdownDescription": "Enable integration services for the [vcpkg dependency manager](https://aka.ms/vcpkg/).",
     "c_cpp.configuration.renameRequiresIdentifier.description": "If true, 'Rename Symbol' will require a valid C/C++ identifier.",
+    "c_cpp.configuration.debugger.useBacktickCommandSubstitution.description": "If true, debugger shell command substitution will use obsolete backtick (`)",
     "c_cpp.contributes.views.cppReferencesView.title": "C/C++: Other references results",
     "c_cpp.contributes.viewsContainers.activitybar.CppRenameActivityBar.title": "C/C++ Rename",
     "c_cpp.contributes.views.cppRenamePendingView.title": "PENDING RENAME",

Extension/src/Debugger/attachToProcess.ts

@@ -5,6 +5,7 @@
 
 import { PsProcessParser } from './nativeAttach';
 import { AttachItem, showQuickPick } from './attachQuickPick';
+import { CppSettings } from '../LanguageServer/settings';
 
 import * as debugUtils from './utils';
 import * as fs from 'fs';
@@ -108,6 +109,16 @@ export class RemoteAttachPicker {
     private getRemoteProcessCommand(): string {
         let innerQuote: string = `'`;
         let outerQuote: string = `"`;
+        let parameterBegin: string = `$(`;
+        let parameterEnd: string = `)`;
+        let escapedQuote: string = `\\\"`;
+
+        let settings: CppSettings = new CppSettings();
+        if (settings.useBacktickCommandSubstitution) {
+            parameterBegin = `\``;
+            parameterEnd = `\``;
+            escapedQuote = `\"`;
+        }
 
         // Must use single quotes around the whole command and double quotes for the argument to `sh -c` because Linux evaluates $() inside of double quotes.
         // Having double quotes for the outerQuote will have $(uname) replaced before it is sent to the remote machine.
@@ -116,7 +127,8 @@ export class RemoteAttachPicker {
             outerQuote = `'`;
         }
 
-        return `${outerQuote}sh -c ${innerQuote}uname && if [ $(uname) = \\\"Linux\\\" ] ; then ${PsProcessParser.psLinuxCommand} ; elif [ $(uname) = \\\"Darwin\\\" ] ; ` +
+        return `${outerQuote}sh -c ${innerQuote}uname && if [ ${parameterBegin}uname${parameterEnd} = ${escapedQuote}Linux${escapedQuote} ] ; ` +
+        `then ${PsProcessParser.psLinuxCommand} ; elif [ ${parameterBegin}uname${parameterEnd} = ${escapedQuote}Darwin${escapedQuote} ] ; ` +
         `then ${PsProcessParser.psDarwinCommand}; fi${innerQuote}${outerQuote}`;
     }
 

Extension/src/LanguageServer/settings.ts

@@ -155,6 +155,7 @@ export class CppSettings extends Settings {
     public get defaultLimitSymbolsToIncludedHeaders(): boolean | undefined { return super.Section.get<boolean>("default.browse.limitSymbolsToIncludedHeaders"); }
     public get defaultSystemIncludePath(): string[] | undefined { return super.Section.get<string[]>("default.systemIncludePath"); }
     public get defaultEnableConfigurationSquiggles(): boolean | undefined { return super.Section.get<boolean>("default.enableConfigurationSquiggles"); }
+    public get useBacktickCommandSubstitution(): boolean | undefined { return super.Section.get<boolean>("debugger.useBacktickCommandSubstitution"); }
 
     public get enhancedColorization(): boolean {
         return super.Section.get<string>("enhancedColorization") === "Enabled"