Make konsole -e "man ls" work as expected, while also coping with paths with spaces in

Review Request #120001 - Created Aug. 30, 2014 and submitted

Information
Mark Raymond
konsole
Reviewers
konsole

A couple of years ago, bug 295648 was fixed, introducing support for konsole -e "man ls" in commit aa75fc8f. However it introduced bug 308553, which meant that konsole -e "path with spaces in.sh" no longer worked, and so the fix was reverted in commit 48b5575c.

This patch fixes bug 295648 without reintroducing bug 308553 by checking whether the first parameter exists before interpreting using ShellCommand.

konsole -e man ls
konsole -e "man ls"

konsole -e /bin/sh ./geany_run_script.sh
konsole -e "/bin/sh ./geany_run_script.sh"

konsole -e "/home/mark/script with spaces.sh"
konsole -e /home/mark/script\ with\ spaces.sh

Mark Raymond
Mark Raymond
Review request changed

Status: Closed (submitted)

Loading...