aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRob Landley <rob@landley.net>2020-05-04 02:27:57 -0500
committerRob Landley <rob@landley.net>2020-05-04 02:27:57 -0500
commit81c1aa3c1d67f7645fda9dddf668321beae302e6 (patch)
tree640d8ca290711f2a5004d583c43b303ed62e22ec
parent729c4b2bfdf9f0551fd231aafe28a7b9c69e9a82 (diff)
downloadtoybox-81c1aa3c1d67f7645fda9dddf668321beae302e6.tar.gz
Restore error message for command not found, don't let ctrl-C kill shell.
-rw-r--r--toys/pending/sh.c33
1 files changed, 23 insertions, 10 deletions
diff --git a/toys/pending/sh.c b/toys/pending/sh.c
index fe5ed195..79509084 100644
--- a/toys/pending/sh.c
+++ b/toys/pending/sh.c
@@ -1477,10 +1477,12 @@ static void sh_exec(char **argv)
char *pp = getvar("PATH" ? : _PATH_DEFPATH), *cc = TT.isexec ? : *argv;
struct string_list *sl;
+ if (getpid() != TT.pid) signal(SIGINT, SIG_DFL);
if (strchr(cc, '/')) shexec(cc, argv);
else for (sl = find_in_path(pp, cc); sl; free(llist_pop(&sl)))
shexec(sl->str, argv);
+ perror_msg("%s", *argv);
if (!TT.isexec) _exit(127);
}
@@ -2566,6 +2568,9 @@ if (BUGBUG) { int fd = open("/dev/tty", O_RDWR); if (fd == -1) fd = open("/dev/c
if (TT.options&OPT_I) {
if (!getvar("PS1")) setvarval("PS1", getpid() ? "\\$ " : "# ");
// TODO Set up signal handlers and grab control of this tty.
+ // ^C SIGINT ^\ SIGQUIT ^Z SIGTSTP SIGTTIN SIGTTOU SIGCHLD
+ // setsid(), setpgid(), tcsetpgrp()...
+ xsignal(SIGINT, SIG_IGN);
}
memset(&scratch, 0, sizeof(scratch));
@@ -2573,16 +2578,14 @@ if (BUGBUG) { int fd = open("/dev/tty", O_RDWR); if (fd == -1) fd = open("/dev/c
// TODO unify fmemopen() here with sh_run
if (cc) f = fmemopen(cc, strlen(cc), "r");
else if (TT.options&OPT_S) f = stdin;
- else {
// TODO: syntax_err should exit from shell scripts
- if (!(f = fopen(*toys.optargs, "r"))) {
- char *pp = getvar("PATH") ? : _PATH_DEFPATH;
+ else if (!(f = fopen(*toys.optargs, "r"))) {
+ char *pp = getvar("PATH") ? : _PATH_DEFPATH;
- for (sl = find_in_path(pp, *toys.optargs); sl; free(llist_pop(&sl)))
- if ((f = fopen(sl->str, "r"))) break;
- if (sl) llist_traverse(sl->next, free);
- else perror_exit_raw(*toys.optargs);
- }
+ for (sl = find_in_path(pp, *toys.optargs); sl; free(llist_pop(&sl)))
+ if ((f = fopen(sl->str, "r"))) break;
+ if (sl) llist_traverse(sl->next, free);
+ else perror_exit_raw(*toys.optargs);
}
// Loop prompting and reading lines
@@ -2592,7 +2595,17 @@ if (BUGBUG) { int fd = open("/dev/tty", O_RDWR); if (fd == -1) fd = open("/dev/c
do_prompt(getvar(prompt ? "PS2" : "PS1"));
// TODO line editing/history, should set $COLUMNS $LINES and sigwinch update
- if (!(new = xgetline(f, 0))) break;
+ if (!(new = xgetline(f, 0))) {
+// TODO: after first EINTR getline returns always closed?
+ if (errno != EINTR) break;
+ free_function(&scratch);
+ prompt = 0;
+ if (f != stdin) break;
+ continue;
+// TODO: ctrl-z during script read having already read partial line,
+// SIGSTOP and SIGTSTP need need SA_RESTART, but child proc should stop
+ }
+
if (BUGBUG) dprintf(255, "line=%s\n", new);
if (sl) {
if (*new == 0x7f) error_exit("'%s' is ELF", sl->str);
@@ -2767,12 +2780,12 @@ void exec_main(void)
if (TT.exec.a || FLAG(l))
*toys.optargs = xmprintf("%s%s", FLAG(l) ? "-" : "", TT.exec.a?:TT.isexec);
sh_exec(toys.optargs);
+ TT.isexec = 0;
// report error (usually ENOENT) and return
perror_msg("%s", TT.isexec);
toys.exitval = 127;
environ = old;
- TT.isexec = 0;
}
void shift_main(void)