Skip to content

Commit

Permalink
screen: Stop setting COLORTERM env var
Browse files Browse the repository at this point in the history
COLORTERM is a long-obsolete slang-only variable used to work around
broken termcap/terminfo entries.
  • Loading branch information
Christian Persch committed Apr 26, 2014
1 parent 7da07d5 commit 1d5c1b6
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 3 deletions.
1 change: 0 additions & 1 deletion src/Makefile.am
Expand Up @@ -84,7 +84,6 @@ nodist_gnome_terminal_server_SOURCES = $(gnome_terminal_server_built)

gnome_terminal_server_CPPFLAGS = \
-DTERMINAL_COMPILATION \
-DEXECUTABLE_NAME=\"gnome-terminal\" \
-DTERM_LOCALEDIR="\"$(datadir)/locale\"" \
-DTERM_HELPDIR="\"$(HELP_DIR)\"" \
-DTERM_LIBEXECDIR="\"$(libexecdir)\"" \
Expand Down
2 changes: 0 additions & 2 deletions src/terminal-screen.c
Expand Up @@ -1084,8 +1084,6 @@ get_child_environment (TerminalScreen *screen,
g_hash_table_remove (env_table, "LINES");
g_hash_table_remove (env_table, "GNOME_DESKTOP_ICON");

g_hash_table_replace (env_table, g_strdup ("COLORTERM"), g_strdup (EXECUTABLE_NAME));

#ifdef GDK_WINDOWING_X11
if (GDK_IS_X11_SCREEN (gtk_widget_get_screen (window)))
{
Expand Down

2 comments on commit 1d5c1b6

@DipSwitch
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This variable is used by whole the world to set TERM to xterm-256color how do I check this now?

@egmontkob
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In https://bugs.launchpad.net/ubuntu/+source/gnome-terminal/+bug/1429584 it seems that you yourself have found the links that point to the answer :)

Please sign in to comment.