CodeMirror/xpipe
1
0
Fork 0
mirror of https://github.com/xpipe-io/xpipe.git synced 2024-07-01 04:21:11 +12:00
Code Issues Packages Projects Releases Wiki Activity

Attempt to prevent double init file execution when terminal opens

Browse source
This commit is contained in:
crschnick 2023-05-11 09:53:54 +00:00
parent 2c9d26f65f
commit d3485e9f6d
3 changed files with 21 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

29
app/src/main/java/io/xpipe/app/prefs/ExternalTerminalType.java
View file

@ -30,12 +30,12 @@ public interface ExternalTerminalType extends PrefsChoiceValue {
}
};
public static final ExternalTerminalType POWERSHELL =
public static final ExternalTerminalType POWERSHELL_WINDOWS =
new SimpleType("powershell", "powershell", "PowerShell") {
@Override
protected String toCommand(String name, String file) {
return "-ExecutionPolicy Bypass -Command cmd /C '" + file + "'";
return "-ExecutionPolicy Bypass -NoProfile -Command " + noInit(file);
}
@Override
@ -44,13 +44,13 @@ public interface ExternalTerminalType extends PrefsChoiceValue {
}
};
public static final ExternalTerminalType PWSH = new SimpleType("pwsh", "pwsh", "PowerShell Core") {
public static final ExternalTerminalType PWSH_WINDOWS = new SimpleType("pwsh", "pwsh", "PowerShell Core") {
@Override
protected String toCommand(String name, String file) {
// Fix for https://github.com/PowerShell/PowerShell/issues/18530#issuecomment-1325691850
var script = ScriptHelper.createLocalExecScript("set \"PSModulePath=\"\r\n\"" + file + "\"\npause");
return "-ExecutionPolicy Bypass -Command cmd /C '" +script + "'";
var script = ScriptHelper.createLocalExecScript("set \"PSModulePath=\"\r\n\"" + noInit(file) + "\"\npause");
return "-ExecutionPolicy Bypass -NoProfile -Command " + noInit(script);
}
@Override
@ -68,7 +68,7 @@ public interface ExternalTerminalType extends PrefsChoiceValue {
// backslash of a filepath to escape the closing quote in the title argument
// So just remove that slash
var fixedName = FileNames.removeTrailingSlash(name);
return "-w 1 nt --title \"" + fixedName + "\" \"" + file + "\"";
return "-w 1 nt --title \"" + fixedName + "\" " + noInit(file);
}
@Override
@ -89,7 +89,7 @@ public interface ExternalTerminalType extends PrefsChoiceValue {
// In order to fix this bug which also affects us:
// https://askubuntu.com/questions/1148475/launching-gnome-terminal-from-vscode
toExecute =
"GNOME_TERMINAL_SCREEN=\"\" nohup " + toExecute + " </dev/null &>/dev/null & disown";
"GNOME_TERMINAL_SCREEN=\"\" nohup " + noInit(file) + " </dev/null &>/dev/null & disown";
pc.executeSimpleCommand(toExecute);
}
}
@ -109,7 +109,7 @@ public interface ExternalTerminalType extends PrefsChoiceValue {
@Override
protected String toCommand(String name, String file) {
return "--new-tab -e \"" + file + "\"";
return "--new-tab -e " + noInit(file);
}
@Override
@ -122,7 +122,7 @@ public interface ExternalTerminalType extends PrefsChoiceValue {
@Override
protected String toCommand(String name, String file) {
return "--tab --title \"" + name + "\" --command \"" + file + "\"";
return "--tab --title \"" + name + "\" --command " + noInit(file);
}
@Override
@ -141,8 +141,8 @@ public interface ExternalTerminalType extends PrefsChoiceValue {
public static final List<ExternalTerminalType> ALL = Stream.of(
WINDOWS_TERMINAL,
PWSH,
POWERSHELL,
PWSH_WINDOWS,
POWERSHELL_WINDOWS,
CMD,
KONSOLE,
XFCE,
@ -164,6 +164,10 @@ public interface ExternalTerminalType extends PrefsChoiceValue {
public abstract void launch(String name, String file, boolean elevated) throws Exception;
default String noInit(String file) {
return ShellDialects.getPlatformDefault().executeWithNoInitFiles(ShellDialects.getPlatformDefault(), file);
}
static class MacOsTerminalType extends ExternalApplicationType.MacApplication implements ExternalTerminalType {
public MacOsTerminalType() {
@ -232,6 +236,7 @@ public interface ExternalTerminalType extends PrefsChoiceValue {
@Override
public void launch(String name, String file, boolean elevated) throws Exception {
var toExecute = noInit(file).replaceAll("\"", "\\\\\"");
try (ShellControl pc = LocalStore.getShell()) {
var cmd = String.format(
"""
@ -252,7 +257,7 @@ public interface ExternalTerminalType extends PrefsChoiceValue {
end tell
end if
EOF""",
file.replaceAll("\"", "\\\\\""), file.replaceAll("\"", "\\\\\""));
toExecute, toExecute);
pc.executeSimpleCommand(cmd);
}
}

6
app/src/main/java/io/xpipe/app/util/TerminalHelper.java
View file

@ -14,15 +14,13 @@ public class TerminalHelper {
}
public static void open(String title, String command) throws Exception {
if (command.contains("\n") || command.contains(" ") || command.contains("\"") || command.contains("'")) {
command = ScriptHelper.createLocalExecScript(command);
}
var type = AppPrefs.get().terminalType().getValue();
if (type == null) {
throw new IllegalStateException(AppI18n.get("noTerminalSet"));
}
command = ScriptHelper.createLocalExecScript(command);
try {
type.launch(title, command, false);
} catch (Exception ex) {

2
core/src/main/java/io/xpipe/core/process/ShellDialect.java
View file

@ -20,6 +20,8 @@ public interface ShellDialect {
String fileArgument(String s);
String executeWithNoInitFiles(ShellDialect parentDialect, String file);
void prepareDumbTerminalCommands(ShellControl sc) throws Exception;
String prepareProperTerminalCommands();