Unnamed repository; edit this file 'description' to name the repository.
editors/code: fix crash due to missing ID= field
Assuming ID=linux in isNixOs by default. You can get away with default "", but why do that if there's a default value in spec?) Also removed toLowerCase — it really shouldn't be needed. Fixes #11709
Vladimir Serov 2022-03-15
parent 683fea4 · commit f7cf3b5
-rw-r--r--editors/code/src/main.ts4
1 files changed, 2 insertions, 2 deletions
diff --git a/editors/code/src/main.ts b/editors/code/src/main.ts
index 4856079f68..06e5c1185d 100644
--- a/editors/code/src/main.ts
+++ b/editors/code/src/main.ts
@@ -269,8 +269,8 @@ function serverPath(config: Config): string | null {
async function isNixOs(): Promise<boolean> {
try {
const contents = (await vscode.workspace.fs.readFile(vscode.Uri.file("/etc/os-release"))).toString();
- const idString = contents.split('\n').find((a) => a.startsWith("ID="));
- return idString?.toLowerCase()?.indexOf("nixos") !== -1;
+ const idString = contents.split('\n').find((a) => a.startsWith("ID=")) || "ID=linux";
+ return idString.indexOf("nixos") !== -1;
} catch {
return false;
}