From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f65.google.com (mail-wm0-f65.google.com [74.125.82.65]) by mail.openembedded.org (Postfix) with ESMTP id B8816731E3 for ; Fri, 6 May 2016 14:16:14 +0000 (UTC) Received: by mail-wm0-f65.google.com with SMTP id w143so9045141wmw.3 for ; Fri, 06 May 2016 07:16:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:date:to:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=SRmZi3QS2menWMoGcZZ0CcUVlXrNIzIrV36GHB+Uxl0=; b=TcZqyMZ4abNbFRqbWZd1eDoAayaH6fauM/AAb5q02MVG/o4SqIDnHn3xlqYc/4FrWO JW3mVmeXUgAi5lwtTHkZ7IObAfnN5uamdNrE0cOs6Nf5SJsc93EvMxIRcPML3Sbgal1m zT1xQZRwJk2XVFEH3q78MukSe/KxxS7m4tCGbnJa3rVGc1e8VNI2yCJ8Cr1yCEN0YaHT t25UDZXNvD7uTSEf41B8VwirlJJZ2khR3jBkoo+AsrScIZyr6Q7IPhgw4jybeiAoN/BZ pLGOGnKmfTl2d92eRDwDv8THLVyVuUzP86y60jQFPokivE+umlB6ZLG8KyPHG5GfpTHY I6xg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:date:to:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=SRmZi3QS2menWMoGcZZ0CcUVlXrNIzIrV36GHB+Uxl0=; b=RAwJQzLixJL7OSyQnp/4JABqWfVM0ZGC5CzkmHwvQol6of/PiUBhfnF0mT9GiBxjkK 0KVvq2+A+DgZEc/fXoUlI56/7QuxYmUy74epee+lXf87dRer6ZmI8RL8Cjp59omjeiVn T2INfb21+DJOeeROPWI95HzVYGjA5UAGXzL9fn/9vQVVSdjKq4A1Snq4VHtebV1RwQ/b F7ACu9o/kfo5LKYLJVQyEZlgp5UPER0Ls4dLx8N779JyisZcjgHSdTRsVaY0MPOsoufJ 6yxXw0U71cSpIg8d5+YY5I6CUk3atXxYdvS412nY7igl8zexgUf2Xbd4DA14xkSn62Pt +WVA== X-Gm-Message-State: AOPr4FXBXoZ6yloNZge6v9oRNF7wmybHSrHLprRdTB9KD7jlMdiXyHEtpsCvWfGzlqGKxg== X-Received: by 10.194.16.233 with SMTP id j9mr461274wjd.51.1462544174655; Fri, 06 May 2016 07:16:14 -0700 (PDT) Received: from localhost (ip-86-49-34-37.net.upcbroadband.cz. [86.49.34.37]) by smtp.gmail.com with ESMTPSA id f8sm15219956wjm.13.2016.05.06.07.16.13 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 06 May 2016 07:16:13 -0700 (PDT) From: Martin Jansa X-Google-Original-From: Martin Jansa Date: Fri, 6 May 2016 16:17:44 +0200 To: openembedded-devel@lists.openembedded.org Message-ID: <20160506141744.GE2544@jama> References: <1460682409-1786-1-git-send-email-lixin.fnst@cn.fujitsu.com> <20160427083526.GF2559@jama> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.6.1 (2016-04-27) Subject: Re: [meta-oe][PATCH] ltrace: Error Fix for ARM X-BeenThere: openembedded-devel@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Using the OpenEmbedded metadata to build Distributions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 06 May 2016 14:16:17 -0000 X-Groupsio-MsgNum: 61386 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="UnaWdueM1EBWVRzC" Content-Disposition: inline --UnaWdueM1EBWVRzC Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Apr 29, 2016 at 01:18:26AM +0000, Li, Xin wrote: >=20 >=20 > >-----Original Message----- > >From: openembedded-devel-bounces@lists.openembedded.org > >[mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of > >Martin Jansa > >Sent: Wednesday, April 27, 2016 4:35 PM > >To: openembedded-devel@lists.openembedded.org > >Subject: Re: [oe] [meta-oe][PATCH] ltrace: Error Fix for ARM > > > >On Fri, Apr 15, 2016 at 04:06:49AM +0300, Li Xin wrote: > >> 1) The error is as following: > >> root # ltrace ls > >> __libc_start_main(0xb6fbc018, 1, 0xbed8fdc4, > >0xb6fd0b88PTRACE_SINGLESTEP: Input/output error > >> 2747 couldn't continue when handling __libc_start_main (0xb6fbbde= 0) at > >0xb6fbbde0 > >> The master branch fix it. > >> > >> 2) remove unused patch: > >> --0001-ltrace-fix-gcc-5-logical-not-parentheses.patch > >> --ltrace-0.7.2-unused-typedef.patch > > > >Now I'm seeing this failure: > > > >| ../../../../git/sysdeps/linux-gnu/arm/fetch.c: In function 'arch_elf_i= nit': > >| ../../../../git/sysdeps/linux-gnu/arm/fetch.c:101:8: error: 'data' may= be used > >uninitialized in this function [-Werror=3Dmaybe-uninitialized] > >| if (elf_read_next_uleb128(data, &offset, &tag) < 0 > >| ^ > >| cc1: all warnings being treated as errors > >| make[4]: *** [fetch.lo] Error 1 > >| make[4]: *** Waiting for unfinished jobs.... > >| make[4]: Leaving directory > >`.../ltrace/1_7.3+gitAUTOINC+01b10e191e-r0/build/sysdeps/linux-gnu/arm' > > > >Possibly caused by using -Os, can you retest with -Os? >=20 > OK, And can you tell me how to test with -Os? TUNE_CCARGS +=3D "-Os" in the recipe is probably simples way to reproduce this. > Regards & Thanks. >=20 >=20 > > > >> Signed-off-by: Li Xin > >> --- > >> ...-ltrace-fix-gcc-5-logical-not-parentheses.patch | 38 -------------= ---- > >> .../ltrace/ltrace-0.7.2-unused-typedef.patch | 49 -------------= --------- > >> meta-oe/recipes-devtools/ltrace/ltrace_git.bb | 6 +-- > >> 3 files changed, 2 insertions(+), 91 deletions(-) delete mode 100644 > >> meta-oe/recipes-devtools/ltrace/ltrace/0001-ltrace-fix-gcc-5-logical-n > >> ot-parentheses.patch delete mode 100644 > >> meta-oe/recipes-devtools/ltrace/ltrace/ltrace-0.7.2-unused-typedef.pat > >> ch > >> > >> diff --git > >> a/meta-oe/recipes-devtools/ltrace/ltrace/0001-ltrace-fix-gcc-5-logical > >> -not-parentheses.patch > >> b/meta-oe/recipes-devtools/ltrace/ltrace/0001-ltrace-fix-gcc-5-logical > >> -not-parentheses.patch > >> deleted file mode 100644 > >> index 681c43d..0000000 > >> --- > >> a/meta-oe/recipes-devtools/ltrace/ltrace/0001-ltrace-fix-gcc-5-logical > >> -not-parentheses.patch > >> +++ /dev/null > >> @@ -1,38 +0,0 @@ > >> -From 876fe5680d77800426f8c4c5680a235732d722e6 Mon Sep 17 00:00:00 > >> 2001 > >> -From: Kai Kang > >> -Date: Mon, 24 Aug 2015 17:37:54 +0800 > >> -Subject: [PATCH] ltrace: fix gcc 5 logical not parentheses > >> - > >> -Upstream-Status: Pending > >> - > >> -Build ltrace with gcc 5.2, it fails with: > >> - > >> -error: logical not is only applied to the left hand side of > >> comparison -[-Werror=3Dlogical-not-parentheses] > >> - if (!need_data(data, offset, SIZE / 8) < 0) \ > >> - ^ > >> - > >> -Function need_data just return 0 on success and -1 if fail. So it is > >> ok -to just test if (need_data(data, offset, SIZE / 8) < 0). > >> - > >> -Signed-off-by: Kai Kang > >> ---- > >> - ltrace-elf.c | 2 +- > >> - 1 file changed, 1 insertion(+), 1 deletion(-) > >> - > >> -diff --git a/ltrace-elf.c b/ltrace-elf.c -index c571d2a..7fe830f > >> 100644 > >> ---- a/ltrace-elf.c > >> -+++ b/ltrace-elf.c > >> -@@ -218,7 +218,7 @@ need_data(Elf_Data *data, GElf_Xword offset, > >GElf_Xword size) > >> - int \ > >> - NAME(Elf_Data *data, GElf_Xword offset, uint##SIZE##_t *retp) \ > >> - { \ > >> -- if (!need_data(data, offset, SIZE / 8) < 0) \ > >> -+ if (need_data(data, offset, SIZE / 8) < 0) \ > >> - return -1; \ > >> - \ > >> - if (data->d_buf =3D=3D NULL) /* NODATA section */ { \ > >> --- > >> -1.9.1 > >> - > >> diff --git > >> a/meta-oe/recipes-devtools/ltrace/ltrace/ltrace-0.7.2-unused-typedef.p > >> atch > >> b/meta-oe/recipes-devtools/ltrace/ltrace/ltrace-0.7.2-unused-typedef.p > >> atch > >> deleted file mode 100644 > >> index 8f3c0ec..0000000 > >> --- > >> a/meta-oe/recipes-devtools/ltrace/ltrace/ltrace-0.7.2-unused-typedef.p > >> atch > >> +++ /dev/null > >> @@ -1,49 +0,0 @@ > >> -diff --git a/value.c b/value.c > >> -index d18db17..b98298e 100644 > >> ---- a/value.c > >> -+++ b/value.c > >> -@@ -1,6 +1,6 @@ > >> - /* > >> - * This file is part of ltrace. > >> -- * Copyright (C) 2011,2012 Petr Machata, Red Hat Inc. > >> -+ * Copyright (C) 2011,2012,2013 Petr Machata, Red Hat Inc. > >> - * > >> - * This program is free software; you can redistribute it and/or > >> - * modify it under the terms of the GNU General Public License as > >> -@@ -282,9 +282,9 @@ value_init_deref(struct value *ret_val, struct va= lue > >*valp) > >> - if (value_extract_word(valp, &l, NULL) < 0) > >> - return -1; > >> - > >> -- /* We need "long" to be long enough to hold platform > >> -+ /* We need "long" to be long enough to hold target > >> - * pointers. */ > >> -- typedef char assert__long_enough_long[-(sizeof(l) < sizeof(void *))= ]; > >> -+ assert(sizeof(l) >=3D sizeof(arch_addr_t)); > >> - > >> - value_common_init(ret_val, valp->inferior, valp, > >> - valp->type->u.ptr_info.info, 0); > >> -diff --git a/lens_default.c b/lens_default.c -index ed3d0e1..5d00814 > >> 100644 > >> ---- a/lens_default.c > >> -+++ b/lens_default.c > >> -@@ -1,6 +1,6 @@ > >> - /* > >> - * This file is part of ltrace. > >> -- * Copyright (C) 2011,2012 Petr Machata, Red Hat Inc. > >> -+ * Copyright (C) 2011,2012,2013 Petr Machata, Red Hat Inc. > >> - * Copyright (C) 1998,2004,2007,2008,2009 Juan Cespedes > >> - * Copyright (C) 2006 Ian Wienand > >> - * Copyright (C) 2006 Steve Fink > >> -@@ -342,9 +342,9 @@ format_array(FILE *stream, struct value *value, s= truct > >value_dict *arguments, > >> - struct expr_node *length, size_t maxlen, int before, > >> - const char *open, const char *close, const char *delim) > >> - { > >> -- /* We need "long" to be long enough to cover the whole address > >> -- * space. */ > >> -- typedef char assert__long_enough_long[-(sizeof(long) < sizeof(void = *))]; > >> -+ /* We need "long" to be long enough to cover the whole target > >> -+ * address space. */ > >> -+ assert(sizeof(long) >=3D sizeof(arch_addr_t)); > >> - long l; > >> - if (expr_eval_word(length, value, arguments, &l) < 0) > >> - return -1; > >> diff --git a/meta-oe/recipes-devtools/ltrace/ltrace_git.bb > >> b/meta-oe/recipes-devtools/ltrace/ltrace_git.bb > >> index d8d037e..850e8e3 100644 > >> --- a/meta-oe/recipes-devtools/ltrace/ltrace_git.bb > >> +++ b/meta-oe/recipes-devtools/ltrace/ltrace_git.bb > >> @@ -11,14 +11,12 @@ LIC_FILES_CHKSUM =3D > >"file://COPYING;md5=3Deb723b61539feef013de476e68b5c50a" > >> > >> PE =3D "1" > >> PV =3D "7.3+git${SRCPV}" > >> -SRCREV =3D "37ecc41b58be3dbdd79592a76e331b5b371e4f81" > >> +SRCREV =3D "01b10e191e99d8cb147e5a2b7da8196e0ec6fb94" > >> > >> DEPENDS =3D "elfutils" > >> RDEPENDS_${PN} =3D "elfutils" > >> -SRC_URI =3D > >"git://anonscm.debian.org/collab-maint/ltrace.git;branch=3Dltrace-0.7 \ > >> - file://ltrace-0.7.2-unused-typedef.patch \ > >> +SRC_URI =3D > >> +"git://anonscm.debian.org/collab-maint/ltrace.git;branch=3Dmaster \ > >> file://configure-allow-to-disable-selinux-support.patch \ > >> - file://0001-ltrace-fix-gcc-5-logical-not-parentheses.patch= \ > >> " > >> S =3D "${WORKDIR}/git" > >> > >> -- > >> 1.8.4.2 > >> > >> > >> > >> -- > >> _______________________________________________ > >> Openembedded-devel mailing list > >> Openembedded-devel@lists.openembedded.org > >> http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > > >-- > >Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com >=20 >=20 > --=20 > _______________________________________________ > Openembedded-devel mailing list > Openembedded-devel@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-devel --=20 Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com --UnaWdueM1EBWVRzC Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iEYEARECAAYFAlcsp4cACgkQN1Ujt2V2gBy4IACfZlBgyci+/TsuNCCVG+I7YjAo Sw8An3PqF1q+7N/JZg3op/Z50Oca7jGI =CNqq -----END PGP SIGNATURE----- --UnaWdueM1EBWVRzC--