diff mbox

[1/1] xterm: add support for 256 colors

Message ID 1418137759-12514-1-git-send-email-tbultel@free.fr
State Accepted
Headers show

Commit Message

Thierry Bultel Dec. 9, 2014, 3:09 p.m. UTC
Signed-off-by: Thierry Bultel <tbultel@free.fr>
---
 package/xterm/Config.in | 9 +++++++++
 package/xterm/xterm.mk  | 4 ++++
 2 files changed, 13 insertions(+)

Comments

Thomas Petazzoni Feb. 3, 2015, 4:58 p.m. UTC | #1
Dear Thierry Bultel,

On Tue,  9 Dec 2014 16:09:19 +0100, Thierry Bultel wrote:
> Signed-off-by: Thierry Bultel <tbultel@free.fr>
> ---
>  package/xterm/Config.in | 9 +++++++++
>  package/xterm/xterm.mk  | 4 ++++
>  2 files changed, 13 insertions(+)

Adding a condition for such a small feature in a package that anyway
has a huge dependency such as X.org didn't seem really useful, so I've
dropped the additional config option, and just enabled 256 colors
support unconditionally.

Thanks!

Thomas
diff mbox

Patch

diff --git a/package/xterm/Config.in b/package/xterm/Config.in
index e17d2c6..7dd4311 100644
--- a/package/xterm/Config.in
+++ b/package/xterm/Config.in
@@ -8,3 +8,12 @@  config BR2_PACKAGE_XTERM
 	  xterm terminal emulator
 
 	  http://invisible-island.net/xterm/
+
+if BR2_PACKAGE_XTERM
+
+config BR2_PACKAGE_XTERM_256_COLOR
+	bool "xterm 256 colors"
+	help
+	   enable 256-color support
+	
+endif
diff --git a/package/xterm/xterm.mk b/package/xterm/xterm.mk
index 9c6bd21..11d6cf4 100644
--- a/package/xterm/xterm.mk
+++ b/package/xterm/xterm.mk
@@ -12,4 +12,8 @@  XTERM_INSTALL_TARGET_OPTS = DESTDIR=$(TARGET_DIR) install
 XTERM_LICENSE = MIT
 XTERM_LICENSE_FILES = version.c
 
+ifeq ($(BR2_PACKAGE_XTERM_256_COLOR),y)
+XTERM_CONF_OPTS += --enable-256-color
+endif
+
 $(eval $(autotools-package))