Re: [PATCH 1/1] lrzsz: check locale.h in configure

[Saul Wold <sgw@linux.intel.com>]

On 05/29/2013 05:27 PM, jackie.huang@windriver.com wrote:
> From: Jackie Huang <jackie.huang@windriver.com>
>
> fix the build failure when using -O0 in debug build:
> lrz.c:284:13: error: 'LC_ALL' undeclared (first use in this function)
>
> Signed-off-by: Jackie Huang <jackie.huang@windriver.com>
> ---
>   .../lrzsz/lrzsz-0.12.20/lrzsz-check-locale.h.patch |   26 ++++++++++++++++++++
>   meta/recipes-bsp/lrzsz/lrzsz_0.12.20.bb            |    3 ++-
>   2 files changed, 28 insertions(+), 1 deletion(-)
>   create mode 100644 meta/recipes-bsp/lrzsz/lrzsz-0.12.20/lrzsz-check-locale.h.patch
>
> diff --git a/meta/recipes-bsp/lrzsz/lrzsz-0.12.20/lrzsz-check-locale.h.patch b/meta/recipes-bsp/lrzsz/lrzsz-0.12.20/lrzsz-check-locale.h.patch
> new file mode 100644
> index 0000000..d628a5a
> --- /dev/null
> +++ b/meta/recipes-bsp/lrzsz/lrzsz-0.12.20/lrzsz-check-locale.h.patch
> @@ -0,0 +1,26 @@
> +From 28b473c3c96682a820e292cc1be006e19aee11bc Mon Sep 17 00:00:00 2001
> +From: Jackie Huang <jackie.huang@windriver.com>
> +Date: Fri, 24 May 2013 01:07:24 -0400
> +Subject: [PATCH] lrzsz check locale.h
> +
What about an Upstream-Status: Tag??

This looks like a resonable patch to submit upstream also, please do so.

Thanks
	Sau!

> +Signed-off-by: Jackie Huang <jackie.huang@windriver.com>
> +---
> + configure.in |    2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/configure.in b/configure.in
> +index 6e2064c..c10460c 100644
> +--- a/configure.in
> ++++ b/configure.in
> +@@ -130,7 +130,7 @@ AC_CHECK_HEADERS(fcntl.h limits.h sys/ioctl.h sys/time.h unistd.h sys/times.h)
> + LRZSZ_HEADERS_TERM_IO
> + AC_CHECK_HEADERS(termios.h sys/termios.h termio.h sys/termio.h sgtty.h)
> + AC_CHECK_HEADERS(sys/mman.h utime.h syslog.h sys/syslog.h sys/param.h)
> +-AC_CHECK_HEADERS(sys/select.h strings.h arpa/inet.h)
> ++AC_CHECK_HEADERS(sys/select.h strings.h locale.h arpa/inet.h)
> +
> + dnl Checks for typedefs, structures, and compiler characteristics.
> + AC_TYPE_SIZE_T
> +--
> +1.7.9.5
> +
> diff --git a/meta/recipes-bsp/lrzsz/lrzsz_0.12.20.bb b/meta/recipes-bsp/lrzsz/lrzsz_0.12.20.bb
> index 83a75e3..38ce493 100644
> --- a/meta/recipes-bsp/lrzsz/lrzsz_0.12.20.bb
> +++ b/meta/recipes-bsp/lrzsz/lrzsz_0.12.20.bb
> @@ -10,13 +10,14 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
>   			file://src/lrz.c;beginline=1;endline=10;md5=5276956373ff7d8758837f6399a1045f"
>   SECTION = "console/network"
>   DEPENDS = ""
> -PR = "r5"
> +PR = "r6"
>
>   SRC_URI = "http://www.ohse.de/uwe/releases/lrzsz-${PV}.tar.gz \
>   	   file://autotools.patch \
>   	   file://makefile.patch \
>   	   file://gettext.patch \
>   	   file://lrzsz_fix_for_automake-1.12.patch \
> +	   file://lrzsz-check-locale.h.patch \
>              "
>
>   SRC_URI[md5sum] = "b5ce6a74abc9b9eb2af94dffdfd372a4"
>