diff mbox

menuconfig: fix extended colors ncurses support

Message ID 1349713102-3301-1-git-send-email-krzysiek@podlesie.net (mailing list archive)
State New, archived
Headers show

Commit Message

Krzysztof Mazur Oct. 8, 2012, 4:18 p.m. UTC
The ncurses library allows for extended colors. The support for extended
colors support depends on wide-character support. ncurses headers
enable extended colors (NCURSES_EXT_COLORS) only when wide-character
support is enabled (NCURSES_WIDECHAR).

The "make menuconfig" uses wide-character ncursesw library, which can be
compiled with wide-character support, but does not define NCURSES_WIDECHAR
and it's using headers without wide-character (and extended colors) support.

This fixes problems with colors on systems with enabled extended colors
(like PLD Linux). Without this patch "make menuconfig" is hard to use.

Signed-off-by: Krzysztof Mazur <krzysiek@podlesie.net>
---
 scripts/kconfig/lxdialog/check-lxdialog.sh | 1 +
 1 file changed, 1 insertion(+)

Comments

Michal Marek Nov. 20, 2012, 9:53 a.m. UTC | #1
On Mon, Oct 08, 2012 at 06:18:22PM +0200, Krzysztof Mazur wrote:
> The ncurses library allows for extended colors. The support for extended
> colors support depends on wide-character support. ncurses headers
> enable extended colors (NCURSES_EXT_COLORS) only when wide-character
> support is enabled (NCURSES_WIDECHAR).
> 
> The "make menuconfig" uses wide-character ncursesw library, which can be
> compiled with wide-character support, but does not define NCURSES_WIDECHAR
> and it's using headers without wide-character (and extended colors) support.
> 
> This fixes problems with colors on systems with enabled extended colors
> (like PLD Linux). Without this patch "make menuconfig" is hard to use.
> 
> Signed-off-by: Krzysztof Mazur <krzysiek@podlesie.net>


Applied to kbuild.git#kconfig, thanks.

Michal
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Krzysztof Mazur Nov. 20, 2012, 12:29 p.m. UTC | #2
On Tue, Nov 20, 2012 at 10:53:28AM +0100, Michal Marek wrote:
> On Mon, Oct 08, 2012 at 06:18:22PM +0200, Krzysztof Mazur wrote:
> > The ncurses library allows for extended colors. The support for extended
> > colors support depends on wide-character support. ncurses headers
> > enable extended colors (NCURSES_EXT_COLORS) only when wide-character
> > support is enabled (NCURSES_WIDECHAR).
> > 
> > The "make menuconfig" uses wide-character ncursesw library, which can be
> > compiled with wide-character support, but does not define NCURSES_WIDECHAR
> > and it's using headers without wide-character (and extended colors) support.
> > 
> > This fixes problems with colors on systems with enabled extended colors
> > (like PLD Linux). Without this patch "make menuconfig" is hard to use.
> > 
> > Signed-off-by: Krzysztof Mazur <krzysiek@podlesie.net>
> 
> 
> Applied to kbuild.git#kconfig, thanks.
> 
> Michal

Thanks,

Krzysiek
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/scripts/kconfig/lxdialog/check-lxdialog.sh b/scripts/kconfig/lxdialog/check-lxdialog.sh
index e3b12c0..5a70e8b 100644
--- a/scripts/kconfig/lxdialog/check-lxdialog.sh
+++ b/scripts/kconfig/lxdialog/check-lxdialog.sh
@@ -21,6 +21,7 @@  ccflags()
 {
 	if [ -f /usr/include/ncursesw/curses.h ]; then
 		echo '-I/usr/include/ncursesw -DCURSES_LOC="<ncursesw/curses.h>"'
+		echo ' -DNCURSES_WIDECHAR=1'
 	elif [ -f /usr/include/ncurses/ncurses.h ]; then
 		echo '-I/usr/include/ncurses -DCURSES_LOC="<ncurses.h>"'
 	elif [ -f /usr/include/ncurses/curses.h ]; then