Re: [PATCH 1/1] wget:fix do_configure failed

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Wed, 2013-06-19 at 10:45 +0800, Hongxu Jia wrote:
> Create a new build enviroment, build wget failed
> ...
> configure:34512: checking for libssl
> configure:34542: i586-poky-linux-gcc  -m32 -march=i586 --sysroot=/home/jiahongxu/yocto/build-20130613-qemu/tmp/sysroots/qemux86 -o conftest - O2 -pipe -g -feliminate-unused-debug-types  -Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed conftest.c -ldl  -lssl /home/jiahongxu/yocto/build-  20130613-qemu/tmp/sysroots/qemux86/lib/libcrypto.so -lz >&5
> /home/jiahongxu/yocto/build-20130613-qemu/tmp/sysroots/x86_64-linux/usr/libexec/i586-poky-linux/gcc/i586-poky-linux/4.7.2/ld: cannot find -lz collect2: error: ld returned 1 exit status
> ...
> 
> From log as we known, the reason is link zlib failed, it isn't
> explicitly in wget's DEPENDS. Add zlib to wget's DEPENDS.
> 
> [YOCTO #4749]
> 
> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
> ---
>  meta/recipes-extended/wget/wget.inc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

I'll take this but the summary of this change should be something like
"wget: Add missing DEPENDS on zlib" which explains more about what the
change does. Fixing a configure failure doesn't tell us much when looked
at in a history of changes. I'll tweak the summary this time.

Cheers,

Richard

> diff --git a/meta/recipes-extended/wget/wget.inc b/meta/recipes-extended/wget/wget.inc
> index ba37a87..de9b620 100644
> --- a/meta/recipes-extended/wget/wget.inc
> +++ b/meta/recipes-extended/wget/wget.inc
> @@ -2,7 +2,7 @@ DESCRIPTION = "A console URL download utility featuring HTTP, FTP, and more."
>  SECTION = "console/network"
>  LICENSE = "GPLv3"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
> -DEPENDS = "openssl"
> +DEPENDS = "openssl zlib"
>  
>  INC_PR = "r16"
>