feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

LinuxDevice: Use LinuxProcessImpl for opening terminal

Browse Source 
Don't use SshDeviceProcess anymore for opening linux terminal.
Fix running the empty command so that it opens ssh shell.
Don't leak terminal processes on shutdown.

One issue is that terminals are closed when closing settings,
as settings dialog operate on a copy of device, and in case
of no modifications applied the copy is being deleted.
The current workaround is to press the "Apply" button
before closing the settings dialog. This issue is to be
addressed separately.

Change-Id: I3c48b035daf1d099a1e8fa0762a6d6d0eca9592c
Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
This commit is contained in:
Jarek Kobus
2022-04-28 21:11:20 +02:00
parent 62f6c1e1d6
commit 709492ef11
1 changed files with 18 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
src/plugins/remotelinux/linuxdevice.cpp
View File

@@ -463,6 +463,7 @@ public:
QThread m_shellThread;
ShellThreadHandler *m_handler = nullptr;
mutable QMutex m_shellMutex;
QList<QtcProcess *> m_terminals;
};
// SshProcessImpl
@@ -601,12 +602,14 @@ QString LinuxProcessImpl::fullCommandLine(const CommandLine &commandLine) const
{
CommandLine cmd;
const QStringList rcFilesToSource = {"/etc/profile", "$HOME/.profile"};
for (const QString &filePath : rcFilesToSource) {
cmd.addArgs({"test", "-f", filePath});
cmd.addArgs("&&", CommandLine::Raw);
cmd.addArgs({".", filePath});
cmd.addArgs(";", CommandLine::Raw);
if (!commandLine.isEmpty()) {
const QStringList rcFilesToSource = {"/etc/profile", "$HOME/.profile"};
for (const QString &filePath : rcFilesToSource) {
cmd.addArgs({"test", "-f", filePath});
cmd.addArgs("&&", CommandLine::Raw);
cmd.addArgs({".", filePath});
cmd.addArgs(";", CommandLine::Raw);
}
}
if (!m_setup.m_workingDirectory.isEmpty()) {
@@ -624,7 +627,8 @@ QString LinuxProcessImpl::fullCommandLine(const CommandLine &commandLine) const
if (m_setup.m_terminalMode == TerminalMode::Off)
cmd.addArg("exec");
cmd.addCommandLineAsArgs(commandLine, CommandLine::Raw);
if (!commandLine.isEmpty())
cmd.addCommandLineAsArgs(commandLine, CommandLine::Raw);
return cmd.arguments();
}
@@ -1041,8 +1045,9 @@ LinuxDevice::LinuxDevice()
}});
setOpenTerminal([this](const Environment &env, const FilePath &workingDir) {
QtcProcess * const proc = createProcess(nullptr);
QObject::connect(proc, &QtcProcess::done, [proc] {
QtcProcess * const proc = new QtcProcess;
d->m_terminals.append(proc);
QObject::connect(proc, &QtcProcess::done, [this, proc] {
if (proc->error() != QProcess::UnknownError) {
const QString errorString = proc->errorString();
QString message;
@@ -1055,15 +1060,13 @@ LinuxDevice::LinuxDevice()
Core::MessageManager::writeDisrupting(message);
}
proc->deleteLater();
d->m_terminals.removeOne(proc);
});
// It seems we cannot pass an environment to OpenSSH dynamically
// without specifying an executable.
if (env.size() > 0)
proc->setCommand({"/bin/sh", {}});
proc->setCommand({ mapToGlobalPath({}), {}});
proc->setTerminalMode(TerminalMode::On);
proc->setEnvironment(env);
proc->setRemoteEnvironment(env);
proc->setWorkingDirectory(workingDir);
proc->start();
});
@@ -1182,6 +1185,7 @@ LinuxDevicePrivate::LinuxDevicePrivate(LinuxDevice *parent)
LinuxDevicePrivate::~LinuxDevicePrivate()
{
qDeleteAll(m_terminals);
auto closeShell = [this] {
m_shellThread.quit();
m_shellThread.wait();