From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 From: "Jan Alexander Steffens (heftig)" Date: Tue, 27 Oct 2020 19:50:30 +0000 Subject: [PATCH] screen: Manually handle env inheritance Don't let VTE handle merging the environment; build the merged block ourselves. This avoids confusion when we reuse the exec data. https://gitlab.gnome.org/GNOME/gnome-terminal/-/issues/303 --- src/terminal-screen.c | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/src/terminal-screen.c b/src/terminal-screen.c index f44de43a..570d4151 100644 --- a/src/terminal-screen.c +++ b/src/terminal-screen.c @@ -943,9 +943,7 @@ terminal_screen_exec (TerminalScreen *screen, &shell); gboolean preserve_cwd = FALSE; - GSpawnFlags spawn_flags = G_SPAWN_SEARCH_PATH_FROM_ENVP; - if (initial_envv) - spawn_flags |= VTE_SPAWN_NO_PARENT_ENVV; + GSpawnFlags spawn_flags = G_SPAWN_SEARCH_PATH_FROM_ENVP | VTE_SPAWN_NO_PARENT_ENVV; gs_strfreev char **exec_argv = NULL; if (!terminal_screen_get_child_command (screen, argv, @@ -1432,25 +1430,27 @@ terminal_screen_get_child_environment (TerminalScreen *screen, { TerminalApp *app = terminal_app_get (); char **env; + gs_strfreev char **current_environ = NULL; char *e, *v; GHashTable *env_table; GHashTableIter iter; GPtrArray *retval; guint i; env_table = g_hash_table_new_full (g_str_hash, g_str_equal, g_free, g_free); - env = initial_envv; - if (env) + if (initial_envv) + env = initial_envv; + else + env = current_environ = g_get_environ (); + + for (i = 0; env[i]; ++i) { - for (i = 0; env[i]; ++i) - { - v = strchr (env[i], '='); - if (v) - g_hash_table_replace (env_table, g_strndup (env[i], v - env[i]), g_strdup (v + 1)); - else - g_hash_table_replace (env_table, g_strdup (env[i]), NULL); - } + v = strchr (env[i], '='); + if (v) + g_hash_table_replace (env_table, g_strndup (env[i], v - env[i]), g_strdup (v + 1)); + else + g_hash_table_replace (env_table, g_strdup (env[i]), NULL); } g_hash_table_remove (env_table, "COLUMNS");