Remove async promise usage in externalTerminalService

Part of #134873
This commit is contained in:
Daniel Imms 2021-10-12 06:56:48 -07:00
parent f1449c4cdc
commit bb7d9597f5

View file

@ -273,10 +273,11 @@ export class LinuxExternalTerminalService extends ExternalTerminalService implem
public static async getDefaultTerminalLinuxReady(): Promise<string> {
if (!LinuxExternalTerminalService._DEFAULT_TERMINAL_LINUX_READY) {
// eslint-disable-next-line no-async-promise-executor
LinuxExternalTerminalService._DEFAULT_TERMINAL_LINUX_READY = new Promise(async r => {
if (env.isLinux) {
const isDebian = await pfs.Promises.exists('/etc/debian_version');
if (!env.isLinux) {
LinuxExternalTerminalService._DEFAULT_TERMINAL_LINUX_READY = Promise.resolve('xterm');
} else {
const isDebian = await pfs.Promises.exists('/etc/debian_version');
LinuxExternalTerminalService._DEFAULT_TERMINAL_LINUX_READY = new Promise<string>(r => {
if (isDebian) {
r('x-terminal-emulator');
} else if (process.env.DESKTOP_SESSION === 'gnome' || process.env.DESKTOP_SESSION === 'gnome-classic') {
@ -290,10 +291,8 @@ export class LinuxExternalTerminalService extends ExternalTerminalService implem
} else {
r('xterm');
}
} else {
r('xterm');
}
});
});
}
}
return LinuxExternalTerminalService._DEFAULT_TERMINAL_LINUX_READY;
}