diff mbox

[PATCHv2] libedit: really needs wchar support

Message ID 1375290755-15756-1-git-send-email-gustavo@zacarias.com.ar
State Accepted
Headers show

Commit Message

Gustavo Zacarias July 31, 2013, 5:12 p.m. UTC
wctype.h is always included, even in configure tests.
Fixes:
http://autobuild.buildroot.net/results/ed8/ed85b9d688ad8389895e3ecd7a6a18151efc328d/

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/libedit/Config.in  |  4 ++++
 package/libedit/libedit.mk | 10 +++-------
 2 files changed, 7 insertions(+), 7 deletions(-)

--
1.8.1.5

Comments

Yann E. MORIN July 31, 2013, 5:14 p.m. UTC | #1
On 2013-07-31 14:12 -0300, Gustavo Zacarias spake thusly:
> wctype.h is always included, even in configure tests.
> Fixes:
> http://autobuild.buildroot.net/results/ed8/ed85b9d688ad8389895e3ecd7a6a18151efc328d/
> 
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>

Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

> ---
>  package/libedit/Config.in  |  4 ++++
>  package/libedit/libedit.mk | 10 +++-------
>  2 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/package/libedit/Config.in b/package/libedit/Config.in
> index 4747a38..72b0cf1 100644
> --- a/package/libedit/Config.in
> +++ b/package/libedit/Config.in
> @@ -1,5 +1,6 @@
>  config BR2_PACKAGE_LIBEDIT
>  	bool "libedit"
> +	depends on BR2_USE_WCHAR
>  	select BR2_PACKAGE_NCURSES
>  	help
>  	  The editline library (from BSD) provides generic line editing
> @@ -12,3 +13,6 @@ config BR2_PACKAGE_LIBEDIT
>  	  in Buildroot as it makes it much simpler than using the upstream
>  	  sources which require pmake:
>  	  http://www.thrysoee.dk/editline/
> +
> +comment "libedit requires a toolchain with WCHAR support"
> +	depends on !BR2_USE_WCHAR
> diff --git a/package/libedit/libedit.mk b/package/libedit/libedit.mk
> index 6769fdc..0ff418a 100644
> --- a/package/libedit/libedit.mk
> +++ b/package/libedit/libedit.mk
> @@ -12,6 +12,9 @@ LIBEDIT_DEPENDENCIES    = ncurses
>  # We're patching configure.ac
>  LIBEDIT_AUTORECONF      = YES
> 
> +# Doesn't really support !wchar, but support is disabled otherwise
> +LIBEDIT_CONF_OPT += --enable-widec
> +
>  # Note: libbsd required for *vis functions, but works only with a toolchain
>  # with __progname; otherwise, some features are disabled, as if libbsd was
>  # missing entirely.
> @@ -19,11 +22,4 @@ ifeq ($(BR2_PACKAGE_LIBBSD),y)
>  LIBEDIT_DEPENDENCIES   += libbsd
>  endif
> 
> -# Wide-char support is not autodetected by configure, we have to help a bit.
> -ifeq ($(BR2_TOOLCHAIN_EXTERNAL_WCHAR),y)
> -LIBEDIT_CONF_OPT       += --enable-widec
> -else
> -LIBEDIT_CONF_OPT       += --disable-widec
> -endif
> -
>  $(eval $(autotools-package))
> --
> 1.8.1.5
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
diff mbox

Patch

diff --git a/package/libedit/Config.in b/package/libedit/Config.in
index 4747a38..72b0cf1 100644
--- a/package/libedit/Config.in
+++ b/package/libedit/Config.in
@@ -1,5 +1,6 @@ 
 config BR2_PACKAGE_LIBEDIT
 	bool "libedit"
+	depends on BR2_USE_WCHAR
 	select BR2_PACKAGE_NCURSES
 	help
 	  The editline library (from BSD) provides generic line editing
@@ -12,3 +13,6 @@  config BR2_PACKAGE_LIBEDIT
 	  in Buildroot as it makes it much simpler than using the upstream
 	  sources which require pmake:
 	  http://www.thrysoee.dk/editline/
+
+comment "libedit requires a toolchain with WCHAR support"
+	depends on !BR2_USE_WCHAR
diff --git a/package/libedit/libedit.mk b/package/libedit/libedit.mk
index 6769fdc..0ff418a 100644
--- a/package/libedit/libedit.mk
+++ b/package/libedit/libedit.mk
@@ -12,6 +12,9 @@  LIBEDIT_DEPENDENCIES    = ncurses
 # We're patching configure.ac
 LIBEDIT_AUTORECONF      = YES

+# Doesn't really support !wchar, but support is disabled otherwise
+LIBEDIT_CONF_OPT += --enable-widec
+
 # Note: libbsd required for *vis functions, but works only with a toolchain
 # with __progname; otherwise, some features are disabled, as if libbsd was
 # missing entirely.
@@ -19,11 +22,4 @@  ifeq ($(BR2_PACKAGE_LIBBSD),y)
 LIBEDIT_DEPENDENCIES   += libbsd
 endif

-# Wide-char support is not autodetected by configure, we have to help a bit.
-ifeq ($(BR2_TOOLCHAIN_EXTERNAL_WCHAR),y)
-LIBEDIT_CONF_OPT       += --enable-widec
-else
-LIBEDIT_CONF_OPT       += --disable-widec
-endif
-
 $(eval $(autotools-package))