quickshell/quickshell
1
1
Fork 2
Code Issues Releases Wiki Activity

io/process: support commands at file:// and root:// paths.

Browse source
This commit is contained in:
outfoxxed 2024-11-17 00:47:22 -08:00
parent 60dfa67ec7
commit 0445eee33a
Signed by: outfoxxed
GPG key ID: 4C88A185FB89301E
2 changed files with 19 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/io/process.cpp
View file

@ -12,6 +12,7 @@
#include <qtypes.h>
#include <qvariant.h>
#include "../core/generation.hpp"
#include "../core/qmlglobal.hpp"
#include "datastream.hpp"
@ -53,6 +54,16 @@ QList<QString> Process::command() const { return this->mCommand; }
void Process::setCommand(QList<QString> command) {
if (this->mCommand == command) return;
this->mCommand = std::move(command);
auto& cmd = this->mCommand.first();
if (cmd.startsWith("file://")) {
cmd = cmd.sliced(7);
} else if (cmd.startsWith("root://")) {
cmd = cmd.sliced(7);
auto& root = EngineGeneration::findObjectGeneration(this)->rootPath;
cmd = root.filePath(cmd.startsWith('/') ? cmd.sliced(1) : cmd);
}
emit this->commandChanged();
this->startProcessIfReady();

8
src/io/process.hpp
View file

@ -51,6 +51,14 @@ class Process: public QObject {
/// started process. If the property has been changed after starting a process it will
/// return the new value, not the one for the currently running process.
///
/// > [!WARNING] This does not run command in a shell. All arguments to the command
/// > must be in separate values in the list, e.g. `["echo", "hello"]`
/// > and not `["echo hello"]`.
/// >
/// > Additionally, shell scripts must be run by your shell,
/// > e.g. `["sh", "script.sh"]` instead of `["script.sh"]` unless the script
/// > has a shebang.
///
/// > [!INFO] You can use `["sh", "-c", <your command>]` to execute your command with
/// > the system shell.
Q_PROPERTY(QList<QString> command READ command WRITE setCommand NOTIFY commandChanged);