From 79d799a99d5fc3b9caba972079eb7804e0c61129 Mon Sep 17 00:00:00 2001 From: agnostic-apollo Date: Mon, 13 Jun 2022 16:11:13 +0500 Subject: [PATCH] Fixed: Fix `ExecutionCommand.pid` not being set for first and background terminal sessions since `TermuxTerminalSessionClientBase` was still being used instead of `TermuxTerminalSessionActivityClient` This commit adds onto 841c41bf and implements the `setTerminalShellPid()` interface method in `TermuxTerminalSessionServiceClient` so that `pid` is set properly for all cases. --- .../app/terminal/TermuxTerminalSessionServiceClient.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/app/src/main/java/com/termux/app/terminal/TermuxTerminalSessionServiceClient.java b/app/src/main/java/com/termux/app/terminal/TermuxTerminalSessionServiceClient.java index 6941a348..e943bb3a 100644 --- a/app/src/main/java/com/termux/app/terminal/TermuxTerminalSessionServiceClient.java +++ b/app/src/main/java/com/termux/app/terminal/TermuxTerminalSessionServiceClient.java @@ -21,4 +21,11 @@ public class TermuxTerminalSessionServiceClient extends TermuxTerminalSessionCli this.mService = service; } + @Override + public void setTerminalShellPid(@NonNull TerminalSession terminalSession, int pid) { + TermuxSession termuxSession = mService.getTermuxSessionForTerminalSession(terminalSession); + if (termuxSession != null) + termuxSession.getExecutionCommand().mPid = pid; + } + }