From 7d9ee3a1276cf744d1a5ecd0e1bc53d61dc05935 Mon Sep 17 00:00:00 2001 From: 0neGal Date: Tue, 30 Apr 2024 21:54:01 +0200 Subject: [PATCH] fixed killing EA Desktop at times failing on Linux Pesky EA Desktop being annoying as always, leaving behind processes, to be fair, I don't exactly think you're supposed to be killing it it like this, but oh well, it seems to function now. --- src/modules/is_running.js | 4 +++- src/modules/kill.js | 6 ++++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/src/modules/is_running.js b/src/modules/is_running.js index 9e45574c..e2a2d965 100644 --- a/src/modules/is_running.js +++ b/src/modules/is_running.js @@ -53,7 +53,9 @@ is_running.game = () => { is_running.origin = () => { return check_processes([ "Origin.exe", - "EADesktop.exe" + "EADesktop.exe", + "CrBrowserMain", + "EABackgroundSer" ]) } diff --git a/src/modules/kill.js b/src/modules/kill.js index 43eda23c..5d03218e 100644 --- a/src/modules/kill.js +++ b/src/modules/kill.js @@ -33,6 +33,12 @@ kill.origin = async () => { let origin = await kill("Origin.exe"); let eadesktop = await kill("EADesktop.exe"); + // these should be Linux only, and the above shouldn't succeed if + // these don't succeed, so we shouldn't have to check whether these + // actually succeeded or not + await kill("CrBrowserMain"); + await kill("EABackgroundSer"); + if (origin || eadesktop) { return true; }