From e3aa71c38cbecb24e6333411ee19814796a5b1d0 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Mon, 15 Nov 2010 23:49:02 +0100 Subject: [PATCH] exec: automatically determine right TERM= setting based on tty name --- TODO | 2 +- src/execute.c | 12 ++++++++++-- src/service.c | 10 ---------- src/test-strv.c | 13 +++++++++++++ src/util.c | 15 +++++++++++++++ src/util.h | 2 ++ units/emergency.service | 1 - units/fedora/halt-local.service | 1 - units/fedora/rc-local.service | 1 - units/getty@.service.m4 | 1 - units/rescue.service.m4 | 1 - units/serial-getty@.service.m4 | 1 - 12 files changed, 41 insertions(+), 19 deletions(-) diff --git a/TODO b/TODO index 945f8d998..8853dab24 100644 --- a/TODO +++ b/TODO @@ -73,7 +73,7 @@ * in the PAM module rely on loginuid to figure out XDG_RUNTIME_DIR -* automatically determine TERM= based on tty name. (TERM=linux vs. TERM=vt100-nav) +* automatically determine TERM= based on tty name even for /dev/console * declare /etc/os-release cross-distro standard diff --git a/src/execute.c b/src/execute.c index 48e55ea4c..05abd5aaa 100644 --- a/src/execute.c +++ b/src/execute.c @@ -1252,7 +1252,7 @@ int exec_spawn(ExecCommand *command, } } - if (!(our_env = new0(char*, 6))) { + if (!(our_env = new0(char*, 7))) { r = EXIT_MEMORY; goto fail; } @@ -1277,7 +1277,15 @@ int exec_spawn(ExecCommand *command, goto fail; } - assert(n_env <= 6); + if (is_terminal_input(context->std_input) || + context->std_output == EXEC_OUTPUT_TTY || + context->std_error == EXEC_OUTPUT_TTY) + if (!(our_env[n_env++] = strdup(default_term_for_tty(tty_path(context))))) { + r = EXIT_MEMORY; + goto fail; + } + + assert(n_env <= 7); if (!(final_env = strv_env_merge( 4, diff --git a/src/service.c b/src/service.c index 184ddf914..023415170 100644 --- a/src/service.c +++ b/src/service.c @@ -1609,16 +1609,6 @@ static int service_spawn( goto fail; } -#ifdef HAVE_SYSV_COMPAT - /* Make sure we set TERM=linux for SysV scripts, since some - * require it to be set from the kernel */ - if (s->sysv_path && !strv_env_get(s->meta.manager->environment, "TERM")) - if (!(our_env[n_env++] = strdup("TERM=linux"))) { - r = -ENOMEM; - goto fail; - } -#endif - if (!(final_env = strv_env_merge(2, s->meta.manager->environment, our_env, diff --git a/src/test-strv.c b/src/test-strv.c index 573436896..cfbf7fddc 100644 --- a/src/test-strv.c +++ b/src/test-strv.c @@ -37,5 +37,18 @@ int main(int argc, char *argv[]) { free(t); } + printf("%s\n", default_term_for_tty("/dev/tty23")); + printf("%s\n", default_term_for_tty("/dev/ttyS23")); + printf("%s\n", default_term_for_tty("/dev/tty0")); + printf("%s\n", default_term_for_tty("/dev/pty0")); + printf("%s\n", default_term_for_tty("/dev/pts/0")); + printf("%s\n", default_term_for_tty("/dev/console")); + printf("%s\n", default_term_for_tty("tty23")); + printf("%s\n", default_term_for_tty("ttyS23")); + printf("%s\n", default_term_for_tty("tty0")); + printf("%s\n", default_term_for_tty("pty0")); + printf("%s\n", default_term_for_tty("pts/0")); + printf("%s\n", default_term_for_tty("console")); + return 0; } diff --git a/src/util.c b/src/util.c index 7f9f2b36a..fb2eea341 100644 --- a/src/util.c +++ b/src/util.c @@ -3470,6 +3470,21 @@ void filter_environ(const char *prefix) { environ[j] = NULL; } +const char *default_term_for_tty(const char *tty) { + assert(tty); + + if (startswith(tty, "/dev/")) + tty += 5; + + if (startswith(tty, "tty") && + tty[3] >= '0' && tty[3] <= '9') + return "TERM=linux"; + + /* FIXME: Proper handling of /dev/console would be cool */ + + return "TERM=vt100-nav"; +} + static const char *const ioprio_class_table[] = { [IOPRIO_CLASS_NONE] = "none", [IOPRIO_CLASS_RT] = "realtime", diff --git a/src/util.h b/src/util.h index 63538fed8..7e1eacc68 100644 --- a/src/util.h +++ b/src/util.h @@ -372,6 +372,8 @@ char *fstab_node_to_udev_node(const char *p); void filter_environ(const char *prefix); +const char *default_term_for_tty(const char *tty); + #define NULSTR_FOREACH(i, l) \ for ((i) = (l); (i) && *(i); (i) = strchr((i), 0)+1) diff --git a/units/emergency.service b/units/emergency.service index 52f875c6f..aa3d9856c 100644 --- a/units/emergency.service +++ b/units/emergency.service @@ -15,7 +15,6 @@ Before=shutdown.target [Service] Environment=HOME=/root -Environment=TERM=vt100-nav WorkingDirectory=/root ExecStartPre=-/bin/plymouth --hide-splash ExecStartPre=-/bin/echo 'Welcome to emergency mode. Use "systemctl default" or ^D to activate default mode.' diff --git a/units/fedora/halt-local.service b/units/fedora/halt-local.service index 855924aec..79f8f12b7 100644 --- a/units/fedora/halt-local.service +++ b/units/fedora/halt-local.service @@ -14,7 +14,6 @@ Before=final.target [Service] Type=oneshot -Environment=TERM=linux ExecStart=/sbin/halt.local TimeoutSec=0 StandardOutput=tty diff --git a/units/fedora/rc-local.service b/units/fedora/rc-local.service index 88846c16f..a21a5578c 100644 --- a/units/fedora/rc-local.service +++ b/units/fedora/rc-local.service @@ -11,7 +11,6 @@ ConditionPathExists=/etc/rc.local [Service] Type=forking -Environment=TERM=linux ExecStart=/etc/rc.local start TimeoutSec=0 StandardOutput=tty diff --git a/units/getty@.service.m4 b/units/getty@.service.m4 index 11a71d716..d6bd9a9db 100644 --- a/units/getty@.service.m4 +++ b/units/getty@.service.m4 @@ -22,7 +22,6 @@ After=rc-local.service Before=getty.target [Service] -Environment=TERM=linux ExecStart=-/sbin/agetty %I 38400 Restart=always RestartSec=0 diff --git a/units/rescue.service.m4 b/units/rescue.service.m4 index d31282d37..6e03c20d8 100644 --- a/units/rescue.service.m4 +++ b/units/rescue.service.m4 @@ -16,7 +16,6 @@ Before=shutdown.target [Service] Environment=HOME=/root -Environment=TERM=vt100-nav WorkingDirectory=/root ExecStartPre=-/bin/plymouth --hide-splash ExecStartPre=-/bin/echo 'Welcome to rescue mode. Use "systemctl default" or ^D to activate default mode.' diff --git a/units/serial-getty@.service.m4 b/units/serial-getty@.service.m4 index da9bd1946..d454dce46 100644 --- a/units/serial-getty@.service.m4 +++ b/units/serial-getty@.service.m4 @@ -22,7 +22,6 @@ After=rc-local.service Before=getty.target [Service] -Environment=TERM=vt100-nav m4_ifdef(`TARGET_FEDORA', ExecStartPre=-/sbin/securetty %I )m4_dnl -- 2.30.2