Summary: | External Tools / Run Shell Script : does not work | ||
---|---|---|---|
Product: | [Applications] kate | Reporter: | remi <remiforkdebugreport> |
Component: | general | Assignee: | KWrite Developers <kwrite-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | minor | ||
Priority: | NOR | ||
Version First Reported In: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Flatpak | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/utilities/kate/-/commit/33ca1767dde9bec02e2f0fc49d6b617a855cfc42 | Version Fixed In: | |
Sentry Crash Report: |
Description
remi
2023-10-25 13:55:53 UTC
Some additional questions : Is it reasonable to change the file permission (the 'chmod' part) by default ? What's the goal of 'pwd' in the command ? The default command has been introduced in this commit : https://invent.kde.org/utilities/kate/-/commit/0d1c47d198f6a3220fb10973635689416feab4e9 Reply to my previous comment : 'pwd' is useful, otherwise the konsole window does not show the path A possibly relevant merge request was started @ https://invent.kde.org/utilities/kate/-/merge_requests/1330 Git commit 33ca1767dde9bec02e2f0fc49d6b617a855cfc42 by Christoph Cullmann, on behalf of Rémi Peuchot. Committed on 25/10/2023 at 21:52. Pushed by cullmann into branch 'master'. Fix run_shell_script default command arguments run_shell_script default command argument contains an invalid 'sh' command : read -n 1 Which returns the following error : -n: 1: read: arg count The 'sh' read command syntax seems to be : read my_variable letting the user to enter a value and press enter, so we can use $my_variable later. This fix consists of replacing the read command by a valid one : read null Which let the konsole window opened until the user press enter. M +1 -1 addons/externaltools/externaltools-config/run_shell_script https://invent.kde.org/utilities/kate/-/commit/33ca1767dde9bec02e2f0fc49d6b617a855cfc42 |