From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50551) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bpAw2-000840-A7 for qemu-devel@nongnu.org; Wed, 28 Sep 2016 05:13:53 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bpAvx-0003gr-1q for qemu-devel@nongnu.org; Wed, 28 Sep 2016 05:13:50 -0400 Received: from mx1.redhat.com ([209.132.183.28]:36464) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bpAvw-0003gf-RU for qemu-devel@nongnu.org; Wed, 28 Sep 2016 05:13:44 -0400 Date: Wed, 28 Sep 2016 10:13:41 +0100 From: "Daniel P. Berrange" Message-ID: <20160928091341.GG21583@redhat.com> Reply-To: "Daniel P. Berrange" References: <1474677379-344832-1-git-send-email-arei.gonglei@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1474677379-344832-1-git-send-email-arei.gonglei@huawei.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v3] qtest: fix make check complaint in crypto module List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gonglei Cc: qemu-devel@nongnu.org, wu.wubin@huawei.com On Sat, Sep 24, 2016 at 08:36:19AM +0800, Gonglei wrote: > CC tests/test-crypto-tlscredsx509.o > CC tests/crypto-tls-x509-helpers.o > CC tests/pkix_asn1_tab.o > tests/pkix_asn1_tab.c:7:22: warning: libtasn1.h: No such file or direct= ory > tests/pkix_asn1_tab.c:9: error: expected =E2=80=98=3D=E2=80=99, =E2=80=98= ,=E2=80=99, =E2=80=98;=E2=80=99, =E2=80=98asm=E2=80=99 or =E2=80=98__attr= ibute__=E2=80=99 before =E2=80=98pkix_asn1_tab=E2=80=99 > make: *** [tests/pkix_asn1_tab.o] Error 1 >=20 > Signed-off-by: Gonglei > --- > v3: fix an error: "LIBGNUTLS_VERSION_NUMBER" is not defined (Daniel) > v2: add condition check for TLS support (Daniel) >=20 > tests/pkix_asn1_tab.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) >=20 > diff --git a/tests/pkix_asn1_tab.c b/tests/pkix_asn1_tab.c > index 903bc02..f15fc51 100644 > --- a/tests/pkix_asn1_tab.c > +++ b/tests/pkix_asn1_tab.c > @@ -4,7 +4,9 @@ > */ > =20 > #include "qemu/osdep.h" > -#include > +#include "tests/crypto-tls-x509-helpers.h" > + > +#ifdef QCRYPTO_HAVE_TLS_TEST_SUPPORT > =20 > const ASN1_ARRAY_TYPE pkix_asn1_tab[] =3D { > {"PKIX1", 536875024, 0}, > @@ -1103,3 +1105,4 @@ const ASN1_ARRAY_TYPE pkix_asn1_tab[] =3D { > {0, 1048586, "2"}, > {0, 0, 0} > }; > +#endif /* QCRYPTO_HAVE_TLS_TEST_SUPPORT */ Thanks, I've queued this patch. Regards, Daniel --=20 |: http://berrange.com -o- http://www.flickr.com/photos/dberrange= / :| |: http://libvirt.org -o- http://virt-manager.or= g :| |: http://entangle-photo.org -o- http://search.cpan.org/~danberr= / :|